-
Notifications
You must be signed in to change notification settings - Fork 237
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
VirtualResolutions minor update #3129
Conversation
- fixed caching bug - moved isChiH0 test above isQuasiLinear
c33f7aa
to
060e16f
Compare
@mikestillman @DanGrayson currently Is there any reason why both This PR is changing Lines 200 to 219 in 060e16f
|
691574a
to
15b8b3d
Compare
@mikestillman I added a fix for the bug from David's email here. |
Otherwise, the following example from Complexes fails: S = kk[a..e] J = ideal(a^3-e^3, a*b^2-c*e^2, a*b*c, a*b*c + a^3 - e^3) C = freeResolution(J, Strategy => 3) dd^C_1 == gens J
Quick update that fixes a few issues and adds some speedups.
One important change in Core: