Update develop-ref after #1000 (cycloneplotter) #1211
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
… dictionaries in GridStat (#858)
…ip instead of custom version, added info to contrib guide about CI
Co-authored-by: George McCabe <mccabe@ucar.edu>
* Added section for ReadTheDocs, added basic info about the Release Guide and Verification Datasets Guide, but more information on those needs to be added later * Added information about the various versions available in the documentation * Updated the ReadTheDocs section with information on hidden branches and how to access them; Modified dtcenter.github.io references * Added section to add_use_case.rst about Accessing the Documentation and modified documentation.rst to indicate the docs can still be built manually * Modified format of list * Modified formatting * Changed GitHub Pages links to Read the Docs links * Modified formatting * Fixed typo * Updated Release Guide for Read The Docs * Took out duplicate data and consolidated sections * Update docs/Contributors_Guide/add_use_case.rst Removed sentence Co-authored-by: George McCabe <23407799+georgemccabe@users.noreply.github.com> * Update docs/Contributors_Guide/add_use_case.rst Fixed typo Co-authored-by: George McCabe <23407799+georgemccabe@users.noreply.github.com> * Update docs/Contributors_Guide/add_use_case.rst Removed end of previously removed sentence. Co-authored-by: George McCabe <23407799+georgemccabe@users.noreply.github.com> Co-authored-by: George McCabe <23407799+georgemccabe@users.noreply.github.com>
…push this change to all the METplus repos.
…ed to all the METplus-related repos.
Co-authored-by: George McCabe <mccabe@ucar.edu>
Co-authored-by: johnhg <johnhg@ucar.edu> Co-authored-by: George McCabe <mccabe@dakota.rap.ucar.edu>
Co-authored-by: George McCabe <mccabe@ucar.edu>
Co-authored-by: George McCabe <mccabe@ucar.edu>
…to configuration
…erence system), remove unneccessary code and comments, add support to read the central longitude from the config file.
* updated file names for use case * updated documentation for renaming, changing data * Update docs/Contributors_Guide/add_use_case.rst * split section into 2 sections for clarity * added info about removing old data from sample input data, added info about conda environments used for running use cases and automated tests, added text to make instructions more clear if one of two commands should be run * added missing parenthesis Co-authored-by: George McCabe <23407799+georgemccabe@users.noreply.github.com>
need to update reference after renaming a use case, affecting the tests
…ature_1000_cycloneplotter
…80 to center the map over the Pacific Ocean
… will always be set to 180 in the plotting script to center the map on the Pacific Ocean.
…ting, cleaning up code, moving hard-coded values from the plotting method into initialization section.
…ng required labels for development and official release.
…ls/cartopy#1401 and removing the original command of ax.gridlines(draw_labels=False, xlocs=[180, -180]) to prevent overlapping of 180E and 180W for central_longitude=180. Also fixed the sorting so the final ASCII (.csv) representation of the final data frame has been reindexed.
removed extra entry for met_tools to test the cycloneplotter wrapper earlier in the process.
update cartopy version from 0.17 to 0.18 (the 0.18 version "fixes" the issue with overlapping 180/-180 longitude labels when the central longitude is set to 180).
Feature 1000 cycloneplotter
bikegeek
added
type: bug
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.
#1000
replace .txt output file with a .csv file
changed the look of the plots:
-no longer have extraneous lines due to crossing the International Dateline
-no longer have overlap of longitude labels at -180/180 degrees when cartopy 0.18 is used