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

Make the citation links go to the DOI links #334

Closed
wd15 opened this issue Sep 19, 2014 · 5 comments
Closed

Make the citation links go to the DOI links #334

wd15 opened this issue Sep 19, 2014 · 5 comments

Comments

@wd15
Copy link
Contributor

wd15 commented Sep 19, 2014

Remove http://www.ctcms.nist.gov/fipy/documentation/refs.bib_cited.html and simply go straight to the DOI line where possible or at least provide the link alongside the listings.

Imported from trac ticket #457, created by wd15 on 08-16-2012 at 16:18, last modified: 09-11-2013 at 17:19

@guyer
Copy link
Member

guyer commented Sep 19, 2014

Linking straight to the DOI would be a mistake, as the link might not always work and then there'd be no trace of what we were referencing.

changeset:e3a63fba4f/fipy switched to [https://github.com/mcmtroffaes/sphinxcontrib-bibtex sphinxcontrib-bibtex], which relies on [http://pybtex.sourceforge.net pybtex]. The [http://bazaar.launchpad.net/~ero-sennin/pybtex/trunk/view/head:/pybtex/style/formatting/unsrt.py#L418 development version of pybtex] renders DOIs, but I can't get this to work with [https://github.com/mcmtroffaes/sphinxcontrib-bibtex sphinxcontrib-bibtex] right now: mcmtroffaes/pybtex-docutils#3.

We should keep an eye on [https://github.com/mcmtroffaes/sphinxcontrib-bibtex sphinxcontrib-bibtex] and update the documentation when this problem is fixed.

Trac comment by guyer on 09-09-2013 at 11:24

@fipymigrate
Copy link

In 4aabf41:

#CommitTicketReference repository="fipy" revision="4aabf41f64a88c223f8c058c19292a56dadb389f"
List requirements for building documentation

We have not kept up to date with what's needed for Sphinx to build the
docs.

Addresses issue #334

Trac comment by Jonathan Guyer guyer@nist.gov on 09-09-2013 at 18:21

@fipymigrate
Copy link

In 5dfdc19:

#CommitTicketReference repository="fipy" revision="5dfdc194c014695405dd2238e54d56d4495c226a"
Remove URLs redundant to DOIs

Addresses issue #334

Trac comment by Jonathan Guyer guyer@nist.gov on 09-09-2013 at 18:21

@guyer
Copy link
Member

guyer commented Sep 19, 2014

Turns out I didn't have the [http://bazaar.launchpad.net/~pybtex-devs/pybtex/trunk absolute latest of pybtex]. With everything at the bleeding edge, DOIs get rendered properly as links.

Pull request:

The following changes since commit a1e9b1e148f1a4190b042789fffbdab75adeb772:

  Document additional Sphinx package (2013-09-09 10:44:31 -0400)

are available in the git repository at:

  ssh://git@code.matforge.org/nist/fipy.git ticket457-Make_the_citation_links_go_to_the_DOI_links

for you to fetch changes up to 5dfdc194c014695405dd2238e54d56d4495c226a:

  Remove URLs redundant to DOIs (2013-09-09 18:16:03 -0400)

----------------------------------------------------------------
Jonathan Guyer (2):
List requirements for building documentation
Remove URLs redundant to DOIs

 documentation/USAGE.txt | 25 +++++++++++++++++++++++--
 documentation/refs.bib  |  6 ------
 2 files changed, 23 insertions(+), 8 deletions(-)

Trac comment by guyer on 09-09-2013 at 18:29

@fipymigrate
Copy link

In 8e96cf5:

#CommitTicketReference repository="fipy" revision="8e96cf5ab75549e4d52ace770bfbce043a75e9e0"
Merge branch 'ticket457-Make_the_citation_links_go_to_the_DOI_links' into develop

Fixed issue #334

Trac comment by Daniel Wheeler daniel.wheeler2@gmail.com on 09-11-2013 at 17:19

@wd15 wd15 closed this as completed Sep 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants