-
Notifications
You must be signed in to change notification settings - Fork 80
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
[MRG] update JOSS paper for v4 #1361
Conversation
Codecov Report
@@ Coverage Diff @@
## latest #1361 +/- ##
==========================================
+ Coverage 85.27% 92.83% +7.55%
==========================================
Files 133 104 -29
Lines 15191 12389 -2802
Branches 2614 2612 -2
==========================================
- Hits 12954 11501 -1453
+ Misses 1936 587 -1349
Partials 301 301
Flags with carried forward coverage won't be shown. Click here to find out more. see 32 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Does anyone have any comments on the text? @luizirber @bluegenes in particular. Also, I can't figure out how to get the references to work properly using https://whedon.theoj.org/, but 🤷 |
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.
Made some suggestions. Otherwise looks good to me!
Co-authored-by: Tessa Pierce Ward <[email protected]>
Co-authored-by: Tessa Pierce Ward <[email protected]>
@luizirber if there aren't any concerns on your part, I'll update the authors one more time and submit to JOSS in the next week or two. |
@ctb @luizirber I'd like to update this further for ANI (adding David and Mahmudur + some ANI text), and then submit to JOSS (after 4.4 is out). I assume there are some other authors we may also need to add or author ordering to change, given the time since this was updated - @ctb is this something you could help with or guide me through? |
Main thing remaining to be done: fix all the missing information in the authors (search for |
@bluegenes I think we should pull in |
I think the author list in pyproject.toml is the most updated (with all the ORCID): https://github.com/sourmash-bio/sourmash/blob/059d98853b0b390c9d69832f132e0812be70448c/pyproject.toml#L14L49 |
Hi coauthors, We're getting ready to submit this to JOSS! Can you please review your information and the manuscript text and provide any comments by April 21st? shortcut to files changed: https://github.com/sourmash-bio/sourmash/pull/1361/files @luizirber @mr-eyes @halexand @itsabhianant @keyabarve @ccbaumler @olgabot @brooksph @dsouzadaniel @lgautier @mahmudhera @hehouts @johnsolk @kloetzl @dkoslicki @marisalim @ricky-lim @xmnlab @taylorreiter @camillescott @druvus @standage @swamidass @connor-reid-tiffany @pranathivemuri @erikyoung85 @ctb |
Everything sounds great to me. |
just a question, I was listed without a affiliation: https://github.com/sourmash-bio/sourmash/pull/1361/files#diff-0403ef06adf405f7b310b4518bd6a3559854f54c61676f676ce9cbfee7172ab6R64 |
Thanks for catching that -- unexpected. Here's the updated version |
thanks @bluegenes ! thanks look good to me :) |
updated @marisalim @johnsolk to UC Davis affil to reflect time of work |
Thanks for including me. This is seems more like an abstract than a paper. But that isn't necessarily a problem for JOSS. My affiliation is: Associate Professor, Laboratory and Genomic Medicine, Washington University in St Louis If there is a full scientific paper written at some point, there are some additional algorithms to consider benchmarking. Please reach out to me early enough on so that I can help with the design. |
Hi @swamidass, Yes - the goal here is an updated general citation for the software. If you're interested in our larger paper efforts, please see our 2022 preprint, Lightweight compositional analysis of metagenomes with FracMinHash and minimum metagenome covers and our 2019 use cases paper, Large-scale sequence comparisons with sourmash. We would welcome any additional contributions or discussion on the issue tracker! I will add your affiliation here. thanks! updated pdf: paper.pdf |
looks good to me! |
Looks good. Thanks! |
Looks good on my end.
…On Thu, Apr 6, 2023, 10:43 AM Mahmudur Rahman Hera ***@***.***> wrote:
Looks good. Thanks!
—
Reply to this email directly, view it on GitHub
<#1361 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHIC6DWZQEDDEHAAQSMXQDLW736ERANCNFSM4YSI5ANQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Looks good to me as well! |
paper.md
Outdated
affiliations: | ||
- name: University of California, Davis | ||
index: 1 | ||
- name: Woods Hole Oceanographic Institute |
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.
Lokos good-- but should be Institution
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.
😂 I should have caught this - happens with Scripps all the time. Will fix!
Hi @bluegenes, could we add the following (not super relevant but unfortunately still required) statement to the acknowledgements?
|
@ctb ready for review and merge Let's revisit making citation guidelines clearer after pyopensci review? |
Per openjournals/joss#423, we are planning to make a new submission to JOSS for sourmash v4.x. The first paper from 2016 is here.
Fixes #1321.
Ref #622, #444.
@@@
in paper.md YAML header