search

machine-shop / mothra

Automated butterfly wing length measurement
Other
27 stars 15 forks source link

Travis CI and Formatting #17

Closed ttran428 closed 5 years ago

ttran428 commented 5 years ago

Put in continuous integration.

Fixed formatting in previous sections.

ttran428 commented 5 years ago

Note: Currently 1 test is not passing due to using FFT rather than RFFT.

stefanv commented 5 years ago

In general, we keep issues related to one issue: "Add Travis-CI". Then do the formatting changes in another PR.

Remember the tip to use git add -p filename, to select parts of the file to add.

theobdt commented 5 years ago

Note: Currently 1 test is not passing due to using FFT rather than RFFT.

This issue is fixed in PR #21

ttran428 commented 5 years ago

@stefanv should I edit the pr so that it only makes those changes and move the other changes to a new pr?

stefanv commented 5 years ago

Please upload changes to the same branch, which will then be reflected on this PR.