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

LSST DESC Note: The Twinkles1 PhoSim Pipeline and Results #451

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

Conversation

drphilmarshall
Copy link
Contributor

Hi @TomGlanzman! I started an LSST DESC Note for you, check it out:

https://github.com/LSSTDESC/Twinkles/blob/twinkles1-phosim-pipeline-note/doc/LSST_DESC_Notes/desc-0000-twinkles-1-phosim-pipeline/main.rst

As you can see, the development branch for this note is called twinkles1-phosim-pipeline-note. We won't merge this PR until you're happy with the Note, and the group has reviewed it. I don't think this needs to be a long Note, but it should include listings of the commands that you used to execute the Run 3 image simulation jobs, I think - and then some results of the kind you've been posting to the issue threads (on job success rate, wallclock vs CPU time, long jobs, and so on). What do you think?

@coveralls
Copy link

Coverage Status

Coverage remained the same at 41.35% when pulling feb2a26 on twinkles1-phosim-pipeline-note into e0da20a on master.

@TomGlanzman
Copy link
Contributor

@drphilmarshall That all sounds very reasonable.

@drphilmarshall
Copy link
Contributor Author

Rock on! Feel free to use this thread to ask me, @danielsf, @tony-johnson or anyone else for input / help.

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

Successfully merging this pull request may close these issues.

3 participants