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

Initiate EML Assembly Line ecology metadata management tutorial #4753

Merged
merged 55 commits into from
Mar 4, 2024

Conversation

yvanlebras
Copy link
Collaborator

First complete version

@yvanlebras yvanlebras self-assigned this Mar 4, 2024
@yvanlebras yvanlebras added the fair label Mar 4, 2024
@yvanlebras
Copy link
Collaborator Author

Hi @shiltemann @hexylena just added you as reviewers for this tutorial I want to use next week (sorry for the delay...) for a training (and I hope I can use it through Tiaas @bgruening ). the idea is to have a first version and then we will gather feedbacks and come back to it to update it

@yvanlebras yvanlebras requested a review from bgruening March 4, 2024 10:48
Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

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

i am happy to merge it right now, if you add draft: true to the top, so you can use it in your training.

otherwise, here is a first round of GTN formatting comments, I guess it will take some time to get it to GTN standards.

topics/ecology/tutorials/MetaShARK_tutorial/tutorial.md Outdated Show resolved Hide resolved
topics/ecology/tutorials/MetaShARK_tutorial/tutorial.md Outdated Show resolved Hide resolved
topics/ecology/tutorials/MetaShARK_tutorial/tutorial.md Outdated Show resolved Hide resolved
topics/ecology/tutorials/MetaShARK_tutorial/tutorial.md Outdated Show resolved Hide resolved

API key format to enter is something like:
```
api 205b1e521f2eaf0ad4a361c438b63205b1e521f2eaf0ad4a361c438b63c438b63
Copy link
Member

Choose a reason for hiding this comment

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

i checked that it isn't a live api key :) and in the process found this could use more documentation.

  1. you need to register
  2. and then go to the 'profile' on cedar.metadatacenter.org
  3. and there you can find the API key (I was having issues locating it, otherwise.)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done, thank you!

topics/ecology/tutorials/MetaShARK_tutorial/tutorial.md Outdated Show resolved Hide resolved
topics/ecology/tutorials/MetaShARK_tutorial/tutorial.md Outdated Show resolved Hide resolved
topics/ecology/tutorials/MetaShARK_tutorial/tutorial.md Outdated Show resolved Hide resolved
topics/ecology/tutorials/MetaShARK_tutorial/tutorial.md Outdated Show resolved Hide resolved
topics/ecology/tutorials/MetaShARK_tutorial/tutorial.md Outdated Show resolved Hide resolved
Co-authored-by: Helena <hexylena@galaxians.org>
@github-actions github-actions bot removed the fair label Mar 4, 2024
@yvanlebras
Copy link
Collaborator Author

THNAK YOU so much @hexylena ! I hope I add the draft: true mention as appropriate so we can merge it now, and I started to ameliorate the content notably thnks to all your feedback!!!! THANK YOU so much! Hope all is ok on your side!

@hexylena hexylena merged commit 0b820e8 into galaxyproject:main Mar 4, 2024
3 checks passed
@hexylena
Copy link
Member

hexylena commented Mar 4, 2024

https://training.galaxyproject.org/training-material/topics/ecology/tutorials/MetaShARK_tutorial/tutorial.html

something is odd with the images, I am not sure why they didn't render.

@yvanlebras
Copy link
Collaborator Author

THANK YOU !!!!
I just make another PR as it seems I forgot to rewrite links to png images in a GTN manner in a lot of images...
#4769

@yvanlebras yvanlebras deleted the metashark branch March 12, 2024 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants