Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • S spiir
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 73
    • Issues 73
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 14
    • Merge requests 14
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • lscsoft
  • spiir
  • Merge requests
  • !21

Cast values used as array idxs to int.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Fiona Panther requested to merge fix_postprocessing into spiir-O4-EW-development Nov 04, 2021
  • Overview 0
  • Commits 1
  • Pipelines 0
  • Changes 1

The postprocessing script modified here (gstlal_cohfar_plot_stats) is called when analyzing injection runs (there will be an error in the log files that automatically create the report pages), and is useful for assigning FARs when checking whether changes to background collection makes differences to the FAR assignment. The purpose of the script is to plot the FAR contours in the chisq-cohsnr parameter space.

Currently the program fails as several values that are used as array indicies are cast to floats, rather than integers.

These values have now been cast to integers, and the script is useable.

Request Andrew and/or Alex to sanity check before merging. I can provide demonstration of the point of failure.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix_postprocessing