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.

Sign up for GitHub

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

Jump to bottom

Update text description and links for reproducible manuscript / code etc. #17

Closed
8 tasks done
egouldo opened this issue Apr 25, 2024 · 3 comments · Fixed by #39
Closed
8 tasks done

Update text description and links for reproducible manuscript / code etc. #17

egouldo opened this issue Apr 25, 2024 · 3 comments · Fixed by #39
Assignees
@egouldo
Labels
documentation
Milestone
Respond Peer Revi...

Comments

@egouldo
Copy link
Owner

egouldo commented Apr 25, 2024 •
edited
Loading

  • The data and code to reproduce the results in the manuscript is provided in their R package (hosted at Zenodo and GitHub account of first author) called ManyEcoEvo.
  • Add link to Zenodo
  • Add link to GH
  • provide the link (https://github.com/egouldo/ManyAnalysts/tree/main) (not just to the rendered HTML manuscript at https://egouldo.github.io/ManyAnalysts/, which I note is in the supplementary file)
  • cite the software they used for writing their manuscript in their references, i.e. Quarto, gt and ggplot2
  • The formatting in HTML manuscript was more pleasurable and it would have been nice to have been alluded to this earlier rather than discovering it later
@egouldo egouldo added this to the Respond Peer Review Comments milestone Apr 25, 2024
@egouldo egouldo self-assigned this Apr 25, 2024
egouldo added a commit that referenced this issue May 23, 2024
@egouldo
#17 describe data analysis pipeline and refs
c74cedd 
- describe data pipeline and location in ManyEcoEvo
- cite ManyEcoEvo and add gh repo link
- cite and describe production of manuscript with quarto
- list repo and rendered manuscript links
@egouldo
Copy link
Owner Author

egouldo commented May 23, 2024

egouldo added a commit that referenced this issue May 23, 2024
@egouldo
#17 add .bib using grateful
a2d48da
egouldo added a commit that referenced this issue May 23, 2024
@egouldo
#17 include ref to pkg-bib in YAML header
5f91087
egouldo added a commit that referenced this issue May 23, 2024
@egouldo
#17 cite pkg refs using grateful
9f4ddc7
egouldo added a commit that referenced this issue May 23, 2024
@egouldo
#17 cite all directly loaded pkgs
86b2056
@egouldo egouldo mentioned this issue May 23, 2024
Merged
egouldo added a commit that referenced this issue May 23, 2024
@egouldo
Merge pull request #19 from egouldo/grateful-pkg-refs
3874bd9 
Cite all pkgs used #17
@egouldo
Copy link
Owner Author

egouldo commented May 23, 2024 •
edited
Loading

Note that some pkg references cited from ms/references.bib are now duplicated

  • citation in text should be switched to grateful-bib
  • Rm corresponding entry from ms/references.bib
  • Ensure no missing refs in grateful-refs.bib

@egouldo
Copy link
Owner Author

egouldo commented May 24, 2024 •
edited
Loading

  • Fix formatting errors of table citations
  • move table caption above table not below

egouldo added a commit that referenced this issue May 24, 2024
@egouldo
#17 fix package referencing
ef1c317 
- rm additional citations to refs that had been removed
- don't echo the code for the output
- change ref from baselga to betapart
- delete baselga from ms refs now that is in grateful refs
egouldo added a commit that referenced this issue May 24, 2024
@egouldo
#17 update .bibs
cad16c7 
- fix formatting in grateful refs
- rm metafor ref from manuscript refs
egouldo added a commit that referenced this issue May 24, 2024
@egouldo
#17 update package referencing
6879caf 
- rm duplicate bibtex refs
- update citations to new bib ref
- fix formatting of reference entries
egouldo added a commit that referenced this issue May 24, 2024
@egouldo
#17 update table caption
ee44c01 
- direct readers to ManyEcoEvo package for futher refs
@egouldo egouldo mentioned this issue Jul 24, 2024
Merged
@egouldo egouldo linked a pull request Jul 24, 2024 that will close this issue
Address major reviewer changes #39
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@egouldo egouldo

Labels
documentation
Projects
None yet
Milestone
Respond Peer Review Comments
Development

Successfully merging a pull request may close this issue.

Address major reviewer changes egouldo/ManyAnalysts
1 participant
@egouldo