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

Packaging and Unit tests #3

Closed
rhiever opened this issue Nov 11, 2015 · 7 comments
Closed

Packaging and Unit tests #3

rhiever opened this issue Nov 11, 2015 · 7 comments

Comments

@rhiever
Copy link
Contributor

rhiever commented Nov 11, 2015

From @rasbt:

I think it would be worthwhile to turn TPOT into an importable python module/package and to add unit tests. This would help with the development, especially in collaboration.

For the first public release, I think it would be a big plus to add continuous integration, e.g,. Travis CI (in terms of trustworthiness)

rhiever added a commit that referenced this issue Nov 11, 2015
Now TPOT can be imported as a regular Python package as well as called
on the command line.
@rhiever
Copy link
Contributor Author

rhiever commented Nov 11, 2015

I'm building some basic unit tests. @rasbt, how should unit tests be written to integrate with Travis CI? Can you lead the way with Travis CI integration?

@rasbt
Copy link
Contributor

rasbt commented Nov 11, 2015

Sure, I can deal with the Travis stuff once the unit tests are there. It should be pretty straightforward. You just need to implement the unit tests in a way that they can be executed by e.g., running nosetests from the tpot main directory

rhiever added a commit that referenced this issue Nov 12, 2015
@rhiever rhiever self-assigned this Nov 12, 2015
rhiever added a commit that referenced this issue Nov 12, 2015
These are just some basic unit tests. There’s certainly room for more.
@rhiever rhiever assigned rasbt and unassigned rhiever Nov 12, 2015
@rhiever
Copy link
Contributor Author

rhiever commented Nov 12, 2015

Unit tests are up. Let's see that shmexy Travis CI integration now, @rasbt? :-)

@rasbt
Copy link
Contributor

rasbt commented Nov 12, 2015

Okay cool. I will setup the files tonight when I get hope. Should be pretty quick. But you need to do 1 more thing since you are the owner: give travis permission to access your repo for the testing. Just realized there is one little problem: this is a private repo. I am sure that you don't want to pay for travis pro, do you? :D https://travis-ci.com/plans But I see that there is a trial for "pro." Maybe we could use the free trial temporarily until the repo is public and then downgrade? alternatively we could just wait until it's public (but I can already set up the files tonight).

@rhiever
Copy link
Contributor Author

rhiever commented Nov 12, 2015

Well, since the repo is probably going public tomorrow, maybe we'll just set it up then?

@rasbt
Copy link
Contributor

rasbt commented Nov 12, 2015

jap okay! I think it's not that urgent anyway. As long as your local unittests pass there is actually no need for travis. It's just a convenience service for users to see if the current dev version is "passing" and if new contributions are "passing". Let's wait until tomorrow then.

@rasbt
Copy link
Contributor

rasbt commented Nov 16, 2015

I think we can close this one after merging #19 , #20 and #21

This was referenced Jul 20, 2018
weixuanfu added a commit that referenced this issue Apr 12, 2019
@andife andife mentioned this issue Oct 3, 2020
This was referenced Aug 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants