-
Notifications
You must be signed in to change notification settings - Fork 12
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 Notes documenting Twinkles 1 results #284
Comments
Alright, no objections to this documentation push so far! I'll go ahead and set up some files for us to work in, and ask @jonathansick from the LSST docs dept to follow along with this effort. He's helping the CI group start setting up a more general DESC Notes system, and, predictably, we are the guinea pigs :-) If you're interested in reading more, check out the epic at LSSTDESC/ComputingInfrastructure#24 and some further discussion in LSSTDESC/ComputingInfrastructure#28 . We'll also chat about the content of this set of DESC Notes at our meeting today. |
Hi Phil, I would say as a courtesy to the reader, and in future cases like this, reviewers, those should be chapters or sections in a single document describing Twinkles project. After thinking about things you might still want more than one (especially describing the analyses of results) but I don't want to have to download 11 documents to get an overview. I think you can still be sensitive to explicitly putting people's names on sections and you might imagine having multiple papers coming out of it, but I really only want to download, print, and read one thing on the airplane :) Just some feedback to consider. |
Thanks Chris! I agree "I want to read a lot on an airplane that doesn't have wifi" is a very important use case for the notes system. However, monoliths that are repeatedly cited as, eg, "LSST DESC (2016)" are a great way to fully disguise the individual contributions of the collaborations' members - and so I think it's worth us at least experimenting with distributing the Twinkles write-up over many separate small documents and then exploring ways to aggregate them. (It might also be that the overview you want is provided by the first Note in my proposed series.) @jonathansick - has this problem come up in the LSST technotes yet? Has anyone asked for the ability to make a compendium of notes X, Y and A through B, for example? |
From a citation perspective, I do want to promote citation of individual technotes for all the reasons you can imagine (give people credit, provide transparency and accuracy in the reference, ...). I haven't received a request yet for a compendium of technotes but I have gotten vocal calls for better PDFs and EPUBs. That's in my queue. I could imagine concatenating PDFs together from a technote series for offline reading. I could probably make that available from the planned documentation hub. |
One additional wrinkle: two of our proposed Twinkles Notes really belong in separate repos, I think. The LSST DESC Notes describing the This issue is about to become an Epic - I'll be editing the top comment to make it so, and then issuing the individual Notes to help us work on them. In our meeting we agreed an end-of-month deadline, which I will implement as a milestone for us. Hope this sounds OK! |
I wanted to follow up a bit on my previous comment about having too many tech notes for a single topic. I think the issue is more complicated/problematic than just "I want to read a lot on an airplane that doesn't have wifi". I'm coming at this from the perspective of someone who actually ran the review (with many people) of a high profile complicated analysis that had 4 or 5 notes to document the work. In those cases the work was really quite separate it really made sense to have them be in different documents. But, even in that case, having the multiple documents really complicated things. It is just so much easier (even if I am online) to have everything combined in one document. There is a single table of contents that allows you to flip around, you can refer to other sections, and you don't have to juggle 11 files around. I really believe there is a big advantage to having the minimum number of documents possible. In this Twinkles case, a lot of these should clearly really be chapters in the same document. I think a lot of this is being driven by the admirable desire to give credit to individual members especially for things that are being made available publicly so they can be cited etc. But, maybe we can think a little more creatively about how to handle that problem? There may be better ways than just splitting every document up into chunks associated with each author. Internally, I think this can be handled by listing all the authors on each tech note and clearly putting people's names on each section. Publicly, I think we need to think about it and maybe this is something the PB will address. You could imagine that public notes are different than internal notes and will be extracted from the main internal document. These would be different documents with more background each one clearly labeled with their authors for citation, or there may be the sort of technical solution that Jonathan proposed above etc. Anyway, I hope we can both address this important credit issue and not unnecessarily complicate our internal documentation simultaneously. Thanks! |
Agreed! :-) If web pages are well designed (or are hosted within a well-designed In the science roadmap latex PDF we attempted to solve the navigation
In the short term, and since this is an experiment, how about we press on |
@TomGlanzman I added a Markdown template to your folder (and everyone else's) for you to edit. If anyone wants latex, please let me know - or submit a pull request with an update to https://github.com/DarkEnergyScienceCollaboration/ComputingInfrastructure/tree/master/doc/LSST_DESC_Notes ... |
A Note to Note authors: as you get started writing your Notes, please note that for this experiment I have protected the Twinkles master branch, so that (I think) your Notes have to be added by pull request. This will ensure that we do actually do some sort of internal review via each PR, which should be interesting from a pub board perspective. Good luck! |
For what it is worth, I would not expect the Pub Board to be interested in following the internal review of DESC Notes, or journal papers. The draft Publication Policy assigns the Pub Board no role in reviewing research notes. "Research notes may be posted to a public web site maintained by the DESC (and possibly the arXiv as well) after they are reviewed for that purpose within the working group." Even for journal papers, the Pub Board will want to understand that a paper has been through internal review, but I suspect will be agnostic regarding the process. A pull request system (with the Review Committee members in charge?) might or might not be how the authors choose to draft papers. |
@sethdigel I like the idea of using pull requests to enable task-force/working group review of Notes. I'll rework our folders into |
PS. The checklist needs updating, based on our weekly meeting notes. |
OK, our note list is now up to date, in the top comment of issue 284. Feel free to edit, especially the preferred format if you have had a change of heart. |
@cwwalter is writing the requirements doc for DC1. He is focusing on the biggie, DC1 PhoSim Deep, as the main DC1 dataset, but wants to cite Twinkles 1. This means we need to provide some citeable objects for him: talking to @wmwv about this, we think we should write up some short DESC research notes, of the kind referred to in the proposed Publication Policy. I asked around a bit for guidance on how we might do this (thanks @nregnault @alexabate @kponder :-) ) and am thinking we might knock up a template at the Hack Day, @heather999 ? Doing this now should be easy (because we just gave a bunch of talks on our results) and should help a lot with specifying exactly what Run 3 (the main data generation run) will involve. What do you think?
Meanwhile, our job in the Twinkles TF is to blat our results into simple markdown files or jupyter notebooks, and check them in to the appropriate
doc
folders, using pull requests from the first authors to enable Task Force review. I guess this review will be led by our TF leaders, @richardxdubois and @wmwv.Here's the list of notes from our March 1 weekly meeting, complete with potential first authors (it's editable, please contribute changes here with explanations in comments below!).
rst
/tex
(with links to other notes) LSST DESC Note: Twinkles Design #299rst
(OpSim, CatSim, sprinkler concept, link to sims cookbook recipe)ipynb
(incl science motivation)ipynb
, (incl science motivation, ipynb) LSST DESC Note: The Twinkles 1 Strong Lens Population #450rst
LSST DESC Note: The Twinkles1 PhoSim Pipeline and Results #451ipynb
/rst
/tex
LSST DESC Note: Machine Learning Prediction of PhoSim CPU Time #452rst
(with links to cookbook recipes, and including workflow details. Results on NERSC performance in Run 2, 3 cf SLAC in Run 1.1, as well as astronomy measurements.) LSST DESC Note: The Twinkles Emulated DM Level 2 Pipeline and Results #453ipynb
LSST DESC Note on Pserv pserv#16ipynb
including examples of SNe and SL, with v brief analysis?ipynb
Other notes that could be written (add ideas here):
We'll all need to help out with each others notes as required, but there's a lot to be said for keeping the author list as small as possible - single author could be ideal for maximum credit visibility but 2-3 authors per note is often going to be needed, I guess.
To-do list:
start_paper
#413. This was done, but later reverted so we can use branches and PRs for reviews.The text was updated successfully, but these errors were encountered: