-
-
Notifications
You must be signed in to change notification settings - Fork 517
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
Update sagenb #19616
Comments
comment:2
I'll remove the dependency if it doesn't make progress, since there are several other things we would want to update anyway. |
comment:4
You'd have to ask François, but my understanding of what he said is that this is indeed necessary for that commit. |
comment:6
Note that there is a circular dependency with #10295. Can somebody please clarify? |
comment:7
Again, fbissey said that the packages need to be updated simultaneously; assuming that this ticket ends up including this commit, then this ticket would be the one. |
comment:8
OK, I understand now, thanks for the pointer. To resolve the deadlock situation: can somebody arrange for this ticket to be set to needs_review? |
comment:9
Got it, I see the problem on your side now. I can package up what we have if you are willing to review that commit (which is surely a trivial thing, I just haven't tested it). I still want to merge sagemath/sagenb#324 and sagemath/sagenb#345 but will try to set aside some time for that this afternoon (US) and then package up a sagenb update for this purpose. |
comment:10
I'm going to make a separate package with other changes for this ticket, and then a followup for #10295. Hopefully done within a couple hours (lots of other things going on today). |
comment:11
Ran into trouble with one of the updates not passing tests but I'm still going to decouple these. |
Changed dependencies from #10295 to none |
comment:13
Yes, that was my plan. |
This comment has been minimized.
This comment has been minimized.
Branch: u/kcrisman/ticket/19616 |
Commit: |
This comment has been minimized.
This comment has been minimized.
New commits:
|
Author: Karl-Dieter Crisman |
comment:17
Could you also make a sagenb version 0.11.6 for #10295? Then I could just test 0.11.6 and test both tickets at the same time. |
comment:18
I won't make a full release, in case it turns out that doesn't get merged very quickly, but I'll make a version just for you and that ticket for testing purposes only. |
comment:19
Replying to @kcrisman:
What is the difference between a "full release" and a release "for testing purposes only"? |
comment:20
The documentation doesn't build:
|
Changed branch from u/kcrisman/ticket/19616 to u/jdemeyer/ticket/19616 |
comment:44
Needs review - not a bad idea to just check things out again, hopefully it will also work well in the 6.11 betas. |
comment:45
Does this include the fix for #10295? |
comment:46
Would I have released this new one otherwise? You also could have just checked the github: |
comment:48
This commit is causing doctest failures
|
comment:49
Apart from this, the notebook seems to work. |
comment:50
Aargh!!! I knew something like this would happen. I apologize for not doing this - I thought SURELY this wouldn't change doctests and I wanted to make a package really quickly for when I wouldn't have a lot of access to Sage... well, shame on me. |
comment:51
|
comment:52
Now I remember why I didn't doctest it - because I didn't want to have to deal with checking #10295 at the same time because it would be complicated to test both... I will try to deal with this now, anyway. |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:54
Okay, I have now hopefully taken care of everything... |
This comment has been minimized.
This comment has been minimized.
comment:56
AFAIK, the notebook works (including the live doc) and doctests pass. Ready to set a positive review? |
This comment has been minimized.
This comment has been minimized.
comment:57
I did not test the latest version, but the versions before seemed to work. So for me, this is good to go. |
Changed reviewer from Jeroen Demeyer to Jeroen Demeyer, Travis Scrimshaw |
Changed branch from u/kcrisman/ticket/19616 to |
It's time again to update sagenb. Package at http://www.math-cs.gordon.edu/~kcrisman/sagenb-0.11.6.1.tar
Note there is a circular dependency with #10295: both tickets need to be merged together.
Depends on #19671
Depends on #10295
CC: @novoselt @kiwifb
Component: notebook
Author: Karl-Dieter Crisman
Branch/Commit:
bed31b2
Reviewer: Jeroen Demeyer, Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/19616
The text was updated successfully, but these errors were encountered: