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

fix equality for ZZGenus and do not copy symbols #1329

Merged
merged 4 commits into from
Dec 27, 2023

Conversation

simonbrandhorst
Copy link
Collaborator

No description provided.

@StevellM
Copy link
Collaborator

I know where the code is breaking in Oscar, I can make a short PR so that CI tests hopefully do not fail

Copy link

codecov bot commented Dec 19, 2023

Codecov Report

Merging #1329 (8e4640f) into master (8563569) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1329      +/-   ##
==========================================
+ Coverage   74.66%   74.68%   +0.02%     
==========================================
  Files         346      346              
  Lines      112094   112088       -6     
==========================================
+ Hits        83692    83718      +26     
+ Misses      28402    28370      -32     
Files Coverage Δ
src/QuadForm/Quad/ZGenus.jl 93.68% <100.00%> (-0.03%) ⬇️

... and 31 files with indirect coverage changes

@simonbrandhorst simonbrandhorst enabled auto-merge (squash) December 22, 2023 08:42
@thofma
Copy link
Owner

thofma commented Dec 22, 2023

I just restarted the Oscar tests, and they are still failing. Any clue @StevellM?

@thofma thofma force-pushed the sb/fix_genus_equality branch from 26e6594 to 8e4640f Compare December 24, 2023 09:30
@StevellM
Copy link
Collaborator

Yes after testing it seems like copy is not enough on the fix in Oscar, we need a deepcopy. I will make a new change again.

@thofma
Copy link
Owner

thofma commented Dec 27, 2023

Merci, I restarted CI

@simonbrandhorst simonbrandhorst merged commit d95b1f0 into master Dec 27, 2023
19 checks passed
@simonbrandhorst simonbrandhorst deleted the sb/fix_genus_equality branch December 27, 2023 21:52
thofma pushed a commit that referenced this pull request Jan 5, 2024
* fix equality for ZZGenus and do not copy symbols
---------

Co-authored-by: Stevell Muller <78619134+StevellM@users.noreply.github.com>
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