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

require any hyphenation to be included in the parquet prefix #842

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jameshalgren
Copy link
Contributor

In the t-route configuration file, to meet the requirements for input to the TEEHR evaluation system, the ids need to be tagged according to their hydrofabric element type. The prefix_ids element allows for this to be set.

This PR allows the prefix to be completely arbitrary, removing the hard-coded hyphen '-'.

The suggested output change to be added to the end of the yaml file after the stream_output section is now:

  parquet_output:
    parquet_output_folder: outputs/troute/
    configuration: short_range
    prefix_ids: wb-

ping @mgdenno @karnesh

@jameshalgren
Copy link
Contributor Author

Note:
The output folder in the example .yaml input is configured as found in the standard file path for NextGen-based t-route simulations generated with the NGIAB_Data_process application and simulation with the NGIAB_CloudInfra package.
https://github.com/CIROH-UA/NGIAB_data_preprocess
and
https://github.com/CIROH-UA/NGIAB-CloudInfra,
respectively.

Also means that if you want a hyphen, it must be included.
See example in LowerColorado_TX/test_AnA_V4_NHD.yaml
@jameshalgren jameshalgren force-pushed the remove_hyphen_from_parquet_prefix branch from e70b09e to 6b19d67 Compare September 5, 2024 12:58
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

Successfully merging this pull request may close these issues.

1 participant