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

Flowpath OFE to Field database schema is inadequate for fields crossing HUC12s #240

Open
1 of 2 tasks
akrherz opened this issue Jun 6, 2024 · 1 comment
Open
1 of 2 tasks
Assignees

Comments

@akrherz
Copy link
Member

akrherz commented Jun 6, 2024

Presently, the database join for the flowpath_ofes to fields relations is based on the (fbndid) field along with an annoying further join between flowpaths and fields on the (huc12, scenario) fields. I really want to get de-duplication running on my end before ACPF comes up with something more robust. So this schema is not going to suffice.

  • flowpath_ofes should use a foreign key to fields(field_id)
  • fbndid should be dropped from flowpath_ofes
  • The huc12 value in fields is super handy, but needs to go away as the value will not be unique to the field. This field is necessary to keep the import straight.
@akrherz
Copy link
Member Author

akrherz commented Jun 6, 2024

Presently, Forest DEP OFEs do not have associated fields. As a temporary hack, I added huc12 geom fields for each so that we have no orphans.

akrherz added a commit to akrherz/dep that referenced this issue Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant