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

crossing missing from upstream access report bcfishpass.crossings_upstream_access #356

Closed
smnorris opened this issue Jul 13, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@smnorris
Copy link
Owner

Wilsey Spillway Dam is missing from the report output:

select ch_cm_co_pk_sk_belowupstrbarriers_network_km  from bcfishpass.crossings_upstream_access where aggregated_crossings_id = 'd5264acf-917b-43af-9c31-14d70733f81c';
 ch_cm_co_pk_sk_belowupstrbarriers_network_km
----------------------------------------------
(0 rows)

It is present in the upstream habitat report.

@smnorris smnorris added the bug Something isn't working label Jul 13, 2023
@smnorris
Copy link
Owner Author

This is a point to stream join issue. The inner join to streams in the at_point CTE is the issue : https://github.com/smnorris/bcfishpass/blob/access_report/model/01_access/sql/crossings_upstream_access_load.sql#L91

Modifying this to a left join might be all that is required but a much better fix would be to add a gradient at point function to fwapg as per smnorris/fwapg#142 rather than trying to get the gradient from two different stream segments.

@smnorris
Copy link
Owner Author

fixed with 821ca9d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant