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

Added a couple of notebooks with some rough analysis #162

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

fjaviersanchez
Copy link

No description provided.

@heather999
Copy link
Collaborator

heather999 commented May 14, 2018

In anticipation of the tentatively scheduled "Working at NERSC" DESC seminar on May 30th - what do we need to do to get this merged? Who should be tasked for review @jchiang87 ? Is there any need for a more current sims releases using a more recent stack weekly @fjaviersanchez ? Currently sims uses w_2018_13. We have jumped to w_2018_18 for the DRP processing, and it's always possible we will push ahead once more as needed. I'll have to inquire with IN2P3.
@danielsf are there any plans for a new sims release in the next week or so?

@fjaviersanchez
Copy link
Author

fjaviersanchez commented May 15, 2018

@heather999 from sims it needs the rotator functions that (I think) @danielsf implemented. For the "basic" depth analysis that's it. For the LSS-like analysis it needs some non-standard (not pip installable) things but it doesn't use the stack at all. I can try to change the code to reduce the number of dependencies or send you the instructions I followed to install this software :). Please, let me know what's best!

@heather999
Copy link
Collaborator

Hi @fjaviersanchez How about instructions for installing the software and then I'll give it a try - then we can go from there!
Thank you!

@fjaviersanchez
Copy link
Author

@heather999 I added instructions for install here: https://github.com/LSSTDESC/DC2_Repo/blob/issues/%23161/Validation/Requirements_and_install.md

I was planning to just go over the notebook called DC2_matching_and_depth.ipynb which shows how to access the data using the data butler, and also shows some data manipulation. I think that you already have everything installed on the shared stack if we just use that notebook.

@heather999
Copy link
Collaborator

Hi @fjaviersanchez Great thanks! I've since installed the latest sims at NERSC and started to really run the DC2_matching_and_depth.ipynb notebook. I ran into an error though, I'll attach a screenshot. Have you seen that before?
Take care,
heather
notebook-err

@fjaviersanchez
Copy link
Author

Sorry, it's a bug! I cleaned too many cells... I am updating it now :)

@fjaviersanchez
Copy link
Author

@heather999 it should be fixed now. Sorry it took so long.

@heather999
Copy link
Collaborator

Hi @fjaviersanchez just wanted to let you know the latest version of the notebook is working well when I'm using sims 2_8_0+lsst_distrib w_2018_19. I do see some warning that I wanted to check with you, to see if you see the same. I'll attach 3 images. Do you see the same?
butler
line28
line25

@fjaviersanchez
Copy link
Author

I've seen the last two warnings but I haven't seen the first one using an older version of the shared stack install at NERSC. I think they aren't very worrisome but I'll take a look to see if I can get rid of them.

@yymao yymao added this to the DC2 Tutorials milestone Jul 6, 2018
@yymao yymao removed this from the DC2 Tutorials milestone Jul 24, 2018
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