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 logics of loading metabolomics data #195

Conversation

CunliangGeng
Copy link
Member

@CunliangGeng CunliangGeng commented Dec 5, 2023

Major changes:

  • update loading process in DatasetLoader._load_metabolomics.
  • add utility functions for metabolomics loading

Add keep_singleton to the base class of GCF loader
- change returned value from dict to list for `MibigLoader.get_bgcs` method and the underlying method `._parse_bgcs`
- update the use of the changed methods
- update unit tests
- change returned value from dict to list for `AntismashBGCLoader.get_bgcs` method and the underlying method `._parse_bgcs`
- update the use of the changed methods
- update unit tests
This function is needed anymore. GCF loader already has the functionality to filter mibig only GCF during loading.
@CunliangGeng CunliangGeng force-pushed the 12-04-remove_unused_mgf_parser branch from 043e2e5 to 7c9daaa Compare December 5, 2023 15:54
@CunliangGeng CunliangGeng force-pushed the 12-04-update_load_metabolomics branch from 9b499d4 to 9a322a7 Compare December 5, 2023 15:54
@CunliangGeng CunliangGeng force-pushed the 12-04-remove_unused_mgf_parser branch from 7c9daaa to d633acd Compare December 11, 2023 13:04
@CunliangGeng
Copy link
Member Author

CunliangGeng commented Dec 11, 2023

Commits from previous PRs are mixed in this PR. Close it now, will create a new one with clean commits (see #196).

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.

1 participant