-
Notifications
You must be signed in to change notification settings - Fork 52
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
Update ISA JSON compliance documentation #1847
Conversation
...from the general ISA-JSON compliant Experiments overview page.
In Experiment View, you can also view Study Sources, Study Samples, and Assay Samples in a searchable table by selecting "samples (n)" from the tree view on the left sidebar. Samples cannot be created or edited via this view. | ||
Study Source(s) | ||
* Study Sources must be created within an ISA Study, using an Experiment Sample Template level "Study Source". | ||
* Each Study Source must be the input of at least one Study Sample (or more) in the same ISA Study. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be confusing. We can define a Study Source without it having a Study Sample associated with it, in contrast to what the sentence implies. Perhaps replace 'must' by 'can' or "should".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you are right. better rephrase it as
- Each Study Source should be the input of at least one Study Sample (or more) in the same ISA Study.
# Experiment Sample Templates | ||
|
||
Experiment Sample Templates act as blueprints to create Sample Types within ISA Studies and ISA Assays. The same Experiment Sample Template can be applied multiple times to create Sample Types in different ISA Studies or ISA Assays. | ||
## Experiment Sample Templates |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@somathias why shouldn't this be main title (only 1 #)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Personally, I prefer if "ISA-JSON compliant investigation" is the main title and all other information in the same file is organised under it at suitable levels. (Formulated in a different way: why did you have multiple main titles in the same .md file?)
|
||
When the ISA-JSON compliance feature is enabled in the platform, it is possible to use the "ISA minimal starter template" as a starting point to create any Experiment Sample Template. One ISA minimal starter template for each ISA Level is provided with the feature. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do you think this paragraph should be removed?
2. One Repository name or SEEK Project | ||
* Repository or data archive (e.g. EBI databases) | ||
* SEEK Project (Project specific templates) | ||
See [Experiment Sample Templates](isajson-templates.html) for more information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok to delete the characteristics and link to isajson-templates page
* Each Assay Sample must have at least one input (or more). Inputs can be: | ||
* Study Samples in the same ISA Study; | ||
* Assay samples from one preceding Assay, in the same Assay Stream. | ||
See [Working with Samples in ISA-JSON compliant Experiments](create-sample-isajson-compliant.html) for more information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The linked page gives different info. I would not link it here.
help/user-guide/isajson-templates.md
Outdated
|
||
|
||
### JSON upload via Server admin panel [FOR SYSTEM ADMINISTRATORS] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please, keep the distinction "for instance administrator" and "for project members" as main title (1 #).
add "alias single page", since in the code, in the URL and admin panel is still called Single Page
add bullet points suggested
edit "must" with "should" as suggested
edit order of paragraphs
@stuzart |
Please feel free to merge, but there is a conflict that needs resolving. Let me know if you have trouble resolving it through the github UI |
@floradanna I took the liberty of resolving the conflict by adding the additional sentence referencing where to find the EBI specific templates to the section for system administrators in the isajson-templates.md. Please merge if you agree. |
Thanks a million @somathias ! I am going to merge it now. |
I was not able to reopen the previous PR #1812 so I have turned my suggested changes into a new PR.
This PR includes