-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add example dataset to new projects #181
base: master
Are you sure you want to change the base?
Conversation
Hello @MichaelTamaki! Thanks for updating the PR.
Comment last updated on May 01, 2017 at 23:10 Hours UTC |
What's the motivation for this? |
@acrellin This main purpose for this is to provide an example dataset for the click through tutorial to use. |
cesium_app/models.py
Outdated
'tests', 'data', 'asas_training_subset_classes.dat') | ||
tarball = pjoin(os.path.dirname(__file__), | ||
'tests', 'data', 'asas_training_subset.tar.gz') | ||
header = shutil.copy2(header, cfg['paths']['upload_folder']) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only copy if not already in place: http://stackoverflow.com/a/82852/214686
Please add a test to ensure that the test data is in place. |
7cc0cab
to
14c3d7c
Compare
Reset branch to remove unnecessary commits |
14c3d7c
to
69e4ba2
Compare
0a7d5e7
to
220c1a9
Compare
Re-opening to show progress to @stefanv. After pressing the button I added, message_proxy crashes and I get 2 errors: |
220c1a9
to
9e27622
Compare
Button now has functionality, just needs tests and style |
afacff9
to
1640cac
Compare
@stefanv what's the status of this? |
I think this functionality would still be great to have; but the code has changed quite a bit, so we'll probably have to restructure some of it at least. |
36a6dcc
to
e4eb913
Compare
No description provided.