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

Assay types are not fully centralized #480

Closed
wasade opened this issue Apr 23, 2019 · 1 comment
Closed

Assay types are not fully centralized #480

wasade opened this issue Apr 23, 2019 · 1 comment
Milestone

Comments

@wasade
Copy link
Member

wasade commented Apr 23, 2019

While flipping through pages, I noticed that "NewKindOfAssay" is an assay type in a test environment (added here). However, it is not valid across the system as it is still accessible and can result in exceptions.

I'm marking this as a nice to have issue right now as I believe we can defer the refactor since I don't believe we're adding in new assays near term. However, we probably only want to present valid assays to users.

assay_is_not_controlled

AN ERROR HAS OCCURED!
Please copy the following into an issue in our github repository

Error
Unrecognized assay type: NewKindOfAssay

Traceback
Traceback (most recent call last): 
File "/Users/dtmcdonald/miniconda3/envs/labcontrol/lib/python3.7/site-packages/tornado/web.py", line 1697, in _execute result = method(*self.path_args, **self.path_kwargs) 
File "/Users/dtmcdonald/miniconda3/envs/labcontrol/lib/python3.7/site-packages/tornado/web.py", line 3174, in wrapper return method(self, *args, **kwargs) 
File "/Users/dtmcdonald/ResearchWork/software/labman/labcontrol/gui/handlers/process_handlers/sequencing_process.py", line 59, in get text = process.generate_sample_sheet() 
File "/Users/dtmcdonald/ResearchWork/software/labman/labcontrol/db/process.py", line 3242, in generate_sample_sheet raise ValueError("Unrecognized assay type: {0}".format(assay)) 
ValueError: Unrecognized assay type: NewKindOfAssay 
@charles-cowart
Copy link
Collaborator

Resolved with #554

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

2 participants