Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TD Tile Flow Plotting #472

Open
3 of 5 tasks
loriabendroth opened this issue May 23, 2022 · 4 comments
Open
3 of 5 tasks

TD Tile Flow Plotting #472

loriabendroth opened this issue May 23, 2022 · 4 comments
Assignees

Comments

@loriabendroth
Copy link

loriabendroth commented May 23, 2022

@akrherz , few things here:

  • remove "(Still in Development)" on this page: https://datateam.agron.iastate.edu/td/dl/#tab_wdata
  • Sum Values is by daily, monthly, yearly. Can you change to be same options as under Nitrate Load tool which are daily, weekly (Mon-Sun), monthly, and yearly?
  • change first sentence under Download Options to: "Download the figure by clicking on the three-bar menu symbol in the upper right corner of the plot."
  • data are not populating for MN_Wilkin1 yet should exist in the db. Can you check this? Same for MN_Wilkin2, NC_Washington, MO_Knox4, and possibly others (I didnt check all). Is an older version of data in backend?
  • Legend for saturated buffer plots have "By Buffer", "From Field", and "To Buffer" but several sites appear to not have any "By Buffer". Is it possible that this legend text is populating by mistake?
akrherz added a commit to akrherz/datateam that referenced this issue May 24, 2022
akrherz added a commit that referenced this issue May 24, 2022
address some TD tileflow plotting issues per #472
@akrherz
Copy link
Member

akrherz commented May 24, 2022

@loriabendroth That default data listing is too generous, since this plot only does discharge, I only find these sites with discharge data:

 "IA_Boone"=>"IA_Boone  (2016-06-09 to 2018-12-31) [935 days]",
 "IA_Grundy"=>"IA_Grundy  (2016-09-02 to 2018-12-31) [850 days]",
 "IA_Hamilton1"=>"IA_Hamilton1  (2011-01-01 to 2018-12-31) [2921 days]",
 "IA_Hamilton2"=>"IA_Hamilton2  (2016-01-01 to 2018-12-31) [1095 days]",
 "IA_Hamilton3"=>"IA_Hamilton3  (2013-06-06 to 2018-12-31) [2034 days]",
 "IA_Story1"=>"IA_Story1  (2016-06-14 to 2018-12-31) [930 days]",
 "IA_Tama"=>"IA_Tama  (2016-01-01 to 2018-12-31) [1095 days]",
 "IN_Tippecanoe"=>"IN_Tippecanoe  (2007-11-04 to 2018-12-31) [4075 days]",
 "MN_Wilkin3"=>"MN_Wilkin3  (2018-01-01 to 2018-12-31) [364 days]",
 "OH_Defiance2"=>"OH_Defiance2  (1999-05-21 to 2008-11-19) [3470 days]",
 "OH_Fulton"=>"OH_Fulton  (2000-12-07 to 2010-12-20) [3665 days]",
 "OH_VanWert"=>"OH_VanWert  (2001-03-12 to 2009-11-10) [3165 days]",

@giorgichi is that right?

@loriabendroth
Copy link
Author

@akrherz , discharge is not the variable of interest. Use "Tile Flow Filled (recommended)" as listed in the data export function. This should be similar to how you did Load as you would have used Nitrate N Load Filled (recommended)

@akrherz
Copy link
Member

akrherz commented May 26, 2022

Thanks, but looking at the tile_flow_filled column of the tile_flow_and_n_loads_data database table, I find no data for MN_Wilkin1 ?

@loriabendroth
Copy link
Author

@akrherz , good catch. I believe this is a mistake and will need to be corrected in the NAL version.

@giorgichi please provide your perspective and memory on this. Was there a reason you did not copy Wilkin1 values over from original to filled? The filled should be everything - measured and interpolated. But in Wilkin1 case, the filled columns are n/a. We can't keep it this way because then people will mistakenly not include that site. Or we need to add very clear statements about how to handle. Can you determine how best to handle and implement?

@akrherz can you move forward with the interface and have Wilkin1 in dropdown but no data currently?

akrherz added a commit to akrherz/datateam that referenced this issue May 27, 2022
akrherz added a commit that referenced this issue May 27, 2022
@loriabendroth loriabendroth changed the title Tile Flow Plotting TD Tile Flow Plotting Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants