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

splits_atts with test against human genome GFF #14

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

joelb123
Copy link

@joelb123 joelb123 commented Dec 9, 2020

The pull incorporates the split_atts code that fixes reading of some GFFs along with some improvements to testing:

  • rather confusing set of symlinks removed
  • test files reformatted with black
  • each test takes place isolated in its own tmp_dir
  • human genome GFF is downloaded and used as a simple test, printing out timing for performance
  • travis build is fixed (I think), with cache of human GFF and repositories turned on

@joelb123
Copy link
Author

joelb123 commented Dec 9, 2020

a problem showed up with the travis build, submitting a new PR

@joelb123 joelb123 closed this Dec 9, 2020
@joelb123
Copy link
Author

joelb123 commented Dec 9, 2020

Fix is done, Travis builds work for me.

@joelb123 joelb123 reopened this Dec 9, 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.

1 participant