Skip to content

Dom model refactor #453

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
Merged
merged 89 commits into from
Jul 21, 2022
Merged

Dom model refactor #453

merged 89 commits into from
Jul 21, 2022

Conversation

terazus
Copy link
Collaborator

@terazus terazus commented Jul 21, 2022

No description provided.

terazus and others added 30 commits May 23, 2022 00:47
@terazus
initial refactor
4eb3ac4
@terazus
updating model
432760d
@terazus
added bunch of tests
291051a
@terazus
added publication tests
8468332
@terazus
forgot the publication.py model file
5a46bf8
@terazus
added more files
72c8303
@terazus
added parameter_value test
d4cede2
@terazus
Added StudyFactor test
72a6e47
@terazus
Added FactorValue test
88d1ed3
@terazus
Added material base test
d9cc692
@proccaserra
WIP: tests for person and source
0e27244
@proccaserra
Merge branch 'DomModelRefactor' of https://github.com/ISA-tools/isa-api…
6c79b10 
… into DomModelRefactor-patch-1
@terazus
Merge pull request #449 from ISA-tools/DomModelRefactor-patch-1
30d9f1b 
Dom model refactor patch 1
@terazus
fixed person and process_sequence tests
b70b9f2
@terazus
finished source tests
ae075a0
@terazus
added sample test
8bf227f
@terazus
finished sample test
8d05c18
@terazus
added material tests
5e070ce
@terazus
added all data files tests
08522b9
@terazus
added process tests
e78527b
@terazus
added mixins tests
9e2d610
@terazus
removed redundant tests
fb04570
@terazus
rolled bacbk protocol components and parameters
51cb620
@terazus
fixed ols tests
52cbeab
@terazus
forced version as an empty string if nan in sampletab
ff6e799
@terazus
fixed missing tests for protocol
bb11976
@terazus
finished mixins tests + moved _build_assay_graph and find to utils
9579775
@terazus
moved plink to utils
07f79be
@terazus
added assay and study tests
f935141
@terazus
finished study tests
1b8329e
terazus added 28 commits June 1, 2022 06:16
@terazus
Added other materials to study serializer
b0d3fb3
@terazus
fixed dumb tests hanging
5659a9b
@terazus
extended process serializer
b6d1f37
@terazus
added StudyFactor to study serializer
3d268b2
@terazus
Added characteristic categories to study serializer
b158040
@terazus
Added unit categories to study serializer
598cbb3
@terazus
Added assay serializer
771ce0c
@terazus
fixed next and previous empty process serializer
930a050
@terazus
fixed process unit serializer
34ed778
@terazus
fixed empty unit in characteristic serializer
9395785
@terazus
replaced serializer + fixed isa create
544e07c
@terazus
removed unused code
11c3ea8
@terazus
added safecheck to the json encoder"
f31ef0e
@terazus
started a from_dict() method on Investigation to deserialize data fro…
4784e99 
…m a dict
@terazus
Added study deserializer
5d66e44
@terazus
Added factors to study deserialiser
2e96a78
@terazus
added source to study loader
dfd0426
@terazus
Added a new indexer class for the JSON deserializer
4534fde
@terazus
Added samples to study serializer + extended loader index
3a19612
@terazus
Added processSequence to Study deserializer
1cb61e9
@terazus
fixed minor typo
2201651
@terazus
Started assay deserializer
052ae27
@terazus
extending assay deserialiser
8c4f51a
@terazus
Added other materials to assay loader
e9abc41
@terazus
finally finished new loader
f17c6be
@terazus
refactor loader test to use new loads function
18a4489
@terazus
fixed issue with characteristicCategory bounds to assay
956d3fb
@terazus
removed old load method
50c693f
@terazus terazus merged commit 4d3c82a into develop Jul 21, 2022
@terazus terazus deleted the DomModelRefactor branch July 21, 2022 12:51
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.
2 participants
@terazus @proccaserra