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

Rename CMSO dataset 1 #3

Merged
merged 1 commit into from
Jun 21, 2017
Merged

Conversation

sbesson
Copy link
Member

@sbesson sbesson commented Jun 20, 2017

Following Essen discussion, this changes the prefix of the CMSO curated datasets from cmso to cmsodataset

@sbesson sbesson changed the title Rename CMSO d ataset 1 Rename CMSO dataset 1 Jun 20, 2017
@pcmasuzzo
Copy link
Member

Good for me!

@agbeltran
Copy link
Member

Good for me too! Merging as it was agreed in the Essen meeting.

@agbeltran agbeltran merged commit de51e8d into CellMigStandOrg:master Jun 21, 2017
@simleo
Copy link
Member

simleo commented Jun 22, 2017

I'm a bit late to the party but I'd still like to leave a few notes for future reference:

  • I understand not wanting to clash with ontology terms and such, but in the long term cmsodataset might be a bit too verbose
  • dataset might suggest the presence of image data while it’s all metadata instead (this also applies to the repo name)

@sbesson
Copy link
Member Author

sbesson commented Jun 22, 2017

@simleo understood. cmsodataset was chosen as it matches the current name of this repository but we could review it if creates confusion. Do you have an alternate suggestion?

@sbesson sbesson deleted the cmso2cmsodataset branch June 22, 2017 13:43
@simleo
Copy link
Member

simleo commented Jun 22, 2017

Perhaps cmsomd, where md stands for metadata? Although I would give another chance to cmso if the risk of ID clashes is not too high. An ontology term should look like CMSO:0001, right? That should be easily distinguishable from cmso0001.

Again, sorry for chiming in late but I was attending the MULTIMOT meeting while this was being merged :)

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.

4 participants