-
Notifications
You must be signed in to change notification settings - Fork 120
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
use stored OD values for factor groups #3248
use stored OD values for factor groups #3248
Conversation
and set obvious `orthogonal_discriminants` values for perfect central extensions
res[mp[l[2]]] = l[4] | ||
end | ||
# Set values for faithful characters if applicable. | ||
perf = (length(filter(i -> degree(tbl[i]) == 1, 1:length(tbl))) == 1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perf = (length(filter(i -> degree(tbl[i]) == 1, 1:length(tbl))) == 1) | |
perf = (count(i -> degree(tbl[i]) == 1, 1:length(tbl)) == 1) |
or even nicer:
perf = (length(filter(i -> degree(tbl[i]) == 1, 1:length(tbl))) == 1) | |
perf = (count(x -> degree(x) == 1, tbl) == 1) |
for i in 1:length(tbl) | ||
chi = tbl[i] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for i in 1:length(tbl) | |
chi = tbl[i] | |
for chi in tbl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I need i
in the assignments.
if mod(deg, 4 ) == 0 | ||
res[i] = "1" | ||
elseif mod(deg, 2 ) == 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if mod(deg, 4 ) == 0 | |
res[i] = "1" | |
elseif mod(deg, 2 ) == 0 | |
if mod(deg, 4) == 0 | |
res[i] = "1" | |
elseif mod(deg, 2) == 0 |
or
if mod(deg, 4 ) == 0 | |
res[i] = "1" | |
elseif mod(deg, 2 ) == 0 | |
if mod(deg, 4) == 0 | |
res[i] = "1" | |
elseif mod(deg, 2) == 0 |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #3248 +/- ##
==========================================
+ Coverage 81.63% 81.66% +0.02%
==========================================
Files 546 546
Lines 73159 73146 -13
==========================================
+ Hits 59726 59732 +6
+ Misses 13433 13414 -19
|
... and set obvious
orthogonal_discriminants
values automatically for perfect central extensionsThis way, the character table display with ODs works automatically for "2.A8", "2.HS", etc.