-
Notifications
You must be signed in to change notification settings - Fork 24
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
adding i5_i7_index #666
adding i5_i7_index #666
Conversation
Should there be any data access changes? |
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.
just questions
|
||
CREATE TABLE pm.shotgun_index_tech ( | ||
shotgun_index_tech_id bigserial NOT NULL, | ||
name varchar(100) , |
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.
what is this one limited?
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.
Honestly, cause that was the default in dbSchema and didn't catch it. However, this is just the name of the tech, which in general is like 20 chars ...
@@ -236,17 +244,17 @@ CREATE TABLE pm.water_lot ( | |||
CONSTRAINT uq_water_lot_name UNIQUE ( name ) | |||
); | |||
|
|||
CREATE TYPE target_region AS ENUM ('16S', '18S', 'ITS'); | |||
CREATE TYPE target_subfragment AS ENUM ('V4'); | |||
CREATE TYPE pm.target_region AS ENUM ('16S', '18S', 'ITS'); |
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.
Should this differentiate ITS1, ITS2, overhanging (e.g., sequencing 3' of 18S into ITS1)? Perhaps as subfragment?
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.
Good question, I just fixed the pm part of this but I can add more ... @josenavas
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.
Let's defer to later
No description provided.