-
Notifications
You must be signed in to change notification settings - Fork 0
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
Transfer code from Tools module of OrthoEvolution #3
Comments
Some of the Tools modules can be deprecated:
Some of the Tools modules need to be left in the package:
Some of the Tools modules need to be moved to snakeutils: master
dev_master
|
When developing this add all of the "for sure" sub-modules" to the master and create the package with that. All of the submodules that are still in the works, can be added after the fact to a dev-master branch or something. |
This is just a thought, so let me know what you think @sdhutchins. I know we have a lot going on here, so what if we made separate packages for each of the submodules? (Please disregard the naming conventions, they are just placeholders for the concept) So we would have:
And then if you wanted all of @datasnakes/snakes utilities you just install I will, of course, have to finish my review of datasnakes/OrthoEvolution#155, but what are your thoughts on this? |
I actually think it's a solid idea tbh - and yes, also more appealing. It also will be probably easier for us to split them up and do things that way! More than anything, Maybe we could make logit and parallel into There's a lot of ways we could do it, but I definitely agree with splitting it into something more modular for people instead of a big-ish library. |
The tools module currently contains the below submodules.
Some of these modules may not need to be added to this package and are incomplete.
As it stands, the modules that currently need to be migrated are:
We may still like to hold onto
pybasher
,slackify
,pandoc
,send2server
, andmpi
as either scripts or add them to a utils file or some other kind of documentation to help people.The text was updated successfully, but these errors were encountered: