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

Update loading of genomics data #197

Merged
merged 6 commits into from
Jan 24, 2024
Merged

Conversation

CunliangGeng
Copy link
Member

@CunliangGeng CunliangGeng commented Dec 12, 2023

Update the genomics loading process and helper functions, making sure they are consistent with those for loading metatbolomics.

Major changes:

  • update the assignment of attributes self.bgcs and self.gcfs to make sure they contain valid BGC/GCF objects that have Strain objects added
  • remove and update helper functions
  • update unit tests on helper functions

@CunliangGeng CunliangGeng requested a review from gcroci2 December 12, 2023 10:57
@CunliangGeng CunliangGeng force-pushed the 12-11-update_loading_genomics branch from c4d78bc to ede6a9a Compare December 12, 2023 11:02
@CunliangGeng CunliangGeng force-pushed the 12-04-update_loading_metabolomics branch from a1be676 to b636ccb Compare December 14, 2023 08:33
@CunliangGeng CunliangGeng force-pushed the 12-11-update_loading_genomics branch from ede6a9a to ed4d425 Compare December 14, 2023 08:33
@CunliangGeng CunliangGeng force-pushed the 12-04-update_loading_metabolomics branch from b636ccb to d736891 Compare December 14, 2023 10:00
@CunliangGeng CunliangGeng force-pushed the 12-11-update_loading_genomics branch from ed4d425 to 36816a7 Compare December 14, 2023 10:00
@CunliangGeng CunliangGeng force-pushed the 12-04-update_loading_metabolomics branch from d736891 to 9c4f3d1 Compare December 19, 2023 13:21
@CunliangGeng CunliangGeng force-pushed the 12-11-update_loading_genomics branch from 36816a7 to bc05232 Compare December 19, 2023 13:21
@CunliangGeng CunliangGeng force-pushed the 12-04-update_loading_metabolomics branch from 9c4f3d1 to 5354122 Compare December 19, 2023 13:38
.vscode/settings.json Show resolved Hide resolved
Copy link
Member Author

CunliangGeng commented Jan 24, 2024

Merge activity

Base automatically changed from 12-04-update_loading_metabolomics to dev January 24, 2024 15:21
- add the missing comma
- the values were updated automatically based on latest schema of vscode
Change raising not found error to only logging it.
- remove raising errors
- add logging info
- add return values
@CunliangGeng CunliangGeng force-pushed the 12-11-update_loading_genomics branch from 83f8e39 to c3bc259 Compare January 24, 2024 15:21
@CunliangGeng CunliangGeng merged commit 9e3d296 into dev Jan 24, 2024
1 of 2 checks passed
@CunliangGeng CunliangGeng deleted the 12-11-update_loading_genomics branch January 24, 2024 15:22
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.

2 participants