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

Module quotients I #4217

Merged
merged 3 commits into from
Oct 18, 2024
Merged

Module quotients I #4217

merged 3 commits into from
Oct 18, 2024

Conversation

wdecker
Copy link
Collaborator

@wdecker wdecker commented Oct 17, 2024

No description provided.

# Examples

```jldoctest
julia> R, (x, y, z) = polynomial_ring(QQ, [:x, :y, :z]);
Copy link
Member

Choose a reason for hiding this comment

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

Just FYI these days you could also write

Suggested change
julia> R, (x, y, z) = polynomial_ring(QQ, [:x, :y, :z]);
julia> R = @polynomial_ring(QQ, [:x, :y, :z]);

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thx, the next time ...

Copy link

codecov bot commented Oct 17, 2024

Codecov Report

Attention: Patch coverage is 96.15385% with 1 line in your changes missing coverage. Please review.

Project coverage is 84.60%. Comparing base (8451da1) to head (a83ef7d).
Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
src/Modules/UngradedModules/SubquoModule.jl 96.15% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4217   +/-   ##
=======================================
  Coverage   84.60%   84.60%           
=======================================
  Files         631      631           
  Lines       84915    84954   +39     
=======================================
+ Hits        71842    71878   +36     
- Misses      13073    13076    +3     
Files with missing lines Coverage Δ
experimental/Schemes/src/ProjectiveModules.jl 83.24% <ø> (-0.92%) ⬇️
src/Modules/UngradedModules/SubquoModule.jl 77.41% <96.15%> (+1.88%) ⬆️

... and 4 files with indirect coverage changes

@wdecker wdecker marked this pull request as draft October 17, 2024 19:36
@wdecker wdecker marked this pull request as ready for review October 17, 2024 21:19
@wdecker
Copy link
Collaborator Author

wdecker commented Oct 18, 2024

@thofma The failing test here signals a problem in number theory. Any idea?

@thofma
Copy link
Collaborator

thofma commented Oct 18, 2024

Yes, this is #4170 can be ignored.

@wdecker
Copy link
Collaborator Author

wdecker commented Oct 18, 2024

@HechtiDerLachs Can you please check whether you agree and if so, merge this PR. In the next PR I wish to do quotients of type M:J, M module, J ideal. Do you have already a generic method for this? Can we reasonably reduce this to annihilator?

@HechtiDerLachs
Copy link
Collaborator

In the next PR I wish to do quotients of type M:J, M module, J ideal. Do you have already a generic method for this?

No, I don't think I or anyone else has implemented a generic method for this, yet.

Can we reasonably reduce this to annihilator?

I don't see this straight away. There are certainly generic implementations of this. But already my implementation for annihilator was probably not the most effective one. I just wanted to have something working at the time. Maybe it's best to implement this as a kernel of an appropriate map M \to (F/M)^k where M \subset F is a submodule of a free module, k is the number of generators of J and the j-th component of the map is multiplication with that generator?

As you said the other day, we should wrap the established Singular routine for the polynomial case.

@HechtiDerLachs HechtiDerLachs merged commit 6385e60 into master Oct 18, 2024
29 of 30 checks passed
@HechtiDerLachs HechtiDerLachs deleted the Wolfram branch October 18, 2024 15:41
HechtiDerLachs pushed a commit to HechtiDerLachs/Oscar.jl that referenced this pull request Oct 30, 2024
Implement quotients `I:J` for subquotients `I` and `J` and bind in the annihilator methods from Singular.
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.

5 participants