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

Overall Project info page changes. #5

Open
cjvillar opened this issue Oct 3, 2019 · 10 comments
Open

Overall Project info page changes. #5

cjvillar opened this issue Oct 3, 2019 · 10 comments

Comments

@cjvillar
Copy link

cjvillar commented Oct 3, 2019

Page Project

Change ordering of sections to:
The preferred order of categories
Project
Wrangling
Submission steps
Post submission tracking

We feel a need to remove the star system (can discuss with Clay if needed)
Consent to use boolean "Yes" or "No"
Sample type → with enums
Add “library preparation methods”
Add organ, organ_part, disease, cell count to this section
Combine Pubs, people & pay into one on the page.

@pnejad
Copy link

pnejad commented Oct 7, 2019

sample type enums:

  • primary tissue
  • organoid
  • stem cell derived
  • cell line

@pnejad
Copy link

pnejad commented Oct 7, 2019

library preparation methods enums
Screen Shot 2019-10-07 at 10 55 04 AM

@JimKent
Copy link
Contributor

JimKent commented Oct 8, 2019

Paris - how are we going to tell if it's frozen or fresh? Maybe what I'm wanting to capture in this field is spread across many in the spreadsheets.

@pnejad
Copy link

pnejad commented Oct 8, 2019

@JimKent that would be captured by 'preservation_method', which has the following enums:

"cryopreservation in liquid nitrogen (dead tissue)",
"cryopreservation in dry ice (dead tissue)",
"cryopreservation of live cells in liquid nitrogen",
"cryopreservation, other",
"formalin fixed, unbuffered",
"formalin fixed, buffered",
"formalin fixed and paraffin embedded",
"hypothermic preservation media at 2-8C",
"fresh"

@JimKent
Copy link
Contributor

JimKent commented Oct 8, 2019

Thanks @pnejad. I added a preservation_method field hooked up to a vocab in the x table of the same name. I just populated it with fresh. I also replaced "fresh sample" in the "sample type" table with "primary tissue."

WHen I release the next version, possibly later today, could you edit the sample_type table inside of Tuatara to match your list, and add the ones (or at least the ones you think we'll see) from the preservation method? Thanks!

@JimKent
Copy link
Contributor

JimKent commented Oct 8, 2019

Ok, the dev version has all the stuff from the initial ticket that I agree with and can change. A few things are reworked enough I'd like you to just have another look, and let me know if you like it.

For the concent I am not using the yes/no. I feel like we are overloading this field trying to make it serve both "concent from donor" and "concent from researcher". With this one I am just trying to capture concent from donor. Also I want to capture the donor concent in a slightly more nuanced way than just yes/no.

The AssayType is now called cDNA library prep. I have not edited out the SE and PE from the smart-seq2, and honestly am tempted not to, as currently the distinction is important, but apparently both SE and PE smartseq2 is going to be handled soon if not already by the pipeline, so we could just merge the fields at that point.

In terms of combining a bunch of stuff into the Overall section, please look at the new arrangement, and let me know if you would still like to. I did rearrange the Overall section quite a bit.

@pnejad
Copy link

pnejad commented Oct 9, 2019

@JimKent how often do we anticipate tracking a dataset where the donor has not given consent?

If consent has not been given by the donor, it should not be added to tuatara in the first place. I can only see this being useful in a scenario where the researchers are going back to the donors to request consent.

The consent that wranglers need to worry about is from the researchers (not applicable to GEO wrangling). We can not accept metadata or data files until the data contributor agrees to the terms in this form: tinyurl.com/HCA-Terms-Conditions

@pnejad
Copy link

pnejad commented Oct 9, 2019

@JimKent

The AssayType is now called cDNA library prep. I have not edited out the SE and PE from the smart-seq2, and honestly am tempted not to, as currently the distinction is important, but apparently both SE and PE smartseq2 is going to be handled soon if not already by the pipeline, so we could just merge the fields at that point.

Where would I see the SE and PE?

@JimKent
Copy link
Contributor

JimKent commented Oct 18, 2019

It is still right after smart-seq2 in library prep controlled vocab.

@JimKent
Copy link
Contributor

JimKent commented Oct 18, 2019

In terms of tracking stuff without consent - I do think we'll want to track some projects before we get the T&C. Historically we have done so with some from Aviv Regev's lab I believe for instance.

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

No branches or pull requests

3 participants