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

Fix file format identifier for input files in setup instructions #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zoe-translates
Copy link

The instructions ask the learner to create a file "chr1-star-index.yaml" that references an input file in the FASTA format, if they are to follow the steps for generating the STAR index themselves.

In the YAML file however, the "format" field for that file referred to something else (FASTQ format). If the user followed the instructions and submitted the workflow to Arvados, the run would fail validation.

Fixed by using the intended URI for the format (tested in Arvados 2.8.0 dev).

Arvados-DCO-1.1-Signed-off-by: Zoë Ma [email protected]

Instructions

Thanks for contributing! ❤️

If this contribution is for instructor training, please email the link to this contribution to
[email protected] so we can record your progress. You've completed your contribution
step for instructor checkout by submitting this contribution!

Keep in mind that lesson maintainers are volunteers and it may take them some time to
respond to your contribution. Although not all contributions can be incorporated into the lesson
materials, we appreciate your time and effort to improve the curriculum. If you have any questions
about the lesson maintenance process or would like to volunteer your time as a contribution
reviewer, please contact The Carpentries Team at [email protected].

You may delete these instructions from your comment.

- The Carpentries

The instructions ask the learner to create a file "chr1-star-index.yaml"
that references an input file in the FASTA format, if they are to follow
the steps for generating the STAR index themselves.

In the YAML file however, the "format" field for that file referred to
something else (FASTQ format).  If the user followed the instructions
and submitted the workflow to Arvados, the run would fail validation.

Fixed by using the intended URI for the format (tested in Arvados 2.8.0
dev).

Arvados-DCO-1.1-Signed-off-by: Zoë Ma <[email protected]>
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.

1 participant