Skip to content
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

Remove mupad bibliography #46

Merged

Conversation

nathanncohen
Copy link
Contributor

Mupad-combinat has apparently been dead for the last 7 years.
http://mupad-combinat.sourceforge.net/

Regardless of that, Mupad is not part of Sage there is no reason why we should host its bibliography.

@haraldschilly
Copy link
Member

I'm not sure sure about that. The text is in the past tense and initially, there must have been a reason why it is listed there.

@nathanncohen
Copy link
Contributor Author

nathanncohen commented Jun 5, 2015 via email

@kcrisman
Copy link
Member

kcrisman commented Jun 5, 2015

I think it's reasonable to advertise any big piece of Sage development here, whether it's combinat or matroids or something symbolic - but I do agree that this should be updated and perhaps rearranged. Certainly all Sage publications of any kind should be advertised :)

@nathanncohen
Copy link
Contributor Author

nathanncohen commented Jun 6, 2015 via email

@nathanncohen nathanncohen force-pushed the remove_mupad_bibliography branch from a300914 to 8ec7ebb Compare June 8, 2015 08:13
@nathanncohen
Copy link
Contributor Author

Updated. With this new branch, the following modifications are made:

  • Only one link in the menu (toward library-publications.html)
  • A link from library-publications.html to the mupad page.

@nathanncohen nathanncohen force-pushed the remove_mupad_bibliography branch from 8ec7ebb to e6456fa Compare June 8, 2015 08:34
@nthiery
Copy link
Contributor

nthiery commented Jun 9, 2015

I looked at the latest commit, and it looks good to me! I am fine with having it merged.
Nathann: I just emmited a pull request going one step further (removing the link from the Sage citation list to the MuPAD citation list). I took the occasion to slightly improve the description of Sage-Combinat / MuPAD-Combinat.

@haraldschilly
Copy link
Member

Uuuh, ok. I see. So @nathanncohen has to merge this: https://github.com/nthiery/website/commit/69dbf9f47722ba0fab4de7f47d385827960319cc and then I can review this here. Cool :-)

For the record: here is my issue #1 for the publications sub-project. Maybe this here is the perfect opportunity to get something started into that direction. Everything on one page is really bad UX.

@nthiery
Copy link
Contributor

nthiery commented Jun 9, 2015

Yeah, sorry, I am still learning the best practices for this kind of things with pull requests. It's now an independent pull request (#53).

@haraldschilly haraldschilly merged commit e6456fa into sagemath:master Jun 10, 2015
@haraldschilly
Copy link
Member

ok, all set over here, too, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants