lsst-epo / citizen-science-notebooks

A collection Jupyter notebooks that can be used to associate Rubin Science Platform data to a Zooniverse citizen science project.
3 stars 1 forks source link

Create Utilities.py module to organize code #22

Closed bnord closed 1 year ago

beckynevin commented 1 year ago

I have submitted a pull request for this - branch name is 'Utils', it moves a lot of the functions that don't need to be seen out of the tutorial notebook. pull request

ericdrosas87 commented 1 year ago

Should I review this PR? @clareh mentioned that @jsv1206 was going to review some of the PRs, I'm just unsure of which one(s).

bnord commented 1 year ago

@ericdrosas87 I think the current PR is just notebooks (if I read correctly). Maybe we can ask you to review the PR when it's in full script?

@beckynevin do I have that right?

beckynevin commented 1 year ago

@bnord , the main thing I worked on in the PR is a .py script. I also moved some of this material out of the Citizen_Science_Testing.ipynb notebook.

beckynevin commented 1 year ago

I'm going to close this issue for now; we have now merged the utils branch with main. In the future, we'll probably have to update it when we merge the variable stars notebook