-
Notifications
You must be signed in to change notification settings - Fork 107
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
JOSS submission draft #361
Conversation
👈 Launch a binder notebook on this branch for commit bbe32f8 I will automatically update this comment whenever this PR is modified 👈 Launch a binder notebook on this branch for commit 7a2ee61 👈 Launch a binder notebook on this branch for commit af41e24 👈 Launch a binder notebook on this branch for commit 39cb6e2 👈 Launch a binder notebook on this branch for commit eb58e46 👈 Launch a binder notebook on this branch for commit b4fe440 👈 Launch a binder notebook on this branch for commit 45307de 👈 Launch a binder notebook on this branch for commit ec8c834 👈 Launch a binder notebook on this branch for commit 5798063 👈 Launch a binder notebook on this branch for commit cba0460 👈 Launch a binder notebook on this branch for commit 1892020 👈 Launch a binder notebook on this branch for commit fb4cb37 👈 Launch a binder notebook on this branch for commit 49ab3d4 👈 Launch a binder notebook on this branch for commit 790fc86 👈 Launch a binder notebook on this branch for commit c931fdd 👈 Launch a binder notebook on this branch for commit 6db0d76 👈 Launch a binder notebook on this branch for commit 47478f4 👈 Launch a binder notebook on this branch for commit eab8bf2 👈 Launch a binder notebook on this branch for commit 0800b8c 👈 Launch a binder notebook on this branch for commit 9cd58a9 👈 Launch a binder notebook on this branch for commit 737602f 👈 Launch a binder notebook on this branch for commit 19a03ec 👈 Launch a binder notebook on this branch for commit 2eeb45a 👈 Launch a binder notebook on this branch for commit b199419 👈 Launch a binder notebook on this branch for commit 40ad988 👈 Launch a binder notebook on this branch for commit 95faac3 👈 Launch a binder notebook on this branch for commit 2d4c612 👈 Launch a binder notebook on this branch for commit d29450c 👈 Launch a binder notebook on this branch for commit 115b47f 👈 Launch a binder notebook on this branch for commit c357432 👈 Launch a binder notebook on this branch for commit d51e88b 👈 Launch a binder notebook on this branch for commit 5cdf5d8 👈 Launch a binder notebook on this branch for commit 2d40766 👈 Launch a binder notebook on this branch for commit 88e5116 👈 Launch a binder notebook on this branch for commit 2ecdd27 👈 Launch a binder notebook on this branch for commit 5531466 👈 Launch a binder notebook on this branch for commit e6e834d 👈 Launch a binder notebook on this branch for commit 16de4be 👈 Launch a binder notebook on this branch for commit 62e284c 👈 Launch a binder notebook on this branch for commit 2abf392 👈 Launch a binder notebook on this branch for commit 7c0e306 👈 Launch a binder notebook on this branch for commit 85c88b1 👈 Launch a binder notebook on this branch for commit 7533866 👈 Launch a binder notebook on this branch for commit 2c0600b |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Jessica for putting this together!
Just wanted to know a bit about the author ordering, is this loosely based on the number of commits in https://github.com/icesat2py/icepyx/graphs/contributors or something? I just feel like surely other people have done a lot more work on icepyx than me, and my impostor syndrome is kicking in a bit 🙂
Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com> Co-authored-by: Tyler Sutterley <tsutterl@uw.edu>
Codecov ReportPatch and project coverage have no change.
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## development #361 +/- ##
============================================
Coverage 59.09% 59.09%
============================================
Files 30 30
Lines 2264 2264
Branches 488 488
============================================
Hits 1338 1338
Misses 843 843
Partials 83 83 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
.github/workflows/draft-pdf.yml
Outdated
# This is the output path where Pandoc will write the compiled | ||
# PDF. Note, this should be the same directory as the input | ||
# paper.md | ||
path: doc/source/tracking/paper.pdf |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GitHub Actions CI has been updated so the draft paper PDF is uploaded as an artifact. Get it by going to the 'Checks' tab at https://github.com/icesat2py/icepyx/pull/361/checks, then .github/workflows/draft-pdf.yml
, and under 'Artifacts', there should be a paper
link you can click to download a zipfile containing the PDF.
For example, the PDF for commit 45307de is at https://github.com/icesat2py/icepyx/actions/runs/2958374263
Add orcid for Zheng Liu. Changed the name of affiliation #4 to University of Washington instead of the generic "Institution" to be consistent with all other UW authors.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks great to me! Simple language, easy to follow, & documents compelling functionality.
doc/source/tracking/paper.md
Outdated
The instruments provide close to 500 GB of data per day, allowing scientists to investigate the surface height of earth's features in unprecedented detail. | ||
|
||
icepyx began during the cryosphere-themed ICESat-2 Hackweek at the University of Washington in June 2019. | ||
At the event, a clear need for a collaborative, shared community space that combined and generalized the tooling (including code and examples past, present, and future) written by Hackweek participants, ICESat-2 Science Team members, and the data user community. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My 2c is I like the 'past, present, future' language as it's simple adn conveys the message well. I like what @weiji14 worked up here
doc/source/tracking/paper.md
Outdated
The instruments provide close to 500 GB of data per day, allowing scientists to investigate the surface height of earth's features in unprecedented detail. | ||
|
||
icepyx began during the cryosphere-themed ICESat-2 Hackweek at the University of Washington in June 2019. | ||
At the event, a clear need for a collaborative, shared community space that combined and generalized the tooling (including code and examples past, present, and future) written by Hackweek participants, ICESat-2 Science Team members, and the data user community. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could also say 'At the event, there was a clear need for a collaborative, shared community space that UNIFIED and generalized the tools and material written by past, present, and future Hackweek participants, ICESat-2 Science Team members, and the data user community.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀🚀🚀
Was causing issues with editorialbot at openjournals/joss-reviews#4912 (comment)
Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
added ORCID for Romina Piunno
We're working on a submission to JOSS. Please post your submission comments here!
JOSS submission at openjournals/joss-reviews#4820
Review at openjournals/joss-reviews#4912