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

MNT: Add Zenodo metadata #732

Merged
merged 10 commits into from
Mar 13, 2019
Merged

MNT: Add Zenodo metadata #732

merged 10 commits into from
Mar 13, 2019

Conversation

effigies
Copy link
Member

@effigies effigies commented Mar 1, 2019

  • Since coveralls is redundant with codecov, doesn't accept multiple submissions to calculate total coverage, and isn't configured correctly to show coverage, I decided to drop it in favor of codecov.
  • Switched the badge to codecov, and added the Zenodo badge.
  • Added a .zenodo.json file to help with Zenodo metadata during releases. I've taken the affiliations and ORCIDs that I found from nipype and pybids. The prep_zenodo.py tool uses the git shortlog to order by number of commits, so I updated the .mailmap to match publication names, when I knew/found them.

@codecov-io
Copy link

codecov-io commented Mar 8, 2019

Codecov Report

Merging #732 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #732   +/-   ##
=======================================
  Coverage   89.39%   89.39%           
=======================================
  Files          93       93           
  Lines       11476    11476           
  Branches     1992     1992           
=======================================
  Hits        10259    10259           
- Misses        881      882    +1     
+ Partials      336      335    -1
Impacted Files Coverage Δ
nibabel/cmdline/parrec2nii.py 32.73% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de09d3a...1a2626b. Read the comment docs.

@effigies effigies marked this pull request as ready for review March 11, 2019 13:52
@effigies effigies changed the title MNT: Miscellaneous maintenance MNT: Add Zenodo metadata Mar 11, 2019
@effigies
Copy link
Member Author

Anybody interested in a review?

Additionally, the following contributors do not have affiliations or ORCIDs (and, in one case, a name) listed in the file: @matthew-brett @MarcCote @pauldmccarthy @chrispycheng @larsoner @grlee77 @cindeem @fmorency @moloney @MichielCottaar @jarrodmillman @jaeilepp @ilogue @rdvincent @ksubramz @raamana @embaker @soichih @bpinsard @chaselgrove @markhymers @skoudoro @nno @MrBago @iannimmosmith @samuelstjean @Garyfallidis @kevlarkevin @ohinds @justbennet @jstutters @mko010 @esc @schwarty @isolovey-robarts @iglpdc @jhlegarreta @jlec @yanlend @pgervais @SyamGadde @Roosted7 @R3DDY97 @freec84

If any of you would like to add this information to the Zenodo metadata, please let me know. Also, anybody can let me know they'd prefer not to be listed as an author, and I'll filter them out.

Three contributors (Chris Burns, Ly Nguyen and Thomas Ballinger) I could not find github users for.

@matthew-brett
Copy link
Member

Chris Burns is Github user cburns

@raamana
Copy link
Contributor

raamana commented Mar 11, 2019

Hi Chris, my orcid ID: 0000-0003-4662-0558

Affiliation: Rotman Research Institute, Baycrest Health Sciences, Toronto, ON, Canada

https://orcid.org/0000-0003-4662-0558

@effigies
Copy link
Member Author

Ah, thanks Matthew. Summoning @cburns to add ORCID/affiliation if desired.

@matthew-brett
Copy link
Member

I'm https://orcid.org/0000-0001-5500-2546 - School of Psychology, University of Birmingham, Birmingham UK.

@jstutters
Copy link
Contributor

Thanks for doing this. I'm https://orcid.org/0000-0002-9151-0844 - University College London, London UK

@yanlend
Copy link
Contributor

yanlend commented Mar 11, 2019

My ORCID is https://orcid.org/0000-0003-3242-9867 - Friedrich-Alexander-Universität Erlangen-Nürnberg, Erlangen, Germany

Add affiliation and ORCiD id for Jon Haitz Legarreta.
DOC: Add affiliation and ORCiD id for Jon Haitz Legarreta.
@MarcCote
Copy link
Contributor

MarcCote commented Mar 11, 2019

Here's mine: https://orcid.org/0000-0002-5147-7859
Affiliation: Microsoft Research, Montréal, Québec, Canada.
Past affiliation: Université de Sherbrooke, Sherbrooke, Québec, Canada

@jarrodmillman
Copy link
Member

I am https://orcid.org/0000-0002-5263-5070
Berkeley Institute for Data Science, UC Berkeley

@grlee77
Copy link
Contributor

grlee77 commented Mar 12, 2019

Mine is http://orcid.org/0000-0001-8895-2740

Deptartment of Radiology, University of Cincinnati College of Medicine, Cincinnati, OH
Deptartment of Radiology, Cincinnati Children's Hospital Medical Center, Cincinnati, OH

@iglpdc
Copy link
Contributor

iglpdc commented Mar 12, 2019

Mine is:

"affiliation": "Athinoula A. Martinos Center for Biomedical Imaging, Charlestown, MA",
"name": "Gonzalez, Ivan",
"orcid": "0000-0002-6451-6909”

Thanks!

@MichielCottaar
Copy link
Contributor

My info:
"affiliation": "Wellcome Centre for Integrative Neuroimaging, University of Oxford, UK",
"name": "Cottaar, Michiel.",
"orcid": "0000-0003-4679-7724"

@chrispycheng
Copy link
Contributor

Mine:
"affiliation": "Center for Open Neuroscience, Dartmouth College",
"name": "Chris Cheng",
"orcid": "0000-0001-9112-9464"

Thank you!

@effigies
Copy link
Member Author

Thanks for the updates, all. I'll plan to make final updates to the zenodo file and merge tonight or tomorrow, unless there's a negative review. (It'd be nice to get Coveralls to stop spamming PRs.)

@grlee77 From my reading of the JSON schema, it looks like I can only add one affiliation. I'll go with the hospital, but let me know if you prefer the med school.

@chrispycheng Your commit author name is "Christopher P. Cheng", do you prefer "Chris Cheng" for citations?

@chrispycheng
Copy link
Contributor

@effigies Sorry about the inconsistency - on second thought, let's go with Christopher P. Cheng. Thanks again!

@grlee77
Copy link
Contributor

grlee77 commented Mar 12, 2019

@grlee77 From my reading of the JSON schema, it looks like I can only add one affiliation. I'll go with the hospital, but let me know if you prefer the med school.

If you don't mind, please use the med school one. Thanks

@effigies effigies merged commit 753f89d into nipy:master Mar 13, 2019
@effigies effigies deleted the mnt/misc branch March 13, 2019 01:11
@R3DDY97
Copy link
Contributor

R3DDY97 commented Mar 13, 2019

Mine:
"affiliation": "National Institute of Mental Health and Neuro-Sciences, India",
"name": "Venkateswara Reddy R",
"orcid": "0000-0001-6817-2966"

Thank you!

@effigies
Copy link
Member Author

@R3DDY97 Zenodo wants names in "<family>, <given> [<given ...]" form. How would your name appear in that form?

@effigies effigies added this to the 2.4.0 milestone Mar 13, 2019
@R3DDY97
Copy link
Contributor

R3DDY97 commented Mar 14, 2019

@effigies

Either

"Reddam, Venkateswara Reddy"

Or

"Reddam, Venky"

Thank you

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.

None yet