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.

Sign up for GitHub

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

Jump to bottom

Suggestion: Add some automated testing like Travis CI, Circle CI, Appveyor #3

Open
cclauss opened this issue Sep 8, 2018 · 3 comments
Open

Suggestion: Add some automated testing like Travis CI, Circle CI, Appveyor #3

cclauss opened this issue Sep 8, 2018 · 3 comments

Comments

@cclauss
Copy link
Contributor

cclauss commented Sep 8, 2018

They are all free for Open Source projects like this one.

https://github.com/marketplace/category/continuous-integration
@yaroslavvb
Copy link
Collaborator

yaroslavvb commented Sep 8, 2018 •
edited
Loading

Ideal thing would be integration testing -- something that runs train.py and checks that results are still the same. But that requires someone paying for an Amazon account with lots of p3's available, and I'm planning to use my p3's for other fun things.

However I will still try to reproduce and fix any issues that come up in the next couple of weeks.

@cclauss
Copy link
Contributor Author

cclauss commented Sep 8, 2018

Lots of projects run train.py on Travis, Circle, AppVeyor.

@yaroslavvb
Copy link
Collaborator

yaroslavvb commented Sep 8, 2018 •
edited
Loading

Unfortunately this project is hardwired for a specific machine type -- p3.16xlarge AWS instance

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yaroslavvb @cclauss