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

on large import of a genome w/o any annotation, divide up only chunk … #43

Merged
merged 4 commits into from
Aug 12, 2020

Conversation

benjiec
Copy link
Contributor

@benjiec benjiec commented Aug 11, 2020

…into workable chunk sizes too

@@ -1,6 +1,6 @@
from django.db.backends.signals import connection_created

__version__ = '2.6.0'
__version__ = '2.8.0'
Copy link
Member

Choose a reason for hiding this comment

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

What's the reason for bumping the version here in the PR? I think the flow is to run https://github.com/ginkgobioworks/edge#versioning on the master branch which will automatically bump the version and push a tag. Were you trying to release this branch on pypi?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think I just didn't know what I was doing.

src/edge/importer.py Outdated Show resolved Hide resolved
src/edge/importer.py Outdated Show resolved Hide resolved
@yaoyuyang
Copy link
Member

So @benjiec, if I understand correctly, when we import genome without annotation before, we don't break up it into any chunks before or is it the case that we don't break up into small enough chunks?

@benjiec
Copy link
Contributor Author

benjiec commented Aug 12, 2020

@yaoyuyang Looks like we don't break it up before

@benjiec benjiec merged commit a20ca33 into master Aug 12, 2020
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.

None yet

3 participants