Opened 12 years ago
Closed 12 years ago
#8468 closed enhancement (fixed)
add "Group Theory and Sage: A Primer" to the classification "Thematic Tutorials"
Reported by: | mvngu | Owned by: | mvngu |
---|---|---|---|
Priority: | major | Milestone: | sage-4.4.1 |
Component: | documentation | Keywords: | group theory |
Cc: | rbeezer | Merged in: | sage-4.4.1.alpha3 |
Authors: | Rob Beezer | Reviewers: | David Joyner, Minh Van Nguyen |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Add the document Group Theory and Sage: A Primer to the documentation category "Thematic Tutorials". The original proposal can be found on sage-devel and sage-combinat-devel.
Notes: The current ticket needs to be coordinated with #8470.
Prerequisites: #8469
Attachments (3)
Change History (15)
comment:1 Changed 12 years ago by
- Description modified (diff)
comment:2 Changed 12 years ago by
- Description modified (diff)
- Summary changed from add "Group Theory and Sage: A Primer" to the classification "Sage HOWTOs" to add "Group Theory and Sage: A Primer" to the classification "Thematic Tutorials"
Changed 12 years ago by
Changed 12 years ago by
comment:3 Changed 12 years ago by
- Status changed from new to needs_review
comment:4 Changed 12 years ago by
- Status changed from needs_review to positive_review
Looks good to me.
Thanks Rob!
comment:5 Changed 12 years ago by
- Description modified (diff)
- Reviewers set to David Joyner, Minh Van Nguyen
- Status changed from positive_review to needs_work
The attachment trac_8468-groups.patch is a ReSTified version of Rob's group theory primer. This patch depends on ticket #8469. So only the patch needs review by anyone but me.
comment:6 Changed 12 years ago by
- Status changed from needs_work to needs_review
comment:7 Changed 12 years ago by
Do the new review require more than just looking at http://sage.math.washington.edu/home/mvngu/8470-newdoc/thematic_tutorials/group_theory.html and checking if it looks okay?
It looks okay to me by maybe Rob Beezer is a better judge since he wrote the original version.
comment:8 Changed 12 years ago by
One point is that it would be nice if the primer showed how to construct character tables. (I don't think that should be a prerequisite for positive review.)
comment:9 follow-up: ↓ 10 Changed 12 years ago by
Minh - the HTML version looks very nice. Thanks for your work on this. As David suggests, I'll do a more careful once-over in the next couple days (ie this is not a review - just a promise to do so).
Dan - This was written to accompany an undergraduate course, where we don't come close to character tables. But it would be a good thing to add I think, maybe under something like an "advanced topic" section so it doesn't scare off the beginners. I've not used this function much (OK, first time was 2 minutes ago). Any "gotchas" I should document besides the occurrence of primitive roots of unity?
Anything else anybody would want to see in an "advanced" section?
Rob
comment:10 in reply to: ↑ 9 Changed 12 years ago by
Replying to rbeezer:
...
Anything else anybody would want to see in an "advanced" section?
Since you ask:-) Other topics could be
- automorphism groups of various structures (graphs, designs, codes, ...)
- Cayley graphs of groups
- actions of groups on sets,
- action of groups on polynomials and invariant theory.
- applications to the Rubik's cube.
I'm not saying you *should* do this or that it really needs to be done but you asked:-)
Rob
comment:11 Changed 12 years ago by
- Status changed from needs_review to positive_review
The attachment trac_8468-groups.patch is a ReSTified version of Rob's group theory primer. This patch depends on ticket #8469. So only the patch needs review by anyone but me.
To summarize the thread, the tutorial could be expanded later but that is not the purpose of this patch. I have built the html and pdf documentation. The only defect I see is long lines in the pdf file. I would suggest breaking these even though Sage output does not. However I don't think this is a prerequisite to merging the patch and I am changing status to positive review.
comment:12 Changed 12 years ago by
- Merged in set to sage-4.4.1.alpha3
- Resolution set to fixed
- Status changed from positive_review to closed
Close since this is merged due to the merging of #8823.
TeX and PDF are posted here. License should be compatible with the rest of Sage documentation now. Any updates in the future should appear at http://abstract.ups.edu