Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
L
lalsuite
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 108
    • Issues 108
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
    • Iterations
  • Merge Requests 28
    • Merge Requests 28
  • Requirements
    • Requirements
    • List
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Security & Compliance
    • Security & Compliance
    • Dependency List
    • License Compliance
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Code Review
    • Insights
    • Issue
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • lscsoft
  • lalsuite
  • Merge Requests
  • !1154

Closed
Opened Jan 06, 2020 by Salvatore Vitale@salvatore-vitaleDeveloper
  • Report abuse
Report abuse

LALInference to save command line to headers and fhigh to PP fixed output params

  • Overview 16
  • Commits 7
  • Pipelines 7
  • Changes 5

Save the actual command line used in the dagman in the header of each output hdf5 files, for both nest and MCMC.

These appear in the header sections of the PP pages:

Nest: https://ldas-jobs.ligo.caltech.edu/~salvatore.vitale/test/O3/PE_Rota/S190425z/C01/IMRPhenomD_NRT_lowspin/1240215503.02-330564/V1L1/posplots.html MCMC: https://ldas-jobs.ligo.caltech.edu/~salvatore.vitale/test/O3/PE_Rota/S190425z/C01/IMRPhenomD_NRT_lowspin/MCMC/1240215503.02-330564/V1L1/posplots.html

Also add fhigh to the summary table, together with flow and f_ref, that where already there.

Notes:

  1. I am sure that what I have done doesn't have any side effects for Nest. An MCMC expert (@vivien, @bfarr) should check the same is true for MCMC. I believe that my hack will make MCMC save the CL also in the resume files, since the same function seems to be used for intermediate checkpointing and for the final writing, but hopefully that is not a problem.
  2. A command line is printed for each parallel chain, if one runs with 40 parallel chains, as we sometimes do for very expensive approximants, that might get very long. Here an example with two parallel runs. We might want to find a way to only print the first one, I have not done that in this patch.
  3. As already the case for flow, only one value of fhigh is printed, not one for each ifo. As such, I am not really sure which one would be printed if the user picked different flow and fhigh for different IFOs. This should be fixable quite easily, but since we don't do that for flow, I have not done it for fhigh for this patch. Might want to discuss this in a PE call.
Assignee
Assign to
Reviewer
Request review from
O3 Point Release
Milestone
O3 Point Release
Assign milestone
Time tracking
Reference: lscsoft/lalsuite!1154
Source branch: better_lalinference_logging