Update develop-ref after 1528 #1571
Merged
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
* first stab at converting to sphinx * removing all slashes * adding new link to README.rst file * working on lists * Made formatting changes * Finished fcst section * fixing spelling, bolding and italics issues * updating web links * working on formatting * updating formatting * formatting * first attempt to clean up formatting completed. * adding README to the index file * fixing warning errors * Bringing README_TC into sphinx. Updating section headers * Adding README_TC * Made formatting updates to README.rst * corrected section under wavelet * small changes * removing met/data/config/README since it is now in met/docs/Users_Guide * Added some formatting for headers * fixing chapters & sections * Fixed warnings from building Co-authored-by: Julie.Prestopnik <[email protected]>
…entoried. This helps debugging issues with MET potentially not reading all input GRIB2 records on WCOSS.
PR #1550 broke the build. It removed the data/config/README file but left a reference to it in Makefile.am. I'm removing that reference directly in the develop branch to get the Docker build, nightly regression test, and nightly Fortify build working.
* first stab at converting to sphinx * removing all slashes * adding new link to README.rst file * working on lists * Made formatting changes * Finished fcst section * fixing spelling, bolding and italics issues * updating web links * working on formatting * updating formatting * formatting * first attempt to clean up formatting completed. * adding README to the index file * fixing warning errors * Bringing README_TC into sphinx. Updating section headers * Adding README_TC * Made formatting updates to README.rst * corrected section under wavelet * small changes * removing met/data/config/README since it is now in met/docs/Users_Guide * Added some formatting for headers * fixing chapters & sections * Fixed warnings from building * adding in code blocks * removing slashes * changes * Made changes to formatting * removing For example code blocks * major updates * fist pass at document conversion complete. * cleaning up questions about dashes * Made some formatting modifications * Removing README_TC because it is being replaced by README_TC.rst in met/docs/Users_Guide * Removing the reference to the README_TC file * Making title capitalization consistent with README * Added a space in timestring * changing to 'time string' with a space between the words. * adding a link to the new README_TC location in met/docs/Users_Guide * Modified references to README and README_TC Co-authored-by: Julie.Prestopnik <[email protected]>
* Per #1562, add the same grid_diag fix for the develop branch. * Per #1562, removing the poly = CONUS.poly mask from GridDiagConfig_TMP. That settting masked a problem in the handling of missing data. Exercising the mask.poly option is tested in another unit test. This will change the output and break the nightly build, but that's good since we'll do more thorough testing.
…to print out some additional log messages that may help in debugging the intermittent file list failure.
* Per #1528, adding default config file for PlotPointObs tool. * Per #1528, added line_width to config file. * Per #1528, adding PlotPointObs config file and it actually compiles. Now I need to do all the work. * Per #1528, making some progress adding Observation objects and storing the unique locations in a set. Next, I need to parse the options from the config file. * Per #1528, add config constants for plot_point_obs options and update PlotInfo to include an on/off flag and colorbar flag. * Per #1528, update plot_point_obs config file options and plot_point_obs_conf_info.h/.cc to process them. * Per #1528, add a couple more config keys for Plot-Point-Obs. * Per #1528, add an obs_gc array to the default Plot-Point-Obs config file. * Per #1528, make a few local plotting functions global so that Plot-Point-Obs can call them. * Per #1528, lots of changes to Plot-Point-Obs. Getting closer. Still need to finish up coding, add tests, and update the documentation. * Per #1528, I found that PlotInfo.colorbar_spacing was never actually used in the code. So I removed it from the MODE and PlotPointObs config files and removed it from the PlotInfo object. * Per #1528, add timestring_to_time_t() utility function to vx_cal library. * Per #1528, cleanup... just removed commented out code. * Per #1528, update plot_point_obs.cc to parse/process the observation valid time correctly. * For #1528, change the library order to make the linker happy. * Per #1528, I had changed message_type to msg_typ but failed to updated the default config file. * Per #1527, making the usage statement slightly more concise. * Per #1528, changing the default line_color and fill_color entries to be empty arrays. * Per #1528, add in support for the convert and data censoring logic. * Per #1528, update the documentation for plot_point_obs to reflect the new usage. * 1528 Consider making the plotting options of plot_point_obs more configurable (#1559) * #1528 Added get_dim_size * #1528 Added qty_list * #1528 Get the quality flag string. Get the character dimensions from the variable * Per #1528, slight refomatting of source code for consistent line lengths. * Per #1528, setting the default dotsize to 1.0, as it was previously (not 10!). * Per #1528, fix logic so that if a fill color table is specified fill_point is set to true. Also adjust margins. * Per #1528, add a new call to plot_point_obs in unit_plot_point_obs.xml to exercise the new config file options. * Per #1528, only make the margins bigger if we're actually plotting a colorbar. Otherwise, retain the previous margin sizes. * Update met/data/config/PlotPointObsConfig_default Co-authored-by: jprestop <[email protected]> * Update PlotPointObsConfig_default * Update test/config/PlotPointObsConfig Co-authored-by: jprestop <[email protected]> * Consistent formatting * Per #1528, based on Michelle's feedback, add the -title string option to the plot_point_obs usage statement. Co-authored-by: John Halley Gotway <[email protected]> Co-authored-by: hsoh-u <[email protected]> Co-authored-by: jprestop <[email protected]>
* Per #1568, add python script to convert NCL *.rgb colormaps to MET *.ctable color tables. Also store 270 new or updated colortables that are the output from this script. Note that some of the existing colortables have changed ever so slightly. That's the result of some imprecise rounding earlier and more precise rounding now. * Per #1568, update and rerun the rgb2ctable.py conversion script to write a header at the top of each colortable file to list the source NCL colormap.
21 tasks
JohnHalleyGotway
changed the title
jprestop
approved these changes
Nov 18, 2020
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.
GitHub issue #1528 and PR #1560 changed the output from the unit tests and caused the NB to fail. However, all changes were expected. Here's the NB log file on kiowa:
/d1/projects/MET/MET_regression/develop/NB20201118/test_regression_20201118.log
New plot_point_obs output file:
plot_point_obs/nam_and_ndas.20120409.t12z.prepbufr_CONFIG.ps
Slightly modified existing plot_point_obs output files:
met_test_scripts/plot_point_obs/sample_pb.ps
plot_point_obs/gdas1.20120409.t12z.prepbufr_TMP_ADPUPA.ps
plot_point_obs/ndas.20120409.t12z.prepbufr.tm00.nr_G218.ps
Pull Request Testing
Describe testing already performed for these changes:
Confirmed that the new output looks good.
Recommend testing for the reviewer to perform, including the location of input datasets:
None.
Will this PR result in changes to the test suite? [Yes]
If yes, describe the new output and/or changes to the existing output:
As described above.
After merging, should the reviewer DELETE the feature branch from GitHub? [No]
Do NOT delete the develop branch!
Pull Request Checklist
See the METplus Workflow for details.
Select: Reviewer(s), Project(s), and Milestone