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
Refactor of YAML config file #147
Comments
Sounds like a good plan to improve the config file. I would however vote against the json schema since it doesn't support comments. In big config files with plenty of upgrades, comments help a lot. VSCode does have a plugin for yaml validation and it works pretty well. And, supporting both schemas might create more confusion than solve, I think. |
Generalize the reporting measures & standardize res/com approach as much as possible@nmerket Over the last few projects we've done w/ ComStock, we've needed to add project-specific reporting measures. The most expedient approach was to be hard-coding them into the 1. Standardize the
|
After further investigation, I think option b) is actually required unless we re-write bsb to do the options_lookup.tsv parsing/translation to actual measure inputs |
@asparke2 Interesting ideas. I'm working on the first part of this now, but I like this more generalizable approach. |
Given the organic growth of the config (yaml) file to include support for additional compute environments and analysis types, there's some confusing ways of specifying details. This seems like a good time to reconsider how we organize the YAML file. Here are my current thoughts:
output_directory
under the compute environment since it varies by compute environment.postprocessing.s3
andaws.s3
.cc @rHorsey @joseph-robertson @rajeee
The text was updated successfully, but these errors were encountered: