-
Notifications
You must be signed in to change notification settings - Fork 8
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
Performance enhancements, util refactor and and clean up tests #153
Conversation
Use Strandedness and Topology enums in multiple places
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This deprecated stuff should just be removed. Is there something stopping you?
I just moved the last of the utils, so not anymore. |
🎉 This PR is included in version 2.0.0-alpha.31 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Performance enhancements
Repo.index_otus()
Repo.iter_otus()
Fixes
Repo._iter_events()
and.get_source_event_ids()
now handle chaff filesUtils refactor
Molecule
tovirtool_cli.utils.models
MolType
,Strandedness
andTopology
toutils.models
and replace relevant duplicate enums inncbi.models
utils.console.console
,utils.id_generator.generate_random_alphanumeric()
and.generate_unique_ids
tolegacy.utils
test_utils_hashing()
moved to legacy utils test dirutils.format
,.ncbi
,.storage
toutils.deprecated
submoduleClean up tests
src_test
with currentEventSourcedRepo
filessrc_test
files toprototype_src_test
dirtest_utils_storage
andtest_utils_reference
Add larger-scale add test
cache_test