USFS-PNW / Fia-Biosum-Manager

User interface and main code repository for Biosum
http://biosum.info/
Other
3 stars 3 forks source link

FVS: Update Post-Append Audit to check for records in FVS_CutTree table #297

Closed lbross closed 1 year ago

lbross commented 1 year ago

If sequence numbers are incorrectly configured, it results in an empty FVS_CutTree table which causes exceptions in the Processor tree species audit and calculations. We should add a first step to the Post-Append Audit (which is checking for valid Processor input) to check for records in FVS_CutTree. Some checks should also be added to the Processor modules to catch this case before throwing an exception, if the analyst doesn't run the audit before trying to run Processor.

lbross commented 1 year ago

Added validation to Post-Append audit, Tree Species audit, and Processor run tasks to check for records in FVS_CutTree before proceeding.

jsfried commented 1 year ago

The post-append audit works (when run on package 999 grow-only, it tells us that there are not cutlist records). However, it is unclear when this would come up in the tree species audit-- when there are NO packages that successfully generate cut list trees? Packages like grow only are already not showing up in the Processor package list so they can't be checked to run.

lbross commented 1 year ago

I believe this would come up when there are NO packages that successfully generate cut list trees. This was the case when Ryan encountered the error with his arizona_four_forests project. This use case would probably only happen when sequence numbers are bad. Using an old version of Ryan's project would test this contingency.

jsfried commented 1 year ago

Have tested an old 4fri project which has seqnums incorrectly loaded and thus no cutlist trees. Processor trapped for this.

lbross commented 1 year ago

From @sebbusby: I can confirm that specific piece of #297 is fixed, It’s no longer giving me a hard-stop, just a warning to proceed.