Skip to content
Snippets Groups Projects

Consistent plot formatting

Merged Colm Talbot requested to merge gw-result-coverage into master

I noticed that the plotting in the gw result module wasn't being run in the CI which put the coverage pretty low.

This MR adds some testing to this module.

I had to change an option in the skymap plot. It looks like the API for the ligo.skymap.kde.Clustered2DSkyKDE changed at ligo.skymap@6ce94db4. Are there complaints with just changing this? ligo.skymap doesn't seem to be in the requirements anywhere.

Update: I set it so that the core plots now use the same basic formatting as the gw plots. This is basically just setting the font to Computer Modern Roman (the default latex font) and attempting to save all plots with usetex on.

Edited by Colm Talbot

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Colm Talbot added 1 commit

    added 1 commit

    • 60d40227 - remove latex formatting from ligo skymap

    Compare with previous version

  • Colm Talbot changed milestone to %0.6.4

    changed milestone to %0.6.4

  • Colm Talbot resolved all threads

    resolved all threads

  • Colm Talbot added 1 commit

    added 1 commit

    Compare with previous version

  • Colm Talbot added 1 commit

    added 1 commit

    • 0c16eb28 - move plot tests into a scheduled build

    Compare with previous version

  • Colm Talbot added 1 commit

    added 1 commit

    • e7a374a6 - temporarily add gw plot test

    Compare with previous version

  • Colm Talbot added 1 commit

    added 1 commit

    • a0a5f8f9 - install ligo skymap for gw plotting

    Compare with previous version

  • Colm Talbot added 1 commit

    added 1 commit

    • 648818dd - remove geo option from skymap test

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading