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

metadata: add Zenodo, tidy badges and keywords #3373

Merged
merged 7 commits into from
Feb 21, 2020

Conversation

casperdcl
Copy link
Contributor

@casperdcl casperdcl commented Feb 20, 2020

  • add Zenodo badge
    • this is a custom "all version" one which is nicer as it always points to the latest release. Yes, that sort of defeats one of the points of a DOI (version pinning) but we have version numbers for that. It's better to promote just one DOI to make it easy to track citations.
  • add formal citation for use in publications
  • add Zenodo metadata file (will control what appears in https://doi.org/10.5281/zenodo.3677553)
  • tidy and re-use badges
  • fix rst image syntax
  • ensure keywords match
    • setup.py
    • GitHib repo
    • Zenodo
  • fixes zenodo #2978
  • related readme: update badges #3233

@casperdcl casperdcl added enhancement Enhances DVC research discussion requires active participation to reach a conclusion labels Feb 20, 2020
@casperdcl casperdcl self-assigned this Feb 20, 2020
@casperdcl casperdcl changed the title Badges metadata: add Zenodo, tidy badges and keywords Feb 20, 2020
"data-science", "data-version-control", "machine-learning", "git",
"developer-tools", "reproducibility", "collaboration", "ai", "python"],
"contributors": [
{"name": "DVC team", "type": "Other", "affiliation": "Iterative"}]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure if we want name: "DVC team" or name: "Iterative" or name: "iterative.ai" or something else... Same with "affilitation".

Copy link
Member

Choose a reason for hiding this comment

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

are there example from Google or other companies? what usually people do? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fyi affiliation can be blank and will show up when name is hovered over.

name should really match whatever we choose for the citation text (i.e. think of it as an "author" field)

Comment on lines +253 to +254
Iterative, *DVC: Data Version Control - Git for Data & Models* (2020)
`DOI:10.5281/zenodo.012345 <https://doi.org/10.5281/zenodo.3677553>`_.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Again, not sure about capitalisation and exact wording

Copy link

@elleobrien elleobrien Feb 20, 2020

Choose a reason for hiding this comment

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

I think you can write, Iterative, *Data Version Control (DVC)*. No need to say "Git for Data & Models"

Copy link
Member

Choose a reason for hiding this comment

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

let's discuss this with @dmpetrov ... we need to get it right.

  • do we need to mention org at all? Iterative?
  • do we optimize for certain keywords. I would say "Git for Data" is important for us, even though Data Version Control is even more important.

but may be you are right, @andronovhopf and we should go with a clean and nice, simple version 🤔

Choose a reason for hiding this comment

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

I don't think it's terribly important to optimize for keywords in this context since Zenodo doesn't look built for discovering new projects; just tagging the repo with a DOI. not sure though.

Copy link
Contributor Author

@casperdcl casperdcl Feb 21, 2020

Choose a reason for hiding this comment

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

I think @shcheklein was referring to keywords (buzzwords) to use in our recommended citation

Choose a reason for hiding this comment

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

ah gotcha

@shcheklein
Copy link
Member

Screen Shot 2020-02-20 at 3 16 11 PM

@casperdcl is it expected? it looks too long? can we do two lines, for example?

@casperdcl
Copy link
Contributor Author

casperdcl commented Feb 21, 2020

@casperdcl is it expected? it looks too long? can we do two lines, for example?

Silly oversight on my part; fixed now.

@efiop efiop merged commit ae99e1b into iterative:master Feb 21, 2020
@casperdcl
Copy link
Contributor Author

Er @efiop I think this was still under review

@efiop
Copy link
Contributor

efiop commented Feb 21, 2020

@casperdcl Oh, sorry, is anything wrong in this PR? It got 2 approvals so I've merged it 🤔

EDIT: ok, I'm blind. Now I do see the comments. Well, if there will be any requested changes, let's just address them on top, I don't think there is any harm done here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion requires active participation to reach a conclusion enhancement Enhances DVC research
Projects
None yet
Development

Successfully merging this pull request may close these issues.

zenodo
4 participants