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

docs: improve documentation of prep_techmix #557

Open
jdhoffa opened this issue Feb 22, 2024 · 0 comments
Open

docs: improve documentation of prep_techmix #557

jdhoffa opened this issue Feb 22, 2024 · 0 comments

Comments

@jdhoffa
Copy link
Member

jdhoffa commented Feb 22, 2024

Indeed, it should be checking for presence of "projected" value in metric column. We missed that. The benchmark is optional. I propose to reword it in the documentation to:

  • The column metric must have a "projected" value and
    a single scenario (e.g. "target_sds").
  • (Optional) The column metric may contain a benchmark
    (e.g. "corporate_economy").

Originally posted by @MonikaFu in #552 (comment)

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

2 participants