We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
A problem with ECHO_EXPORTER's CD113 field has been that there are many facilities with invalid CDs.
This script only produces real CDs for 118, an improvement on ECHO's data:
https://colab.research.google.com/drive/1y73SNnrd--Qp5m5SAPsLlIwyBQMCapkp
To make it part of the SBU database workflow, the script would have to be adjusted to load the scraped ECHO_EXPORTER CSV. It would probably be called in between the scrape and the import here: https://github.com/sunggheel/edgipgdb/blob/main/edgi_postgis/echo_scripts/10_scrapeECHOEPA
Additionally, we'd have to adjust the schema (one-time manual change?) and the views->materialized script here https://github.com/sunggheel/edgipgdb/blob/6afa24d4b5ec8207619eb5db1936892e5179790b/edgi_postgis/echo_scripts/viewsToMaterialized.py#L38 to include the new field.
The text was updated successfully, but these errors were encountered:
We've at least devised a way to make this information available for the report card process.
Sorry, something went wrong.
No branches or pull requests
A problem with ECHO_EXPORTER's CD113 field has been that there are many facilities with invalid CDs.
This script only produces real CDs for 118, an improvement on ECHO's data:
https://colab.research.google.com/drive/1y73SNnrd--Qp5m5SAPsLlIwyBQMCapkp
To make it part of the SBU database workflow, the script would have to be adjusted to load the scraped ECHO_EXPORTER CSV. It would probably be called in between the scrape and the import here: https://github.com/sunggheel/edgipgdb/blob/main/edgi_postgis/echo_scripts/10_scrapeECHOEPA
Additionally, we'd have to adjust the schema (one-time manual change?) and the views->materialized script here https://github.com/sunggheel/edgipgdb/blob/6afa24d4b5ec8207619eb5db1936892e5179790b/edgi_postgis/echo_scripts/viewsToMaterialized.py#L38 to include the new field.
The text was updated successfully, but these errors were encountered: