LSSTDESC / SSim_DC1

Configuration, production, validation specifications and tools for the DC1 Data Set.
4 stars 2 forks source link

Review PhoSim DC1 configurations #20

Closed richardxdubois closed 6 years ago

richardxdubois commented 7 years ago

Given the long apparent running times for Twinkles, we thought it prudent to ask @johnrpeterson to review the PhoSim config files. @connolly, @TomGlanzman - could you please point John at the configs?

TomGlanzman commented 7 years ago

The most recent run of Twinkles run 1 was with the updated phoSim v3.5.2 code back in July. The command file (aka physics overrides) was common to all visits and is located here @SLAC:

/nfs/farm/g/desc/u1/Pipeline-tasks/Twinkles-phoSim-352/config/twinkles_I_physics_override.txt

The instanceCatalogs and SED files exist as compressed txt/tar files, respectively, and are located in this directory:

/nfs/farm/g/desc/u1/data/Twinkles/phoSim/Run1_InstCats_SEDs

For example, the files for the first visit are:

phosim_input_200.txt.gz and phosim_input_200.tar.gz

Together they are about 70 MB in size.

jchiang87 commented 7 years ago

A version of the physics override file is also in the Twinkles github repo: https://github.com/DarkEnergyScienceCollaboration/Twinkles/blob/master/workflows/PhoSim/twinkles_I_physics_override.txt We should make sure the copy in the Twinkles repo is up-to-date with the version used in the phosim runs.

johnrpeterson commented 7 years ago

please remove these two lines below. i’ve already checked that these will not cause DM zero-pointing intrachip errors, and LSS (and others) wants these on if they are quasi-realistic (which was fixed in v3.5).

the rest are: 1) detector details that DM is not fully equiped to handle or 2) keeping the sky background fixed so observations within several minutes of each other don’t have drastically different background. the latter we should perhaps remove for non-twinkles runs, but i don’t know the planned cadence for DC1 well enough to say for sure.

Also turn off clouds and airglow variation

    clearclouds
    airglowvariation 0

On Sep 27, 2016, at 1:01 PM, James Chiang notifications@github.com<mailto:notifications@github.com> wrote:

A version of the physics override file is also in the Twinkles github repo: https://github.com/DarkEnergyScienceCollaboration/Twinkles/blob/master/workflows/PhoSim/twinkles_I_physics_override.txt We should make sure the copy in the Twinkles repo is up to date with the version used in the phosim runs.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/DarkEnergyScienceCollaboration/SSim_DC1_Roadmap/issues/20#issuecomment-249928955, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJbT8q_zywQRhNKn8EooZLcg4n_bIkkJks5quUv4gaJpZM4KH0v6.

———————————

John R. Peterson Assoc. Professor of Physics and Astronomy Department of Physics and Astronomy Purdue University 525 Northwestern Ave. West Lafayette, IN 47906 (765) 494-5193

cwwalter commented 7 years ago

After the PhoSim run it was determined that an altitude line in the configuration file caused the sky background to be reduced by a pointing dependent value. Details on this issue can be found here.

https://bitbucket.org/phosim/phosim_release/issues/32/functional-dependence-of-sky-brightness-on

cwwalter commented 6 years ago

I'm not sure why this is still open...