Skip to content

Comments

Support for vector_space_dim and friends for modules over MPoly(Quo)LocRing localized at a point #27

Draft
Syz-MS wants to merge 9 commits intoHechtiDerLachs:kbase_for_modulesfrom
Syz-MS:ms/modules_over_MPoly(Quo)LocRings_as_vector_space
Draft

Support for vector_space_dim and friends for modules over MPoly(Quo)LocRing localized at a point #27
Syz-MS wants to merge 9 commits intoHechtiDerLachs:kbase_for_modulesfrom
Syz-MS:ms/modules_over_MPoly(Quo)LocRings_as_vector_space

Conversation

@Syz-MS
Copy link

@Syz-MS Syz-MS commented Feb 11, 2026

@HechtiDerLachs feel free to decide whether to include this in your current open PR or not, since it is already late and the deadline for the 1.7.0 release is approaching quickly.

To sum up the changes in this PR:

  • refactores the existing code for the ungraded MPolyLocRing case and extends it to modules with true sub part (left the old code commented out)

  • in doing so swiches to using 'present_as_cokernelinstead ofcokernel` when presenting the module, since the later one sometimes also returns the canonical projection

  • also left the code for the "total degree graded" MPolyLocRing case untouched, since there are two test in an old testset, that would break otherwise (should probably be removed)

  • copys and refactors the "base_ring_lift" -functions for modules over a MPolyQuoRing to the case of modules over aMPolyQuoLocRing

  • adds full support for ungraded modules over a MPolyQuoLocRing

@Syz-MS Syz-MS marked this pull request as draft February 12, 2026 08:06
@Syz-MS
Copy link
Author

Syz-MS commented Feb 12, 2026

Will open a new PR, since your PR is now merged

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.

1 participant