-
Notifications
You must be signed in to change notification settings - Fork 5
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
Well libs #229
Well libs #229
Conversation
Reimplemented the PacBioExperiment class, removed from_orm method, replaced it by a pre-init hook.
... for mlwh ORM classes so that common methods can be implemented. Customised __repr__ method for one of db classes.
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.
Seems fine, a few questions to ease my concerns.
lang_qc/db/mlwh_schema.py
Outdated
A base class for declarative class definitions for the ml warehouse database. | ||
""" | ||
|
||
def get_row_description(self, fields: list[str]) -> str: |
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.
Maybe make this guy private?
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.
done
returned by the constructor. | ||
""" | ||
|
||
db_library: PacBioRun = Field(init_var=True) |
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.
Spot check - a "library" is the prep of one sample, therefore 1:1 with a row in pac_bio_run
? As distinct from pool, which is a pool of libraries?
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.
Correct. A pool might contain one library as long as the library is indexed. The term pool
is only used with indexed libraries.
Also made the helper function of the parent class 'private'.
No description provided.