-
-
Notifications
You must be signed in to change notification settings - Fork 488
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
Thematic tutorials: using the notebook, programming python, comprehensions #14090
Comments
comment:1
Sébastien: can you have a look at the main index page? We had discussed it together last time I was in Montreal. |
comment:2
I think that intersphinx linking to the reference manual should be fixed by #6495. See also a sage-support thread. |
comment:3
By the way, was the |
This comment has been minimized.
This comment has been minimized.
comment:5
Replying to @jhpalmieri:
Ouch, thanks for catching this. The sandpile document should definitely be there. Where would you put it? In the combinatorics section maybe? Cheers, |
comment:6
Replying to @jhpalmieri:
That would be excellent news to have this feature back shortly! (not counting on the exciting parallel feature!
|
Attachment: Image 27.png |
comment:9
Replying to @nthiery:
Yes, I remember vaguely but I can't remember exactly... Maybe, it was a bigger patch no? with more sections? Anyway, I looked at it and I think the sections and their order are fine.
|
comment:10
I get the following 16 warning when building the html :
|
comment:11
Hi Sébastien! Replying to @seblabbe:
Thanks! This is fixed in the updated patch. I also fixed the missing
Yes. That's what #14091 is about (links to the reference manual). My
Yes, I am not sure how to handle this. The toctree approach is not What do you think? Cheers, |
comment:12
Replying to @seblabbe:
Ok, thanks for checking!
Yup. That's #14091
They work in the live documentation ... The right thing to do would be Cheers, |
This comment has been minimized.
This comment has been minimized.
comment:14
Actually the link to the first PREP document was wrong. I just fixed that. In waiting for a better solution, the link seem to work in the static and the live documentation ... |
comment:15
Reinstated the copyright notice I had inadvertently removed. Added proper patch header. |
comment:16
I like this a lot after only a few minutes of looking at it! What remains to be reviewed here? (I'm not sure whether most of it has positive review or not.) And this does not depend on #6495, correct? |
comment:17
Here come some comments. First, generalities about the front page.
Maybe not any more. At the time the thematic tutorials were pretty clearly thematic tutorials. Not really a lot of overlap between abelian sandpiles and an intro to doing calculus in Sage. In any case, one should keep existing links working. Maybe that would be a next step - re-organizing all information in the Sage references. Currently there is also
Some of these things have been translated, which gives additional annoyance...
I only see primers and thematic tutorials. Is this old language? Also, what is the reason for the different formatting - there is italic, regular, bold, and even tt - on the front page? With respect to the iteration tutorial, on request of nthiery höchstpersönlich! Mostly very minor English stuff.
The syntax has to be perfect. Does this work for ones with also
What happens if you try to continue the iterator now? I think that would be useful to warn folks.
Overall I like it, though! |
comment:18
Python tutorial:
|
Changed reviewer from Samuel Lelièvre, Sébastien Labbé to Samuel Lelièvre, Sébastien Labbé, Karl-Dieter Crisman |
comment:19
Front page:
The introductory tutorial:
Patchbot is giving this a green light, so I'd say fix the (mostly very minor) things I pointed out and you've got a nice upgrade to the Sage documentation on your hands! |
comment:20
Hi Karl! Thanks for the proofreading and suggestions! Replying to @kcrisman:
Link fixed. It remains good stuff!
I added a TODO at the end about those.
This is just using the standard Sphinx markup
Done.
Yeah, indeed. This stems from the fact that we usually run this tutorial by having someone gloss over the first part and then have the student work on their own on the rest.
+1. It's mentionned later on.
Done. Altogether, it could use some reorganisation and complements not perfect. However I would go for getting this in and improving it further in a later ticket! Ok with that? |
comment:21
Replying to @kcrisman:
Agreed, this is ugly. But it's about style and not content. So yes,
I just added a link from the intro to the programming tutorials.
I see the potential confusion. Yet I am strongly attached to the word
:-) This sounds nice! I'll try to have a more detailed look! For now I'll
The \ is there in the .rst file.
I think it is.
:-) Thanks again for the review! |
comment:22
Replying to @kcrisman:
Ok.
Definitely!
Yup. Do you mind opening a ticket for this?
No it's new language :-) There are a bunch of demos in the
I changed this to "idioms" :-)
Added.
I am not sure what you mean here.
Done!
Done.
Better be indeed :-) Done!
It is!
:-) |
Attachment: trac_14090-thematic_tutorials-nt.patch.gz |
Changed reviewer from Samuel Lelièvre, Sébastien Labbé, Karl-Dieter Crisman to Samuel Lelièvre, Sébastien Labbé, Karl-Dieter Crisman, Darij Grinberg |
comment:23
Status: ready to go, assuming the patchbot goes green.
Thanks everybody! |
This comment has been minimized.
This comment has been minimized.
comment:24
I have a couple minor quibbles yet but not worth mentioning as they are to some extent stylistic. HOWEVER I think you should ask Harald to reorganize the standard doc page in light of this, AND I think that somewhere in there is also the source for the Sage doc which should be reorganized in light of this. After all, that is where people will be going first, and these need to make it easy for people to get to the page organized here. |
comment:25
Replying to @kcrisman:
Further work on those tutorials in a followup ticket is welcome :-)
Yup. I just sent him an e-mail, and added him in CC. I can't wait for the patchbot to actually run the tests ... |
Merged: sage-5.8.beta3 |
comment:28
Yippee! Thanks everybody for the writing and review! That will be super handy to have all those tutorials right under hand during upcoming Sage days. Cheers, |
comment:29
We don't support "et al." as Author |
Changed author from Franco Saliola, Florent Hivert, Nicolas M. Thiéry, et al. to Franco Saliola, Florent Hivert, Nicolas M. Thiéry |
This comment has been minimized.
This comment has been minimized.
comment:30
yet another TAB removed in the desc field. |
This ticket adds three tutorials that have been developped and extensively used during Sage(-Combinat) events:
There is some redundancy with the PREP's tutorial, but that's fine:
the point of views are complementary.
At this occasion, this ticket also refactors the main Thematic
Tutorials page, grouping them by theme, and cross-linking to thematic
tutorials that are already in the Sage sources (combinatorics, padics)
or other documents (PREP).
A preview of the result is available here:
http://combinat.sagemath.org/doc/thematic_tutorials/
It might be slightly outdated w.r.t. the latest version of the
patch. Also one should ignore the SEEALSO section which is added by a
later patch in the Sage-Combinat queue.
CC: @sagetrac-sage-combinat @seblabbe @novoselt @haraldschilly
Component: documentation
Keywords: thematic tutorials
Author: Franco Saliola, Florent Hivert, Nicolas M. Thiéry
Reviewer: Samuel Lelièvre, Sébastien Labbé, Karl-Dieter Crisman, Darij Grinberg
Merged: sage-5.8.beta3
Issue created by migration from https://trac.sagemath.org/ticket/14090
The text was updated successfully, but these errors were encountered: