Skip to content

Conversation

acrellin
Copy link
Member

@acrellin acrellin commented Jan 26, 2018

Fixes #231
Fixes #232

@pep8speaks
Copy link

pep8speaks commented Feb 6, 2018

Hello @acrellin! Thanks for updating the PR.

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

Comment last updated on May 04, 2018 at 20:21 Hours UTC

@acrellin acrellin requested a review from stefanv February 12, 2018 21:55
@acrellin acrellin force-pushed the download_features_models branch from c68ea9a to 6afdb5c Compare February 15, 2018 23:13
@acrellin acrellin force-pushed the download_features_models branch from 6afdb5c to 9152e08 Compare April 12, 2018 18:13
@acrellin acrellin added this to the v1.0 release milestone Apr 16, 2018
@acrellin acrellin force-pushed the download_features_models branch 2 times, most recently from 2efa767 to 13b5297 Compare May 2, 2018 20:49
@acrellin acrellin force-pushed the download_features_models branch from 41ee78f to 0181115 Compare May 4, 2018 19:40
acrellin added 5 commits May 4, 2018 12:54
Improve downloaded featureset formatting
Add featureset/model/prediction name & timestamp to default download file name

Add project name to default download file names
@acrellin acrellin force-pushed the download_features_models branch from 0181115 to f53f6da Compare May 4, 2018 19:54
@acrellin acrellin merged commit bfca4eb into cesium-ml:master May 4, 2018
@stefanv
Copy link
Contributor

stefanv commented May 8, 2018

Thanks, Ari! What do you think: should we consider gzipped up/downloads of data in the future? Might make it quite a bit more efficient.

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