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

Tweaks to suggested citation formatting #32

Closed
dhimmel opened this issue Aug 23, 2019 · 4 comments · Fixed by #37
Closed

Tweaks to suggested citation formatting #32

dhimmel opened this issue Aug 23, 2019 · 4 comments · Fixed by #37
Assignees

Comments

@dhimmel
Copy link
Member

dhimmel commented Aug 23, 2019

Currently, https://het.io/about/#cite looks like:

image

It would be great if we could add the additional PMID and PMCID identifiers, since they are often helpful for people wanting to cite these works. You can find all three identifiers and the suggested formatting in the manubot cite output below:

manubot cite doi:10.7554/eLife.26726 doi:10.1371/journal.pcbi.1004259 --format=html --render
<div id="refs" class="references" role="doc-bibliography">
<div id="ref-O21tn8vf">
<p>1. <strong>Systematic integration of biomedical knowledge prioritizes drugs for repurposing</strong><br />
Daniel Scott Himmelstein, Antoine Lizee, Christine Hessler, Leo Brueggeman, Sabrina L Chen, Dexter Hadley, Ari Green, Pouya Khankhanian, Sergio E Baranzini<br />
<em>eLife</em> (2017-09-22) <a href="https://doi.org/cdfk">https://doi.org/cdfk</a><br />
DOI: <a href="https://doi.org/10.7554/elife.26726">10.7554/elife.26726</a> · PMID: <a href="https://www.ncbi.nlm.nih.gov/pubmed/28936969">28936969</a> · PMCID: <a href="https://www.ncbi.nlm.nih.gov/pmc/articles/PMC5640425">PMC5640425</a></p>
</div>
<div id="ref-WkPlH1ds">
<p>2. <strong>Heterogeneous Network Edge Prediction: A Data Integration Approach to Prioritize Disease-Associated Genes</strong><br />
Daniel S. Himmelstein, Sergio E. Baranzini<br />
<em>PLOS Computational Biology</em> (2015-07-09) <a href="https://doi.org/98q">https://doi.org/98q</a><br />
DOI: <a href="https://doi.org/10.1371/journal.pcbi.1004259">10.1371/journal.pcbi.1004259</a> · PMID: <a href="https://www.ncbi.nlm.nih.gov/pubmed/26158728">26158728</a> · PMCID: <a href="https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4497619">PMC4497619</a></p>
</div>
</div>
@dhimmel
Copy link
Member Author

dhimmel commented Aug 23, 2019

I'm also thinking it may be good mention the publications at https://het.io/studies/

image

Would something like:

  • Results from Himmelstein et al (2017) eLife
  • Results from Himmelstein et al (2015) PLOS Computational Biology

help readers immediately know what the studies corresponded to?

I think we also probably want to add the citations (or abbreviated versions) to:

Since redirecting to "Cite these results" doesn't show which study it actually is.

@dhimmel
Copy link
Member Author

dhimmel commented Aug 23, 2019

Should we add title/tooltip text to "Read the Paper" with the paper title and journal info?

@vincerubinetti
Copy link
Collaborator

Since redirecting to "Cite these results" doesn't show which study it actually is.

What if we split up that "Cite" section on the "About" page into two sections for each paper, and then we can link directly to which one it is.

@dhimmel
Copy link
Member Author

dhimmel commented Jun 24, 2020

I think the solution in #37 works for me

vincerubinetti added a commit that referenced this issue Jun 24, 2020
closes #32 
closes #36 

- splits cite into two individually linkable sections
- adds PMID and PMCID to citations
- fixes "biologic class" to "biological process"
- renames some rephetio titles to be consistent
- adds some other small text
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 a pull request may close this issue.

2 participants