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 inefficiency when dealing with certain algebra modules #2058

Merged

Conversation

fingolfin
Copy link
Member

The issue was originally reported by Rudolf Tange on the GAP support mailing list. The fix is virtually identical to a fix for a similar issue, see 7aec45f

@fingolfin fingolfin added the kind: bug Issues describing general bugs, and PRs fixing them label Jan 1, 2018
@fingolfin
Copy link
Member Author

Note that this PR is against stable-4.9, and the fix should IMHO go into 4.9.0 or 4.9.1.

@fingolfin fingolfin force-pushed the mh/IsMonomialElementVectorSpace branch from aa2e060 to afc698a Compare January 2, 2018 23:51
@codecov
Copy link

codecov bot commented Jan 2, 2018

Codecov Report

Merging #2058 into stable-4.9 will increase coverage by 0.04%.
The diff coverage is 90.56%.

@@              Coverage Diff               @@
##           stable-4.9    #2058      +/-   ##
==============================================
+ Coverage       67.01%   67.05%   +0.04%     
==============================================
  Files             899      899              
  Lines          273517   273534      +17     
==============================================
+ Hits           183307   183430     +123     
+ Misses          90210    90104     -106
Impacted Files Coverage Δ
lib/lierep.gi 83.91% <100%> (+1.62%) ⬆️
lib/algrep.gi 73.56% <87.8%> (+4.36%) ⬆️
src/gap.c 65.17% <0%> (-0.32%) ⬇️
hpcgap/lib/hpc/stdtasks.g 38.61% <0%> (-0.26%) ⬇️
src/funcs.c 78.92% <0%> (+0.13%) ⬆️
src/hpc/traverse.c 84.16% <0%> (+0.38%) ⬆️
lib/queue.g 69.59% <0%> (+3.19%) ⬆️

@fingolfin fingolfin force-pushed the mh/IsMonomialElementVectorSpace branch 2 times, most recently from 5c9737d to 18e748e Compare January 4, 2018 09:13
The issue was originally reported by Rudolf Tange on the GAP support
mailing list. The fix is virtually identical to a fix for a similar
issue, see 7aec45f
@fingolfin fingolfin force-pushed the mh/IsMonomialElementVectorSpace branch from 18e748e to dfb8827 Compare January 4, 2018 11:27
@fingolfin fingolfin merged commit 00cc9bc into gap-system:stable-4.9 Jan 11, 2018
@fingolfin fingolfin deleted the mh/IsMonomialElementVectorSpace branch January 11, 2018 08:56
@olexandr-konovalov olexandr-konovalov added this to the GAP 4.9.0 milestone Jan 11, 2018
@olexandr-konovalov olexandr-konovalov added the release notes: added PRs introducing changes that have since been mentioned in the release notes label Jan 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: bug Issues describing general bugs, and PRs fixing them release notes: added PRs introducing changes that have since been mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants