mwilensky768 / SSINS

BSD 2-Clause "Simplified" License
6 stars 7 forks source link

Plot revamp #115

Closed mwilensky768 closed 2 years ago

mwilensky768 commented 2 years ago

Redo a few things in the plotting code to avoid using deprecated matplotlib functionality. Would close #105

Description

matplotlib warns against using FixedFormatter for ticks without using FixedLocator. Even though it's never been a problem so far, I've made the plotting code explicitly use FixedLocator when using FixedFormatter (which is implicitly called when handing e.g. set_xticklabels a list or other sequence). It also warns against modifying the globally registered colormap. I've thus made all colormap alternations operate on a copy of the global cmap object.

Checklist:

Main Code Body Checklist:

Scripts Checklist:

codecov-commenter commented 2 years ago

Codecov Report

Merging #115 (68cc13a) into master (390aa9b) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #115   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines          854       862    +8     
=========================================
+ Hits           854       862    +8     
Impacted Files Coverage Δ
SSINS/Catalog_Plot.py 100.00% <ø> (ø)
SSINS/plot_lib.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.