Skip to content

Allow Nemo v0.50#100

Closed
lgoettgens wants to merge 1 commit intoalgebraic-solving:mainfrom
lgoettgens:lg/Nemo-0.50
Closed

Allow Nemo v0.50#100
lgoettgens wants to merge 1 commit intoalgebraic-solving:mainfrom
lgoettgens:lg/Nemo-0.50

Conversation

@lgoettgens
Copy link
Contributor

@lgoettgens lgoettgens commented May 16, 2025

@ederc could you please release a patch release containing this so we can continue with making the new Nemo available in Oscar? Thanks

There should be no breaking changes that are relevant to AlgebraicSolving. But note that the hashing of Vector{ZZRingElem} was slightly changed to be more in line with Vector{Int} (at least in the case that all entries of the vector are bounded by 2^63), so this could cause issues if some test relies on the iteration order of elements of a set or dict (as e.g. doctests printing such sets or dicts do).

@ederc
Copy link
Collaborator

ederc commented May 16, 2025

Problem is that we are just preparing a breaking release, i.e. there are already PRs included which changes the interface.

@lgoettgens
Copy link
Contributor Author

Hm, in understand, that's unfortunate. Do you have a time horizon for this new version?
Oscar triage planned the 1.4 release (that is supposed to contain the new AA and Nemo) mid of next week. If that does not match with your plans for the next AlgebraicSolving release, would it instead be possible to create a branch on the 0.8.3 tag, cherry-pick this PR here into that branch and release that as 0.8.4? This way, the breaking changes in AlgebraicSolving would be disentangled from the ones in AA and Nemo.

@ederc
Copy link
Collaborator

ederc commented May 16, 2025

Obsolete via #102. Seems like v0.9.0 is ready for Oscar v1.4.

@ederc ederc closed this May 16, 2025
@lgoettgens lgoettgens deleted the lg/Nemo-0.50 branch May 16, 2025 20:03
@lgoettgens
Copy link
Contributor Author

Great to hear! Thanks for your work!

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.

2 participants