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

remove hypersuperposition #401

Merged
merged 1 commit into from
Jul 12, 2023
Merged

Conversation

MichaelRawson
Copy link
Contributor

See #157. Inferences::HyperSuperposition does not appear in schedules and nobody seems to think it is used. Please object if you do want/need this.

@quickbeam123
Copy link
Collaborator

Hmm, I just tested it over the 870 FOF 2020+2021 problems on top of casc mode getting 6 unique solutions problems (casc mode 776, casc+hs-on 746). Maybe we could just test it and incorporate in schedules next years?

@MichaelRawson
Copy link
Contributor Author

OK - it looks a little unloved so maybe it doesn't work for e.g. polymorphic problems, but happy to leave it for now. Thanks for testing!

@MichaelRawson
Copy link
Contributor Author

Reopened at the request of @quickbeam123.

@MichaelRawson MichaelRawson reopened this Aug 17, 2022
@MichaelRawson MichaelRawson force-pushed the michael-remove-hyper-superposition branch from 60e1987 to d31a0d8 Compare August 19, 2022 10:14
@MichaelRawson MichaelRawson force-pushed the michael-remove-hyper-superposition branch from d31a0d8 to 20b7af5 Compare July 12, 2023 08:40
@MichaelRawson
Copy link
Contributor Author

Rebased onto master. Post-competition, do we have any evidence that hypersuperposition is useful? Paging @quickbeam123 @easychair.

@quickbeam123
Copy link
Collaborator

I think vampire can live without hypersuperposition. I don't think I ever tested it, so it's potentially dangerous. So unless @easychair remembers what it was and feels some regret, I think it should go.

@easychair
Copy link
Contributor

easychair commented Jul 12, 2023 via email

@MichaelRawson
Copy link
Contributor Author

@easychair - this one's hyper-superposition - I think hyperresolution is already gone or never existed. Thanks for confirmation anyway, axing hypersuperposition.

@MichaelRawson MichaelRawson merged commit 4db16ba into master Jul 12, 2023
1 check passed
@MichaelRawson MichaelRawson deleted the michael-remove-hyper-superposition branch July 12, 2023 14:21
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