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

Space Germs and Milnor Algebra/Number #2860

Merged
merged 40 commits into from Sep 29, 2023

Conversation

afkafkafk13
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Sep 27, 2023

Codecov Report

Merging #2860 (469a5e6) into master (227f63f) will decrease coverage by 0.15%.
Report is 15 commits behind head on master.
The diff coverage is 63.38%.

@@            Coverage Diff             @@
##           master    #2860      +/-   ##
==========================================
- Coverage   80.67%   80.53%   -0.15%     
==========================================
  Files         456      456              
  Lines       64805    65208     +403     
==========================================
+ Hits        52282    52514     +232     
- Misses      12523    12694     +171     
Files Coverage Δ
src/Rings/mpoly-localizations.jl 76.53% <100.00%> (+0.18%) ⬆️
src/Rings/mpolyquo-localizations.jl 73.58% <100.00%> (+0.54%) ⬆️
experimental/Schemes/SpaceGerms.jl 68.43% <63.09%> (-14.90%) ⬇️

... and 25 files with indirect coverage changes

@afkafkafk13 afkafkafk13 changed the title Space Germs and Milnor Algebra/Number Space Germs and Milnor Algebra/Number WIP Sep 27, 2023
@afkafkafk13
Copy link
Collaborator Author

Remark: code-coverage goes down due to functionality which is prepared for upcoming extensions of the localization framework -- obviously, testing this is not feasable before the extension.

Copy link
Collaborator

@HechtiDerLachs HechtiDerLachs left a comment

Choose a reason for hiding this comment

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

Thanks for putting this up again. I left some comments.

experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Show resolved Hide resolved
@afkafkafk13
Copy link
Collaborator Author

@HechtiDerLachs : Thank you very much for careful reading and all the suggestions. I tried to incorporate everything.

experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
Copy link
Member

@fingolfin fingolfin left a comment

Choose a reason for hiding this comment

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

Thank you. I left a bunch of nitpick comments, only regarding style things. Don't be alarmed by the seemingly large number, they are small, and if you like, you can just go through them in the web browser and apply them by clicking

experimental/Schemes/SpaceGerms.jl Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
experimental/Schemes/SpaceGerms.jl Outdated Show resolved Hide resolved
@afkafkafk13 afkafkafk13 changed the title Space Germs and Milnor Algebra/Number WIP Space Germs and Milnor Algebra/Number Sep 29, 2023
anne and others added 23 commits September 29, 2023 18:18
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
Co-authored-by: Max Horn <max@quendi.de>
@afkafkafk13
Copy link
Collaborator Author

@fingolfin : Thank you for the beautifying of my code. That helps a lot for the future.

@afkafkafk13 afkafkafk13 enabled auto-merge (squash) September 29, 2023 21:40
@afkafkafk13 afkafkafk13 merged commit 5d5ac29 into oscar-system:master Sep 29, 2023
13 of 15 checks passed
@afkafkafk13 afkafkafk13 deleted the Milnor_revised branch March 20, 2024 19:47
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.

None yet

3 participants