LakeCat published comparison #92
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
debboutr
changed the title
mhweber
pushed a commit
that referenced
this pull request
Dec 15, 2021
* add print stmnt for LakeCat published comparison (#92) Co-authored-by: Debbout <Debbout.Rick@epa.gov> * Stream temp modelling (#102) * updates for nlcd 2019 rasters * update os.environ addition for arcpy, control table needs col to be float * update control table * run values at 0 causing problems * add temp-modelling layers * update check_metrics * remove duplicative lookup tables * update docstring for check_metrics command * add ability for alternate control table * implement click command * add temp-modelling layers * rename arg * add ability to use absolute path on landscape layer * add z_stats for public viewing * add script for float precision (#114) * add script for float precision * precision title change * adjust control table * update Published in Control, use HTTPS/bs4 for identifying published Co-authored-by: Debbout <Debbout.Rick@epa.gov> Co-authored-by: Tgephart <gephart.tyler@epa.govgit config --global --list configc --global --listgit config --global user.email gephart.tyler@epa.gov> Co-authored-by: Debbout <Debbout.Rick@epa.gov> Co-authored-by: Tgephart <gephart.tyler@epa.govgit config --global --list configc --global --listgit config --global user.email gephart.tyler@epa.gov>
debboutr
added a commit
that referenced
this pull request
Jan 13, 2022
* add print stmnt for LakeCat published comparison (#92) Co-authored-by: Debbout <Debbout.Rick@epa.gov> * Stream temp modelling (#102) * updates for nlcd 2019 rasters * update os.environ addition for arcpy, control table needs col to be float * update control table * run values at 0 causing problems * add temp-modelling layers * update check_metrics * remove duplicative lookup tables * update docstring for check_metrics command * add ability for alternate control table * implement click command * add temp-modelling layers * rename arg * add ability to use absolute path on landscape layer * add z_stats for public viewing * add script for float precision (#114) * add script for float precision * precision title change * adjust control table * update Published in Control, use HTTPS/bs4 for identifying published Co-authored-by: Debbout <Debbout.Rick@epa.gov> Co-authored-by: Tgephart <gephart.tyler@epa.govgit config --global --list configc --global --listgit config --global user.email gephart.tyler@epa.gov> * convert polys to crs of points instead of vice-versa * formatting * remove exit statement Co-authored-by: Debbout <Debbout.Rick@epa.gov> Co-authored-by: Tgephart <gephart.tyler@epa.govgit config --global --list configc --global --listgit config --global user.email gephart.tyler@epa.gov>
mhweber
pushed a commit
that referenced
this pull request
Feb 7, 2022
* remove year from NLCD lookup table * Point poly update (#115) * add print stmnt for LakeCat published comparison (#92) Co-authored-by: Debbout <Debbout.Rick@epa.gov> * Stream temp modelling (#102) * updates for nlcd 2019 rasters * update os.environ addition for arcpy, control table needs col to be float * update control table * run values at 0 causing problems * add temp-modelling layers * update check_metrics * remove duplicative lookup tables * update docstring for check_metrics command * add ability for alternate control table * implement click command * add temp-modelling layers * rename arg * add ability to use absolute path on landscape layer * add z_stats for public viewing * add script for float precision (#114) * add script for float precision * precision title change * adjust control table * update Published in Control, use HTTPS/bs4 for identifying published Co-authored-by: Debbout <Debbout.Rick@epa.gov> Co-authored-by: Tgephart <gephart.tyler@epa.govgit config --global --list configc --global --listgit config --global user.email gephart.tyler@epa.gov> * convert polys to crs of points instead of vice-versa * formatting * remove exit statement Co-authored-by: Debbout <Debbout.Rick@epa.gov> Co-authored-by: Tgephart <gephart.tyler@epa.govgit config --global --list configc --global --listgit config --global user.email gephart.tyler@epa.gov> * always write out control file from ListPublishedUnpublishedFiles.py Co-authored-by: Debbout <Debbout.Rick@epa.gov> Co-authored-by: Tgephart <gephart.tyler@epa.govgit config --global --list configc --global --listgit config --global user.email gephart.tyler@epa.gov>
mhweber
added a commit
that referenced
this pull request
Mar 21, 2022
* remove year from NLCD lookup table * Point poly update (#115) * add print stmnt for LakeCat published comparison (#92) Co-authored-by: Debbout <Debbout.Rick@epa.gov> * Stream temp modelling (#102) * updates for nlcd 2019 rasters * update os.environ addition for arcpy, control table needs col to be float * update control table * run values at 0 causing problems * add temp-modelling layers * update check_metrics * remove duplicative lookup tables * update docstring for check_metrics command * add ability for alternate control table * implement click command * add temp-modelling layers * rename arg * add ability to use absolute path on landscape layer * add z_stats for public viewing * add script for float precision (#114) * add script for float precision * precision title change * adjust control table * update Published in Control, use HTTPS/bs4 for identifying published Co-authored-by: Debbout <Debbout.Rick@epa.gov> Co-authored-by: Tgephart <gephart.tyler@epa.govgit config --global --list configc --global --listgit config --global user.email gephart.tyler@epa.gov> * convert polys to crs of points instead of vice-versa * formatting * remove exit statement Co-authored-by: Debbout <Debbout.Rick@epa.gov> Co-authored-by: Tgephart <gephart.tyler@epa.govgit config --global --list configc --global --listgit config --global user.email gephart.tyler@epa.gov> * always write out control file from ListPublishedUnpublishedFiles.py * Added functions to load existing StreamCat ftp csv tables to the Stre… (#117) * Added functions to load existing StreamCat ftp csv tables to the StreamCat API database * fix to populate tables function * Added function to list published / unpublished metrics in API * Updated delete table function and control table * updates to API * updates for StreamCat_API.py and ControlTable * updated control table Co-authored-by: debboutr <debboutr@gmail.com> Co-authored-by: Debbout <Debbout.Rick@epa.gov> Co-authored-by: Tgephart <gephart.tyler@epa.govgit config --global --list configc --global --listgit config --global user.email gephart.tyler@epa.gov>
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.
This PR updates the
ListPublishedUnpublishedFiles.pyscript to examine and update what StreamCatFinal_Table_Names are up on the FTP site. I have added a print statement that will show any FinalTables that are published in StreamCat but not in LakeCat.I also took a look at potentially consolidating the ControlTables for both repos and have found the following inconsistencies that need to be addressed in this PR or potentially others (comment below):