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

Add "JGI - metagenomics - long read" class to submission schema #168

Closed
Tracked by #413
bmeluch opened this issue Jan 2, 2024 · 11 comments · Fixed by #185 or microbiomedata/nmdc-server#1191
Closed
Tracked by #413
Assignees

Comments

@bmeluch
Copy link
Contributor

bmeluch commented Jan 2, 2024

Due in Dev Feb 23rd

Part of the effort to make the submission portal accommodate JGI user projects.

The current "JGI MG" tab was created based on short read data. Long read data will use the same slots, with the difference being that dna_absorb1 and dna_absorb2 are required. (See microbiomedata/issues#433)

This will require creating a new JGI metagenomics long reads class and submission portal tab (microbiomedata/issues#413 (comment)).

@bmeluch bmeluch self-assigned this Jan 2, 2024
@bmeluch bmeluch changed the title Update NMDC submission schema Add "JGI - metagenomics - long read" class to submission schema Jan 2, 2024
@bmeluch bmeluch transferred this issue from microbiomedata/issues Jan 2, 2024
@turbomam
Copy link
Member

turbomam commented Jan 3, 2024

Hi @bmeluch

Can you say more about the difference between a "tab" and a class? Where could I find "JGI MG" in the submission schema? I did the following search and I don't think I'm finding anything with that name.

https://github.com/search?q=repo%3Amicrobiomedata%2Fsubmission-schema+JGI+MG&type=code

"JGI MG - long reads" would be an unusual name for a class

@bmeluch
Copy link
Contributor Author

bmeluch commented Jan 3, 2024

@turbomam yes good call, the class name shouldn't have been in quotes it was just describing the idea for the class.

"JGI MG" is from here https://github.com/microbiomedata/submission-schema/blob/main/schemasheets/tsv_in/classes.tsv line 11, as well as from the tab name that displays in the submission portal (see below).

I'm proposing to change the one line "JgiMgInterface" to two lines, "JgiMgSrInterface" and "JgiMgLrInterface" to distinguish short and long read data.

If my understanding is correct, the interface classes (e.g. "SoilInterface") correspond to sample types, and sample types selected in setting up a submission determine the tabs that appear in the last step of the submission portal (example photos).

image
image

@mslarae13
Copy link
Contributor

@bmeluch are you currently working on this, or waiting on something?

@mslarae13
Copy link
Contributor

mslarae13 commented Jan 30, 2024

@pkalita-lbl
I checked with @bmeluch. She feels like she's done about all she can here. Would you have some time to either take this on and wrap it up or help guide her to close this out?

We think the remaining things to happen are UI updates.

  • Update the UI to show the additional / update the metagenomics- long and -short
  • Update version / prod to show the 2 interface tabs

@mslarae13
Copy link
Contributor

Technically, schema is updated. UI needs updated. Should this be a separate issue?

@pkalita-lbl
Copy link
Collaborator

Yeah I can take over from here. Don't worry about a new issue for right now I'll make one for myself if needed. I'll add myself as an assignee and add this to the next sprint so I don't lose track of it.

@pkalita-lbl pkalita-lbl self-assigned this Jan 31, 2024
@mslarae13
Copy link
Contributor

Need confirmation on additional analysis type. Then into submission schema.
Rescheduled for March 8th in dev

@pkalita-lbl
Copy link
Collaborator

Reopening until the submission portal work is also done.

@pkalita-lbl pkalita-lbl reopened this Mar 5, 2024
@pkalita-lbl
Copy link
Collaborator

Still in progress, awaiting review on this PR which upgrades the submission-schema version.

@ssarrafan
Copy link

It looks like there is a validation issue on the linked PR. Moving this to the next sprint.
@bmeluch @pkalita-lbl if this isn't high priority we can move it to the post-GSP sprint. Let me know.

@pkalita-lbl
Copy link
Collaborator

Just waiting on final code review here. Moving to next sprint in case it doesn't happen today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment