-
Notifications
You must be signed in to change notification settings - Fork 23
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
create "recommended" option for requiredness in template.csv #2
Labels
Comments
@paolinomanghi @edoardopasolli where should there be a "warning" option in the template? If it is added we can use it. |
I would put it in: |
I would use as a mandatory also
- subject ID
- the number of reads
- the number of bases
- the type of sequencing machine (not really the 2000/4000, just the type,
as it is now)
I'd rather not consider sex (gender) as mandatory because is not guaranteed
to have it (some modern policies prefer not to deploy it, see the latest
Zeharnakova paper),
also when other nice metadata are available. Instead, for example,
age_category is almost alwasy retrievable looking into the paper.
…On Mon, Jan 4, 2021 at 5:06 PM edoardopasolli ***@***.***> wrote:
I would put it in:
antibiotics_current_use
disease
study_condition
age_category
gender
country
non_westernized
These are "optional" fields, but I think a curator should be able to get
them in most of the cases
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AGZXSV6MQWNUNT77AKQRMIDSYHRRHANCNFSM4EBPGEVQ>
.
|
Call this "recommended" instead. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, only "required" or "optional" are permissible. Add "warning" as a possibility, and this will return a warning in the check and in the produced vignette.
The text was updated successfully, but these errors were encountered: