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 manuscript #669

Closed
3 tasks done
tsalo opened this issue Feb 2, 2021 · 41 comments · Fixed by #693 or #815
Closed
3 tasks done

JOSS manuscript #669

tsalo opened this issue Feb 2, 2021 · 41 comments · Fixed by #693 or #815
Labels
discussion issues that still need to be discussed paused issues that are put on hold while other issues or PRs are handled

Comments

@tsalo
Copy link
Member

tsalo commented Feb 2, 2021

Please see openjournals/joss-reviews#3669

Summary

@emdupre and I have been working on a manuscript for JOSS, and we think it's almost ready for the dev team to review.

Additional Detail

One pending question is how we want to handle the authorship order. @emdupre and I were hoping to be co-first authors, and we discussed having Dan as the last author, but we weren't sure what the order should be from there. I think two good options are (1) alphabetical or (2) sorted by commits/PRs, then alphabetical for non-code contributors.

  1. DuPre*, Salo*, Caballero-Gaudes, Dowdle, Heunis, Kundu, Markello, Markiewicz, Maullin-Sapey, Moia, Staden, Teves, Uruñuela, Vaziri-Pashkam, Whitaker, & Handwerker
  2. DuPre*, Salo*, Dowdle, Teves, Markello, Whitaker, Heunis, Uruñuela, Moia, Markiewicz, Caballero-Gaudes, Maullin-Sapey, Kundu, Staden, Vaziri-Pashkam, & Handwerker

Next Steps

  1. Determine authorship order.
  2. Finish manuscript.
  3. Coauthor review.
  4. Push manuscript to repo.
  5. Submit to JOSS.

EDIT: Just so everyone knows, the author list above is derived from our Zenodo file. For the folks who are on the OHBM abstract, but who aren't in our Zenodo file, I would ask you to respond here saying if you want to be included or not. We would love to include you! The folks who are missing from the Zenodo file are @62442katieb, @angielaird, @notZaki, and Peter Bandettini. I don't think Dr. Bandettini is on GitHub, so it would be great if someone who works with him (@jbteves or @handwerkerd?) could follow up about this.
EDIT 2: Here is the link to the draft. I have set the permissions so that anyone with the link can comment, but most folks should have edit access already. If you don't have edit access, and would like it, please email or message me and I'll add you.

Manuscript to-do list:

  • Translate to markdown.
  • Review references. Paperpile is not perfect and we know at least a couple of references are out-of-date or incorrect.
  • Open pull request to main.
@tsalo tsalo added the discussion issues that still need to be discussed label Feb 2, 2021
@eurunuela
Copy link
Collaborator

Thank you for the work you're putting into this!

I'm happy with any of the two ordering possibilities and looking forward to reading the manuscript.

@jbteves
Copy link
Collaborator

jbteves commented Feb 3, 2021

Thanks @tsalo and @emdupre !
@handwerkerd has emailed Peter, so we'll keep you posted on that front.
I would opt for alphabetical. I don't think that either will really be a reflection of work put in (number of PRs != total productive output IMO), and alphabetical will be a little more clear for outsiders who the primary and supervisory authors are.

@CesarCaballeroGaudes
Copy link
Contributor

Thank you for this work!! I am also happy with any of the proposed ordering.

@handwerkerd
Copy link
Member

If others are ok with it, I'm happy to be senior author and fine with either option for middle authors.

I was initially slightly leaning towards sorting by commits, as a very flawed, but clearly applicable metric of contribution amount. That said, it's really just a snapshot of contributions on the day the manuscript is submitted, so it feels not particularly clear either. I'll leave it to the middle authors - particularly the 3 who are highest by commits after Elizabeth & Taylor: ( @jbteves @dowdlelt @rmarkello ) to voice a preference.

@rmarkello
Copy link
Member

Hey all 👋 ! Thanks for doing all this—excited to see tedana turned into a JOSS paper! 🙌

In response to @handwerkerd's query: I'm personally in favor of purely alphabetical, as I don't love the distinction between "code" / "non-code" contributions. In my mind one of the primary goals of tedana was / is to foster a community around multi-echo imaging (tools, resources, data, etc.), and differentiating contributions to that community based on perceived technical merit seems mildly antithetical to that goal. That said, since I'm not very involved in tedana anymore, I'm happy to defer to those who are more active community members if they have a strong preference.

@dowdlelt
Copy link
Collaborator

dowdlelt commented Feb 3, 2021

Either way is fine, I have no issues with alphabetical. Thanks so much for the work on it folks - I'm happy to read over, revise, and such - however I can help.

@tsalo
Copy link
Member Author

tsalo commented Feb 3, 2021

Thanks to everyone for responding. It looks like most folks are fine with either approach, but there's a slight preference for alphabetical. I'll move forward with that, and if anyone has strong objections we can revisit.

I have offline confirmations from @angielaird, @62442katieb, and Peter Bandettini to include them as coauthors, so I will open a pull request to add them to the Zenodo file.

@notZaki
Copy link
Contributor

notZaki commented Feb 3, 2021

Thanks for putting the draft together.
Either author order works for me. I am just happy to included.

@tsalo
Copy link
Member Author

tsalo commented Feb 10, 2021

@emdupre and I were thinking of requesting feedback by the end of the month, after which I can convert to markdown and open a PR to the repository. I will add the deadline to the tedana calendar. Does anyone have an issue with this deadline?

@tsalo
Copy link
Member Author

tsalo commented Mar 9, 2021

Addressed by #693. I will submit in about an hour. Closing this now.

@tsalo tsalo closed this as completed Mar 9, 2021
@tsalo
Copy link
Member Author

tsalo commented Mar 14, 2021

I realized we should probably keep an issue open to discuss the review process within the project, so I'm reopening this. Also, the pre-review has begun with openjournals/joss-reviews#3103.

@tsalo tsalo reopened this Mar 14, 2021
@tsalo tsalo pinned this issue Mar 14, 2021
@tsalo
Copy link
Member Author

tsalo commented Mar 14, 2021

Does anyone have thoughts on reviewers? Most of the folks I'd ask to review a multi-echo software manuscript are on this one, but I'm sure there are some folks out there who I'm missing.

@emdupre
Copy link
Member

emdupre commented Mar 14, 2021

Does anyone have thoughts on reviewers? Most of the folks I'd ask to review a multi-echo software manuscript are on this one, but I'm sure there are some folks out there who I'm missing.

I don't think they have to be multi-echo experts in particular, but someone who has significant MRI / python packaging experience would be a great fit. You can check who's agreed to review for JOSS already here, and then we'll want to filter by Python and neuroscience expertise !

@tsalo
Copy link
Member Author

tsalo commented Mar 14, 2021

I did see that somewhat overwhelming list of ~1700 people... Here are the results of an initial search for folks who mention neuroimaging/MRI in their "topic areas" and Python in their preferred or "other" languages: SRSteinkamp, richford, Athanasiamo, a3sha2, peerherholz, lukassnoek, matteomancini, dr-xenia, puolival, mwegrzyn, mih, grlee77, spinicist, sealhuang

@emdupre
Copy link
Member

emdupre commented Mar 14, 2021

Some initial ideas from the spreadsheet :

  • snastase
  • peerherholz
  • anibalsolon
  • martinagvilas
  • JonathanReardon
  • sappelhoff
  • TomDonoghue

EDIT : jinx 😆

@KirstieJane
Copy link
Member

KirstieJane commented Mar 14, 2021

There are some great names on those (very similar 😉) lists. Endorse from me ✔️

(Stefan might be a conflict of interest from a BIDS perspective but he may not be and that's for him to say no to not us!)

@tsalo
Copy link
Member Author

tsalo commented Mar 15, 2021

Merged list:

  • Athanasiamo
  • JonathanReardon
  • SRSteinkamp
  • TomDonoghue
  • a3sha2
  • anibalsolon (possible COI based on coauthored manuscript with @62442katieb)
  • arokem
  • dr-xenia (possible COI based on coauthored manuscript with @62442katieb)
  • grlee77
  • hanayik
  • lukassnoek
  • martinagvilas
  • matteomancini (possible COI based on coauthored manuscript with @62442katieb)
  • mih (coauthorship COI with @emdupre)
  • mwegrzyn
  • peerherholz (lab COI with @emdupre and possible COI based on coauthored manuscript with @62442katieb)
  • puolival
  • richford (was a TA in the ABCD-ReproNim course. maybe a COI with @tsalo and @angielaird?)
  • sappelhoff (possible COI with @tsalo and @KirstieJane based on BIDS maintainership)
  • sealhuang
  • snastase (possible COI based on coauthored manuscript with @62442katieb)
  • spinicist

@handwerkerd
Copy link
Member

I don't know many of the names listed above. Of those, I'm familiar with TomDonoghue & snastase.
Additional names:
arokem (I'm familiar with him)
hanayik (Don't know, but seems like relevant skills)

@62442katieb
Copy link

62442katieb commented Mar 15, 2021

Of that list, there are a few individuals that might have a conflict of interest (dr-xenia, matteomancini, snastase, peerherholz, anibalsolon) as we currently have coauthored publication(s) in prep/under review (the Brainhack NeuroView paper and/or the OSSIG Inclusivity in Virtual Conferences paper) neither of which are published yet. I don't know how if that counts, per JOSS's COI policy, but as @KirstieJane mentioned above it might also be up to them to decide.

@tsalo
Copy link
Member Author

tsalo commented Mar 15, 2021

Good thinking @62442katieb! Can everyone try to flag anyone with a possible COI as soon as possible? It would be great to have a list by EOD.

@handwerkerd
Copy link
Member

Oy. COIs for high authorship papers in open science are fun.
No additional conflicts of interest for me. I am a co-author on the OSSIG Inclusivity in Virtual Conferences paper and a consortium author on the Brainhack NeuroView paper. For both of those manuscripts, i don't think I directly interacted with anyone on the list. If I were a potential JOSS reviewer, I wouldn't recuse myself from peer reviewing any of their manuscripts due to COI guidelines.

@emdupre
Copy link
Member

emdupre commented Mar 15, 2021

mih would be a COI for me since we co-authored a paper together in the last 4 years; peerherholz and I are in the same lab so also a definite COI.

The bigger OS papers / initiatives I'm leaning towards letting the reviewers themselves decide if they count as a COI (or not). But that's not a strong sense -- I guess it will depend on how many reviewers we can retain after this pass ! 😄

@tsalo
Copy link
Member Author

tsalo commented Mar 15, 2021

How about arokem, martinagvilas, TomDonoghue, and Athanasiamo?

@jsheunis
Copy link
Contributor

Another option could be marcelzwiers (not on the list of existing JOSS reviewers)

@handwerkerd
Copy link
Member

martinagvilas' profile says she's a core contributor and maintainer of The Turing Way. Is there a COI with @KirstieJane

The short list is fine with me.

@tsalo
Copy link
Member Author

tsalo commented Mar 16, 2021

@emdupre and I talked a bit about this offline, and she brought up the great idea to provide a longer list to the editor. Specifically, we can eliminate anyone from that summary list with a clear COI, and anyone who has a more ambiguous one can recuse themselves in they feel it's necessary. Here's that list of 15 potential reviewers:

  • Athanasiamo
  • JonathanReardon
  • SRSteinkamp
  • TomDonoghue
  • a3sha2
  • arokem
  • grlee77
  • hanayik
  • lukassnoek
  • martinagvilas (can recuse herself if necessary)
  • mwegrzyn
  • puolival
  • sappelhoff (can recuse himself if necessary)
  • sealhuang
  • spinicist

@tsalo tsalo added the paused issues that are put on hold while other issues or PRs are handled label Aug 19, 2021
@tsalo
Copy link
Member Author

tsalo commented Aug 30, 2021

A small update- we are now under active review and have a new JOSS issue: openjournals/joss-reviews#3669.

@tsalo
Copy link
Member Author

tsalo commented Sep 28, 2021

@ME-ICA/tedana-devs the JOSS editor has requested that we make a new release to generate a new Zenodo DOI. I think we talked about making a release after our meeting earlier this month, but we just haven't gotten around to it yet. Does anyone have an issue with me releasing 0.0.11 today?

@emdupre
Copy link
Member

emdupre commented Sep 28, 2021

Are we merging in the JOSS branch before cutting the release, or should we cut a release from the JOSS branch itself ?

@jbteves
Copy link
Collaborator

jbteves commented Sep 28, 2021

I think we should merge in the JOSS branch.

@tsalo
Copy link
Member Author

tsalo commented Sep 28, 2021

+1 to merging in the JOSS branch.

@tsalo
Copy link
Member Author

tsalo commented Sep 28, 2021

I put together some release notes. If anyone has time to take a look, that would be great: https://github.com/ME-ICA/tedana/releases/tag/untagged-6eb42636f944dd6e9578

If there anything we need to do before merging in the JOSS branch?

@tsalo
Copy link
Member Author

tsalo commented Sep 29, 2021

I just want to check before we move forward with the next release. Does anyone want me to wait for them to review the release notes? Otherwise, I'd like to release by the end of today (~5pm EST).

@dowdlelt
Copy link
Collaborator

@tsalo that link gives me a 404

@jbteves
Copy link
Collaborator

jbteves commented Sep 29, 2021

It also 404s for me, and is not visible in "release notes;" last I see is from May.

@tsalo
Copy link
Member Author

tsalo commented Sep 29, 2021

Whoops! I just edited the release-drafter draft, so the May draft (0.0.11) is the one to look at.

@dowdlelt
Copy link
Collaborator

I don't see any issues, and the language seems clear. I've been a bit disconnected from the code for a bit, so maybe can't provide the most useful feedback...but I do think it looks good.

@eurunuela
Copy link
Collaborator

Looks good to me @tsalo !

@tsalo
Copy link
Member Author

tsalo commented Sep 30, 2021

Okay, releasing now.

@handwerkerd
Copy link
Member

I didn't comment earlier, but the release notes look great. It's impressive to see how much has changed since the last release, particularly knowing many were substantial changes!

@tsalo
Copy link
Member Author

tsalo commented Oct 12, 2021

It's been accepted! Now we just need a PR to add badges to our README and docs/index.rst. The code is available in openjournals/joss-reviews#3669 (comment).

@emdupre emdupre unpinned this issue Feb 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion issues that still need to be discussed paused issues that are put on hold while other issues or PRs are handled
Projects
None yet
Development

Successfully merging a pull request may close this issue.