-
Notifications
You must be signed in to change notification settings - Fork 13
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
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Dec 12, 2023
Merged
Merged
CunliangGeng
force-pushed
the
12-11-update_loading_genomics
branch
from
December 12, 2023 11:02
c4d78bc
to
ede6a9a
Compare
This was referenced Dec 12, 2023
CunliangGeng
force-pushed
the
12-04-update_loading_metabolomics
branch
from
December 14, 2023 08:33
a1be676
to
b636ccb
Compare
CunliangGeng
force-pushed
the
12-11-update_loading_genomics
branch
from
December 14, 2023 08:33
ede6a9a
to
ed4d425
Compare
CunliangGeng
force-pushed
the
12-04-update_loading_metabolomics
branch
from
December 14, 2023 10:00
b636ccb
to
d736891
Compare
CunliangGeng
force-pushed
the
12-11-update_loading_genomics
branch
from
December 14, 2023 10:00
ed4d425
to
36816a7
Compare
CunliangGeng
force-pushed
the
12-04-update_loading_metabolomics
branch
from
December 19, 2023 13:21
d736891
to
9c4f3d1
Compare
CunliangGeng
force-pushed
the
12-11-update_loading_genomics
branch
from
December 19, 2023 13:21
36816a7
to
bc05232
Compare
CunliangGeng
force-pushed
the
12-04-update_loading_metabolomics
branch
from
December 19, 2023 13:38
9c4f3d1
to
5354122
Compare
CunliangGeng
force-pushed
the
12-11-update_loading_genomics
branch
from
December 19, 2023 13:38
bc05232
to
83f8e39
Compare
This was referenced Dec 20, 2023
gcroci2
approved these changes
Dec 21, 2023
Merge activity
|
- 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
force-pushed
the
12-11-update_loading_genomics
branch
from
January 24, 2024 15:21
83f8e39
to
c3bc259
Compare
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update the genomics loading process and helper functions, making sure they are consistent with those for loading metatbolomics.
Major changes:
self.bgcs
andself.gcfs
to make sure they contain valid BGC/GCF objects that have Strain objects added