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

Implement syzygy_generators for lists of module elements #4204

Merged

Conversation

HechtiDerLachs
Copy link
Collaborator

Address #4203 until we have more standard basis methods implemented.

Copy link

codecov bot commented Oct 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.60%. Comparing base (24c8f63) to head (da98c6e).
Report is 5 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4204   +/-   ##
=======================================
  Coverage   84.59%   84.60%           
=======================================
  Files         631      631           
  Lines       84906    84929   +23     
=======================================
+ Hits        71830    71853   +23     
  Misses      13076    13076           
Files with missing lines Coverage Δ
src/Modules/UngradedModules/FreeMod.jl 87.59% <100.00%> (+3.28%) ⬆️
src/Modules/UngradedModules/SubquoModule.jl 75.70% <100.00%> (+0.17%) ⬆️
src/Rings/groebner.jl 89.21% <ø> (-0.15%) ⬇️

@thofma
Copy link
Collaborator

thofma commented Oct 15, 2024

Did you check this this gives a different error message? As far as I see it will still crash in the default_ordering(M) call.

Copy link
Collaborator

@wdecker wdecker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx!

@HechtiDerLachs HechtiDerLachs changed the title Add informative error message for standard basis computation. Implement syzygy_generators for lists of module elements Oct 15, 2024
Co-authored-by: Tommy Hofmann <thofma@gmail.com>
@fingolfin fingolfin merged commit 780f939 into oscar-system:master Oct 17, 2024
29 checks 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.

4 participants