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

Corrected defn of deg(ideal); correction from W Decker #3412

Merged
merged 2 commits into from
Feb 22, 2024

Conversation

JohnAAbbott
Copy link
Contributor

Two corrections:

  • return type is now ZZRingElem (previously Int)
  • defn is now correct
  • no longer triggers SEGV when given zero ideal

@wdecker wdecker added the backport 1.0.x Should be backported to the release 1.0 branch label Feb 22, 2024
@wdecker wdecker marked this pull request as ready for review February 22, 2024 17:06
@wdecker wdecker merged commit 290734f into oscar-system:master Feb 22, 2024
20 checks passed
@thofma
Copy link
Collaborator

thofma commented Feb 22, 2024

Some tests for all these three issues would have been nice.

@wdecker
Copy link
Collaborator

wdecker commented Feb 22, 2024

I will add a relevant test example tomorrow.

benlorenz pushed a commit that referenced this pull request Feb 23, 2024
* Corrected defn of deg(ideal); correction from W Decker

* Removed spaces at end of lines (to force GitHub to try again)

(cherry picked from commit 290734f)
@benlorenz benlorenz mentioned this pull request Feb 23, 2024
33 tasks
@benlorenz benlorenz removed the backport 1.0.x Should be backported to the release 1.0 branch label Feb 23, 2024
wdecker added a commit that referenced this pull request Feb 23, 2024
benlorenz added a commit that referenced this pull request Feb 23, 2024
### Backported PRs

- [x] #3367
- [x] #3379 
- [x] #3369
- [x] #3291
- [x] #3325
- [x] #3350 
- [x] #3351
- [x] #3365 
- [x] #3366
- [x] #3382
- [x] #3373
- [x] #3341
- [x] #3346
- [x] #3381
- [x] #3385
- [x] #3387 
- [x] #3398 
- [x] #3399 
- [x] #3374 
- [x] #3406 
- [x] #2823
- [x] #3298
- [x] #3386 
- [x] #3412 
- [x] #3392 
- [x] #3415 
- [x] #3394
- [x] #3391
thofma pushed a commit that referenced this pull request Feb 24, 2024
* Add test example to PR #3412 as requested by @thofma
wdecker added a commit that referenced this pull request Feb 26, 2024
fieker pushed a commit that referenced this pull request Feb 27, 2024
* Add test example to PR #3412 as requested by @thofma

* docu invariants tori

* readd algebraic.md

* typo

* Update docs/doc.main

* add one more functions and corrections

* correction

* correction

* addressing review

* forgot to add corrected file

---------

Co-authored-by: Lars Göttgens <lars.goettgens@rwth-aachen.de>
benlorenz pushed a commit that referenced this pull request Feb 28, 2024
* Add test example to PR #3412 as requested by @thofma

* docu invariants tori

* readd algebraic.md

* typo

* Update docs/doc.main

* add one more functions and corrections

* correction

* correction

* addressing review

* forgot to add corrected file

---------

Co-authored-by: Lars Göttgens <lars.goettgens@rwth-aachen.de>
(cherry picked from commit 3c5f7fe)
mjrodgers pushed a commit to mjrodgers/Oscar.jl that referenced this pull request Feb 28, 2024
* Add test example to PR oscar-system#3412 as requested by @thofma

* docu invariants tori

* readd algebraic.md

* typo

* Update docs/doc.main

* add one more functions and corrections

* correction

* correction

* addressing review

* forgot to add corrected file

---------

Co-authored-by: Lars Göttgens <lars.goettgens@rwth-aachen.de>
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.

4 participants