Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Record and Use Theta levels for the plot function and a few enhancements #16

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

delphinedobler
Copy link

Records and uses saved theta levels in plots in case of multiple calibration series. Additionally makes a few plots enhancements.

  • modify calculate_piecewisefit.m
    
    • 4.1 Record theta levels
  • modify plt_diagnostics_ow.m
    
    • 4.1 Use of theta levels from those now saved during calculate_piecewisefit function call removing lines with unused parameters (also corrects graph 2 and 4)
    • 4.2 For graph 6 and 8: Change indexes names by a more explicit terminology: usage of iseq, istep, ilevel, iplot instead of i,j,k ... Remove unused parameter lines. These changes were not marked done for the sake of lisibility
    • 4.3 graph format as a configuration parameters
    • 4.4 Enhance Graph 3: curate legend content, split title in two lines, add a grid, position the legend outside the graph.
    • 4.5 Automalically set graph 5 y-axis limits from selected thetas and set the upper bound to 14°C.

DD (2024/08/01) creation
If la_wmo_boxes_file is removed, automatically recreate it from climatology repository (useful when changing climatology version).
auto-creation of output directories if they do not exist already.
Includes:
3.1 - Performance: save boxes_id from get_region_ow and recompute the index to provide as input to retr_region_ow
3.2 - Performance: Inside retr_region_ow, save loaded data, and at each following profile, reload only new boxes and delete unused.
3.3 - Performance: Inside map_data_grid, comment calculation of vdataerror as this is never used afterwards.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant