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.

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

Python Data Fetching #103

Open
davidthomas5412 opened this issue Oct 20, 2016 · 2 comments
Open

Python Data Fetching #103

davidthomas5412 opened this issue Oct 20, 2016 · 2 comments
Assignees
Labels

Comments

@davidthomas5412
Copy link
Collaborator

davidthomas5412 commented Oct 20, 2016

Problem:

  1. Fetch.csh requires C shell
  2. Have to fetch data for demos 'by hand'
  3. Have long paths, and very wide lines in editor, throughout the codebase, ex:

S = pangloss.Shearmap([PANGLOSS_DIR+'/data/GGL_los_8_0_0_N_4096_ang_4_rays_to_plane_37_f.gamma_1',PANGLOSS_DIR+'/data/GGL_los_8_0_0_N_4096_ang_4_rays_to_plane_37_f.gamma_2'],FITS=False)

Proposed Solution:
Write static python class that has methods that can (i) check whether files have been downloaded, (ii) downloads them, and (iii) has static variable for all the files. This will lead to a better user experience and only requires that the user has python.

@drphilmarshall
Copy link
Owner

Looks like this is coming along well over in your fork, David! Feel free to open the pull request into drphilmarshall/wl any time.

@drphilmarshall
Copy link
Owner

Good plan. I do this in my StatisticalMethods class all the time, you
shoudl be able to find starter code in the "examples" notebooks there, You
can also find it in the SLTimer project that I work on, in the
DarkEnergyScienceCollaboration GitHub repo.

On Thu, Oct 20, 2016 at 10:17 AM, David Thomas notifications@github.com
wrote:

Problem:

  1. Fetch.csh requires C shell
  2. It is annoying to fetch data for demos 'by hand'

Proposed Solution:
Write static python class that has methods that can (i) check whether
files have been downloaded, and (ii) downloads them. This will lead to a
better user experience and only requires that the user has python.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#103, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AArY9522zWMs2tjbqpgUUjnRc7A9Zk32ks5q16IjgaJpZM4KcYgl
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants