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

JOSS submission draft #361

Merged
merged 54 commits into from
Apr 11, 2023
Merged

JOSS submission draft #361

merged 54 commits into from
Apr 11, 2023

Conversation

JessicaS11
Copy link
Member

@JessicaS11 JessicaS11 commented Aug 29, 2022

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

@github-actions
Copy link

github-actions bot commented Aug 29, 2022

Binder 👈 Launch a binder notebook on this branch for commit bbe32f8

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit 7a2ee61

Binder 👈 Launch a binder notebook on this branch for commit af41e24

Binder 👈 Launch a binder notebook on this branch for commit 39cb6e2

Binder 👈 Launch a binder notebook on this branch for commit eb58e46

Binder 👈 Launch a binder notebook on this branch for commit b4fe440

Binder 👈 Launch a binder notebook on this branch for commit 45307de

Binder 👈 Launch a binder notebook on this branch for commit ec8c834

Binder 👈 Launch a binder notebook on this branch for commit 5798063

Binder 👈 Launch a binder notebook on this branch for commit cba0460

Binder 👈 Launch a binder notebook on this branch for commit 1892020

Binder 👈 Launch a binder notebook on this branch for commit fb4cb37

Binder 👈 Launch a binder notebook on this branch for commit 49ab3d4

Binder 👈 Launch a binder notebook on this branch for commit 790fc86

Binder 👈 Launch a binder notebook on this branch for commit c931fdd

Binder 👈 Launch a binder notebook on this branch for commit 6db0d76

Binder 👈 Launch a binder notebook on this branch for commit 47478f4

Binder 👈 Launch a binder notebook on this branch for commit eab8bf2

Binder 👈 Launch a binder notebook on this branch for commit 0800b8c

Binder 👈 Launch a binder notebook on this branch for commit 9cd58a9

Binder 👈 Launch a binder notebook on this branch for commit 737602f

Binder 👈 Launch a binder notebook on this branch for commit 19a03ec

Binder 👈 Launch a binder notebook on this branch for commit 2eeb45a

Binder 👈 Launch a binder notebook on this branch for commit b199419

Binder 👈 Launch a binder notebook on this branch for commit 40ad988

Binder 👈 Launch a binder notebook on this branch for commit 95faac3

Binder 👈 Launch a binder notebook on this branch for commit 2d4c612

Binder 👈 Launch a binder notebook on this branch for commit d29450c

Binder 👈 Launch a binder notebook on this branch for commit 115b47f

Binder 👈 Launch a binder notebook on this branch for commit c357432

Binder 👈 Launch a binder notebook on this branch for commit d51e88b

Binder 👈 Launch a binder notebook on this branch for commit 5cdf5d8

Binder 👈 Launch a binder notebook on this branch for commit 2d40766

Binder 👈 Launch a binder notebook on this branch for commit 88e5116

Binder 👈 Launch a binder notebook on this branch for commit 2ecdd27

Binder 👈 Launch a binder notebook on this branch for commit 5531466

Binder 👈 Launch a binder notebook on this branch for commit e6e834d

Binder 👈 Launch a binder notebook on this branch for commit 16de4be

Binder 👈 Launch a binder notebook on this branch for commit 62e284c

Binder 👈 Launch a binder notebook on this branch for commit 2abf392

Binder 👈 Launch a binder notebook on this branch for commit 7c0e306

Binder 👈 Launch a binder notebook on this branch for commit 85c88b1

Binder 👈 Launch a binder notebook on this branch for commit 7533866

Binder 👈 Launch a binder notebook on this branch for commit 2c0600b

Copy link
Member

@weiji14 weiji14 left a 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 🙂

doc/source/tracking/paper.md Outdated Show resolved Hide resolved
doc/source/tracking/paper.md Show resolved Hide resolved
JessicaS11 and others added 3 commits August 30, 2022 11:30
Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
Co-authored-by: Tyler Sutterley <tsutterl@uw.edu>
@codecov-commenter
Copy link

codecov-commenter commented Aug 30, 2022

Codecov Report

Patch and project coverage have no change.

Comparison is base (13c9e12) 59.09% compared to head (2c0600b) 59.09%.

📣 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.
📢 Do you have feedback about the report comment? Let us know in this issue.

# 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
Copy link
Member

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.

image

For example, the PDF for commit 45307de is at https://github.com/icesat2py/icepyx/actions/runs/2958374263

doc/source/tracking/paper.md Outdated Show resolved Hide resolved
doc/source/tracking/paper.md Outdated Show resolved Hide resolved
zachghiaccio and others added 7 commits September 8, 2022 16:44
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.
Copy link
Contributor

@kelseybisson kelseybisson left a 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.

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.
Copy link
Contributor

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

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.
Copy link
Contributor

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.

Copy link
Member

@weiji14 weiji14 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀🚀🚀

Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
@JessicaS11 JessicaS11 mentioned this pull request Mar 17, 2023
3 tasks
weiji14 and others added 2 commits March 21, 2023 11:31
@JessicaS11 JessicaS11 changed the base branch from development to pubs/joss April 6, 2023 14:13
@JessicaS11 JessicaS11 changed the base branch from pubs/joss to development April 6, 2023 14:14
@JessicaS11 JessicaS11 mentioned this pull request Apr 6, 2023
@JessicaS11 JessicaS11 merged commit d9a9d94 into development Apr 11, 2023
@JessicaS11 JessicaS11 deleted the joss branch April 11, 2023 20:55
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.