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.

Sign up for GitHub

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

Jump to bottom

Record Samples Fix #209

Closed
wants to merge 19 commits into from
Closed

Record Samples Fix #209

wants to merge 19 commits into from

Conversation

svarona
Copy link
Member

@svarona svarona commented Jul 28, 2023 •
edited
Loading

Fix the module to record samples in Wetlab

  • Fix record new Samples using Input table with everything OK
    • Fix cocde not working when Sample's species (not mandatory) is empty
  • Fix Load New Samples in batch with everything OK
  • Fix record new Samples using Input table when something is missing ("break the code")
    • Repeated Samples
    • Missing mandatory data
  • Fix Load New Samples in batch with everything OK when something is missing ("break the code")
    • Repeated Sample
    • Missing mandatory data
    • Missing project field and/or data

@svarona
handled non existing species in samples
9751e88
@svarona
handled non existing species in samples as attributeError
939ae45
@svarona
handled non existing species when requesting a service
d717b1f
@svarona
Renamed super long function name
eea8c69
@svarona
Removed pdb set trace
f6d5ac5
@svarona
Removed molecule and protocol functions that break the code
3c6025f
@svarona
Fixed typo
47a54ee
@svarona
Fixed incorrect column names
6d6c701
@svarona
Fixed a typo and reformatted some of the error messages
7fca2b4
@svarona
Renamed a variable
44fc190
@svarona
Removed unnecesary code for molecule type
ec51fea
@svarona
Fixed some error messages returning
7bda75b
@svarona
Created function and error messages to test for repeated sample in ba…
07b727c 
…tch file and database
@svarona
Added function to validate sample names
9294c7e
@svarona
Made heading validation check for project fieleds and unique project
fb2de17
@svarona
Move unique sample type check and renamed error variable
b23b32d
@svarona
Renamed duplicated variable name
cae9038
@svarona
Reordered some code
9f13a4f
@svarona
Unified error managing
5f84786
@svarona svarona closed this by deleting the head repository Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@svarona