Closed padilla410 closed 2 years ago
Also, cannot express enough how easy to follow and jump in you make these PRs! π₯°πβ€ love it
Fabulous. I am going to accept your recommendation and add a bit more context to the "overview" at the top of the PR. Then I will merge.
Closes #289
I manually updated lat/longs for the following reservoirs from the University of MO data set:
The provided lat/longs that came along with the temperature data did not intersect with NHDHR because they were located on land. I manually selected new lat/longs using google maps. I then manually updated
1_crosswalk_fetch/in/UniversityofMissouri_2017_2020_Profiles.csv
and updated the following explainer files (UniversityofMissouri_****_Profiles_explainer.docx
) on google drive.To verify the work, I completed 3 checks:
all_coop_dat_linked.feather
and then built the pipeline through
8_viz
Verification that the lat/longs are plotting appropriately
Lake Wapapello
Table Rock Lake (the zoomed in map labels are wrong on this one)
Verification that the NHDHR crosswalk includes eight lakes
Verification of record update in
all_coop_dat_linked.feather
Checking on the number of records in the final dataset:
We can see that the four additional lakes are in
all_coop_dat_linked.feather
when we compare the old data to the new:Digging in to why we don't have 8 lakes in
all_coop_dat_linked.feather
:I did a quick check on the files in
UniversityofMissouri_LimnoProfiles_2017_2020.zip
and verified that "Missouri_100" (aka Bull Shoals) is not present in the unparsed data set.Results from
8_viz
This is the updated summary table from
lakes_summary_fig.html
: This summary table is quite a bit different than my most recent run in PR #338 (9,369 vs 8,672 GLM lakes). I have not been able to ID the PR where the number of GLM lakes increased.But, to verify what I've done here, I have a few screenshots.
Table Rock Lake (2022-05-09):
Table Rock Lake (2022-05-19):
Wapapello Lake (2022-05-09):
Wapapello Lake (2022-05-19) π :