search

djk2120 / CLM5PPE

place for some of the preparatory work towards generating the CLM5 parameter perturbation experiment
MIT License
15 stars 12 forks source link

rank_plot changes #8

Open katiedagon opened 1 year ago

katiedagon commented 1 year ago

@djk2120 the modifications to rank_plot and top_n in analysis.py, specifically the indexing by minmax, appear to not work with the OAAT data format being read in by get_exp. I think this has something to do with how the minmax coordinate is set up. Do you have another processed data location for OAAT output that works with the new rank_plot function?

katiedagon commented 1 year ago

I found a few more issues with the current analysis.py e.g., reading h1 tape output with get_exp doesn't appear to work anymore.

The rank_plot issue is resolved by switching to the version you have in /glade/u/home/djk2120/ppe_tools - perhaps we could merge those changes into the version in this repo? Reading h1 tape doesn't appear to be fixed by this version though.