Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix one Lean misformalisation #217

Merged
merged 1 commit into from
Sep 25, 2024
Merged

Conversation

ocfnash
Copy link
Contributor

@ocfnash ocfnash commented Sep 24, 2024

No description provided.

The hypotheses were contradictory without the missing positivity guard
as the following succeeds without the change included here:
```
  exfalso
  have : IsQualifying S := hS.1
  rw [IsQualifying_def] at this
  obtain ⟨h₁, h₂, h₃, h₄⟩ := this
  have : -7 ∈ S := h₃ (-7) (h₄ 2 h₂)
  replace this : 0 < -7 := h₁ _ this
  linarith
```

This was previously "fixed" in c4512ca
but unfortunately an error remained.
@ocfnash
Copy link
Contributor Author

ocfnash commented Sep 24, 2024

I've just realised that I presumably have to add the informal statements for the four new problems also. I'll come back to this tomorrow.

@GeorgeTsoukalas
Copy link
Collaborator

To clarify, these are "missing" in the sense that we never formalized them, right? I don't believe we've formalized these in any of the other languages either. Presumably there was a barrier in our understanding or in the naturality of the formalization.

@ocfnash
Copy link
Contributor Author

ocfnash commented Sep 25, 2024

To clarify, these are "missing" in the sense that we never formalized them, right?

Yes, exactly.

I don't believe we've formalized these in any of the other languages either.

Agreed.

Presumably there was a barrier in our understanding or in the naturality of the formalization.

I can't comment here but they are definitely slightly awkward to state.

@GeorgeTsoukalas
Copy link
Collaborator

GeorgeTsoukalas commented Sep 25, 2024

Thanks for taking this on! Do you think you will be adding more new formalizations? I might need to add other team members to assist with reviewing, and also possibly to add corresponding Isabelle formalizations, if this is the case.

@eric-wieser
Copy link
Contributor

Perhaps in that case the fixed formalization should be split to a separate PR from the new ones?

@ocfnash ocfnash changed the title Fix one misformalisation and add four missing problems (Lean) Fix one Lean misformalisation Sep 25, 2024
@ocfnash
Copy link
Contributor Author

ocfnash commented Sep 25, 2024

Perhaps in that case the fixed formalization should be split to a separate PR from the new ones?

Agreed, done. The new statements now exist in this branch and I will create a PR if / when George indicates he is open to this (as well as after I have had a chance to add the informal statements).

@ocfnash
Copy link
Contributor Author

ocfnash commented Sep 25, 2024

Do you think you will be adding more new formalizations?

I'm not sure, for now I have other priorities but things may change. Sorry not to have a better answer.

I might need to add other team members to assist with reviewing, and also possibly to add corresponding Isabelle formalizations, if this is the case.

Totally understandable. For now I'll assume you are not open to adding new statements but I might ask you again in the near future to see if that changes (no problem if not!).

@ocfnash
Copy link
Contributor Author

ocfnash commented Sep 25, 2024

For emphasis: this PR now has just one commit with a small fix.

@GeorgeTsoukalas
Copy link
Collaborator

Totally understandable. For now I'll assume you are not open to adding new statements but I might ask you again in the near future to see if that changes (no problem if not!).

By default I am definitely open to contributions to the benchmark! Please feel free to contribute new statements as you please, I'll handle on our end coordinating who on the team can assist with reviews in the case that effort required is such that I alone cannot make the review process speedy and effective while balancing other priorities.

@GeorgeTsoukalas GeorgeTsoukalas merged commit 6c3df0b into trishullab:main Sep 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants