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

[MAINT] annotations/tests #188

Merged
merged 24 commits into from
Feb 1, 2022
Merged

[MAINT] annotations/tests #188

merged 24 commits into from
Feb 1, 2022

Conversation

jdkent
Copy link
Member

@jdkent jdkent commented Jan 29, 2022

testing/modification of model relationships to ensure proper behavior when deleting/updating tables.

  • remove annotation analyses for particular study when the study is removed from a dataset
  • ensure all notes have the same column names
  • add more tests surrounding annotations

@jdkent
updates
99ff58e
@jdkent
move analysis relationship with analysis_annotations to analysis table
8201f36
@jdkent
test that annotation_analyses are deleted when the analysis is deleted
d793147
@jdkent
update openapi
1b0d5e3
@jdkent
add patch functionality
66a8aa0
@jdkent
Revert "update openapi"
974bf3d 
This reverts commit 1b0d5e3.
@jdkent
add openapi changes
29e9a28
@jdkent
Revert "add patch functionality"
efcd2e3 
remove patch functionality
This reverts commit 66a8aa0.
@jdkent
check when annotations are updated that they all contain the same keys
24a1d91
@jdkent
test proper/improper additions
a264874
@jdkent
abject mess
287fa66
@jdkent
Revert "abject mess"
16f0e54 
This reverts commit 287fa66.
@jdkent
stable models
a196b40
@jdkent
delete annotationanalyses when a study is removed from dataset
a058643
@jdkent
refactor ingestion to account for annotations
a80d949
@jdkent
fix wonkiness with deleting studies
e5c2dde
@jdkent
add datasetStudies schema
bcf391c
@jdkent
refactor how annotationanalyses are handled
563eb47
@jdkent
handle tests for new annotation situations
289e3a0
@jdkent
add new changes to openapi
c27b2c4
@jdkent
merge openapi spec with master
5c88edf
@jdkent
fix type
78ce511
@jdkent
fix conflicts
f5ab8d5
@jdkent
change annotation to annotation_csv when exporting
f9d3b41
@jdkent jdkent merged commit b82b3e0 into master Feb 1, 2022
@jdkent jdkent mentioned this pull request Mar 22, 2022
Closed
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.

None yet
1 participant
@jdkent