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

Adding module that produces a subset of the data repository containin… #212

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rbiswas4
Copy link
Member

@rbiswas4 rbiswas4 commented Apr 7, 2016

script to create subsets of the data repository for smaller size.
new file: copyData.py

…g only a subset of visits.

	new file:   copyData.py
@rbiswas4
Copy link
Member Author

rbiswas4 commented Apr 7, 2016

See the issue described in #213

@drphilmarshall
Copy link
Contributor

Thanks @rbiswas4 ! This looks reasonable, but there's no demo notebook or markdown doc describing its use. Could be a good idea, if the script is to be run at SLAC or NERSC by someone else?

@drphilmarshall
Copy link
Contributor

@rbiswas4 What do you need here? Someone else to try and use the code, and document it in the process? Is this something that could be done as part of one of the LSST DESC Note write-ups?

@rbiswas4
Copy link
Member Author

@drphilmarshall Sorry, noticing this after a while. There is an example script using this at the bottom of the module (after the if __name__=='__main__' part). Should we make it a notebook?

@drphilmarshall
Copy link
Contributor

That's a good idea! Should be quick to make and test, no? Thanks!

On Thu, Sep 29, 2016 at 12:19 AM, rbiswas4 [email protected] wrote:

@drphilmarshall https://github.com/drphilmarshall Sorry, noticing this
after a while. There is an example script using this at the bottom of the
module (after the if name=='main' part). Should we make it a
notebook?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#212 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArY96Ntd9QIVaNCn6Tr4bzZPyYmIJlTks5qu2Z9gaJpZM4IBpfa
.

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.

2 participants