Pygrb injection plot formatting fixes

[a-r-williamson]

There have been a few plotting bugs in need of squashing.

• Make the colormap used for FAP colour scale colour-blind friendly and uniform perception. The colormap to use here appears to be cividis.
• Attempt to fix the odd y-axis scaling that sometimes happens in injection plots.
• Duplicate injection plots but with one version having missed injections at front, another with found at front
• Ensure plot titles fit in figure bounds
• Consider changing marker size in injection recovery plots for easier consumption

[pannarale]

Thanks, @a-r-williamson. It looks all fine. Will wait for the other 3 tasks to be done before merging.

[a-r-williamson]

Yeah I need to test things properly too, it's not ready.

[titodalcanton]

Hi @a-r-williamson, you could make the found-missed plot look like the one from the all-sky search for consistency (gwastro/pycbc#3218).

[a-r-williamson]

This PR is to address #54

[a-r-williamson]

Rebased to master after bugfix in #56.

[a-r-williamson]

As of now the injection recovery plots have significant changes to their appearance. The changes so far incorporate:

• A colourblind friendly colormap for FAP scale in found/missed injections, cividis_r (following Add colorbar labels to foundmissed plots. General tidy-up of the foundmissed script pycbc#3218).
• Axis scaling and figure bounds are also fixed.
• Each plot has now one version with found injections in foreground, and one with missed injections in foreground
• The marker appearance has been altered to improve function (comments/recommendations very welcome)

Together these changes turn...

...into...

The alternate version of this plot with found injections foregrounded is...

[pannarale]

Nice, you spared me some future work @a-r-williamson!

[a-r-williamson]

Small change made to order the filled circles by their FAP, like in all-sky search. The above two examples now become:

[a-r-williamson]

This will need corresponding changes to the result page, which I'll make next. I believe all required changes will be in pycbc_make_grb_summary_page and within pycbc.results, so if approved this can be merged.

[a-r-williamson]

Hi @pannarale, are you happy with how this is now?