Skip to content

Conversation

acrellin
Copy link
Member

@acrellin acrellin commented Feb 20, 2018

Closes #243
Closes #240

@pep8speaks
Copy link

pep8speaks commented Feb 20, 2018

Hello @acrellin! Thanks for updating the PR.

Line 150:81: E501 line too long (81 > 80 characters)

Line 180:81: E501 line too long (86 > 80 characters)

Comment last updated on May 08, 2018 at 18:18 Hours UTC

@acrellin acrellin force-pushed the upload_precomputed_features branch from ea36000 to 59bcccd Compare February 21, 2018 00:17
@acrellin acrellin force-pushed the upload_precomputed_features branch from e42f9df to 63fa55b Compare April 12, 2018 17:50
@acrellin acrellin added this to the v1.0 release milestone Apr 16, 2018
@acrellin acrellin force-pushed the upload_precomputed_features branch 2 times, most recently from 11421b6 to 4c31a07 Compare May 2, 2018 20:09
@acrellin acrellin force-pushed the upload_precomputed_features branch from aaf45fa to 2ded9d5 Compare May 7, 2018 18:27
@acrellin
Copy link
Member Author

acrellin commented May 8, 2018

Finally got the tests passing - ready for review @stefanv

@acrellin acrellin requested a review from stefanv May 8, 2018 18:36
@stefanv
Copy link
Contributor

stefanv commented May 8, 2018

Excellent work, Ari!

@stefanv stefanv merged commit 124d803 into cesium-ml:master May 8, 2018
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.

3 participants