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

EDAM terms for Circos, Jbrowse #3269

Merged
merged 5 commits into from
Nov 23, 2020
Merged

EDAM terms for Circos, Jbrowse #3269

merged 5 commits into from
Nov 23, 2020

Conversation

hexylena
Copy link
Member

@hexylena hexylena commented Oct 30, 2020

FOR CONTRIBUTOR:

  • - I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • - License permits unrestricted use (educational + commercial)
  • - This PR adds a new tool or tool collection
  • - This PR updates an existing tool or tool collection
  • - This PR does something else (explain below)

xref galaxyproject/galaxy#10592

Copy link
Member

@bgruening bgruening left a comment

Choose a reason for hiding this comment

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

Can you maybe also add the bio.tools identifier? https://docs.galaxyproject.org/en/latest/dev/schema.html#tool-xrefs

Thanks @hexylena!

@hexylena
Copy link
Member Author

hexylena commented Oct 30, 2020

@bgruening sure, added to jbrowse.

I've also added the "real" circos ID to the circos tool which only had our 'galactic circos' identifier. Is that correct to do? Or should we keep it with just the 'galactic circos' id?

(the rest of the tools are custom scripts or simply not registered in bio.tools.)

@hmenager
Copy link

hmenager commented Nov 2, 2020

Thanks a lot for pushing this @hexylena !
Here are a few recommendations, up for discussion:

  • EDAM topics I would have the same for all circos wrappers, since they designate the general subfield that the tool is concerned with. I would annotate circos with Comparative genomics (topic_0797)and Data visualisation (topic_0092) just like in bio.tools, unless there is a specificity in Galaxy circos which would have it diverge significantly from this. For jbrowse, the topic in bio.tools is Genomics and I would add Data visualisation (topic_0092).
  • As a general rule, try to avoid topic_0003 (which is the Topic terms root) for tool annotation.
  • EDAM operations look fine to me, nothing to say 😃
  • xrefs are fine, but I would only keep galactic_circos (and not circos) since from what you say @hexylena it is an overlay on circos.

Please tell me if there is anything else I can do to help out.

@hmenager
Copy link

hmenager commented Nov 2, 2020

One thing that comes to my mind is that some of these remarks I made could be useful as documentation. I can help out with that as well.

@hexylena
Copy link
Member Author

hexylena commented Nov 2, 2020

Hey @hmenager, that's incredibly useful advice! Thank you !

remarks I made could be useful as documentation

Yes, definitely!

@hmenager
Copy link

hmenager commented Nov 2, 2020

Yes, definitely!

@hexylena Would a PR for this documentation page the most appropriate location?

@hexylena
Copy link
Member Author

hexylena commented Nov 2, 2020 via email

hmenager added a commit to hmenager/standards that referenced this pull request Nov 2, 2020
@bernt-matthias
Copy link
Contributor

As a follow up: I just found this old one: #1897 .. could be interesting to get EDAM for more tools

@bgruening
Copy link
Member

@hexylena is that one ready?

@hexylena
Copy link
Member Author

Yep

@wm75 wm75 mentioned this pull request Nov 23, 2020
15 tasks
@bgruening bgruening merged commit 041a4d1 into master Nov 23, 2020
@bgruening bgruening deleted the edam-viz branch November 23, 2020 10:44
@bgruening
Copy link
Member

Danki!

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