catch an exception in metsis_ts_bokeh.utils.inc due to new plotting API #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
I added an hackish ' if statment ' which temporary handles the new plotting API to support the plotting of vertical profiles. In such case the get `adc_get_ts_bokeh_plot_vars` will return `x_axis` instead of `y_axis` based on the NetCDF dimensions.
catch an exception in metsis_ts_bokeh.utils.inc due to new plotting API
|
my hack seems doesn't work .. neither with the unset variables |
|
seems the module use some cache system .. which prevent the list of vars to be updated when using a different netcdf resource. testing as admin it seems to work, while non-admin doesn't work .. .and uses a cached version of last url |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I added an hackish ' if statment ' which temporary handles the new plotting API to support the plotting of vertical profiles.
In such case the get adc_get_ts_bokeh_plot_vars will return x_axis instead of y_axis based on the NetCDF dimensions.
the hack works but one time only .. If I use a necdf that reurns a non empty y_axis .. when trying again the netcdf which requires the x_axis .. it doesn't work anymore. seems the variables are cached in the system.
To get it to work I added unset($y_vars); at the end after the loop.