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

feat: include molecular profile segment data in molecular profile response #144

Merged
merged 2 commits into from
Apr 19, 2024

Conversation

korikuzma
Copy link
Collaborator

@korikuzma korikuzma commented Aug 12, 2023

close #143 . This is needed for this hackathon issue.

@susannasiebert I can't remember how the test_cache.pkl is populated for the tests. I ran civicpy update --hard and used this as my cache. The molecular profile tests had 6353 as an ID. I tried using the graphiql api and looked at https://civicdb.org/molecular-profiles/6353/summary , but the molecular profile does not exist. 3 of the tests are failing (on both my cache with changes from my branch and the cache from the master branch). Depending on how the test_cache.pkl is populated, I can update in a separate issue.

@susannasiebert
Copy link
Collaborator

susannasiebert commented Jan 5, 2024

From what I remember, you did the right thing for creating the test cache pkl file. I think when I made these updates, I created a pkl from my local database since the changes weren't actually deployed yet and the ID I used in the tests would've been one from my local database and it was possibly a molecular profile that I created manually. You did the right thing by updating the ID and tests to something that matches a molecular profile in production.

Copy link
Collaborator

@susannasiebert susannasiebert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@susannasiebert susannasiebert merged commit 96cfffe into master Apr 19, 2024
6 checks passed
@acoffman acoffman deleted the issue-143 branch April 19, 2024 17:34
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

Successfully merging this pull request may close these issues.

Add molecular profile component operators
2 participants