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

guide: confusion between params (templating) and param dependencies #2412

Closed
shcheklein opened this issue Apr 23, 2021 · 1 comment
Closed
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: guide Content of /doc/user-guide p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately.

Comments

@shcheklein
Copy link
Member

shcheklein commented Apr 23, 2021

https://stackoverflow.com/questions/67215839/use-parameters-from-additional-configs-in-dvc-2-0/67221065#comment118833797_67221065

In link it says "To load additional params files, list them in the top vars, in the desired order, e.g.:". This is where my initial confusion originated from.

We need to be explicit that those could not (?) be used as dependencies.

(that's comes back to the discussion on parametrization and params - two terms overlap and confuse people)

@shcheklein shcheklein added type: enhancement Something is not clear, small updates, improvement suggestions A: docs Area: user documentation (gatsby-theme-iterative) labels Apr 23, 2021
@jorgeorpinel jorgeorpinel changed the title ug: fix confusion params (templating) and param dependencies guide: confusion between params (templating) and param dependencies Apr 25, 2021
@jorgeorpinel
Copy link
Contributor

jorgeorpinel commented Apr 25, 2021

Yep, we discussed this. AFAIR we don't want to separate the concepts? Params are params whether you use them as substitutions in a dvc.yaml template or as stage dependencies.

But since both uses go in dvc.yaml in different ways, we can def. be more explicit in the UG, would that be enough to solve this issue, or do we want to revisit the terminology itself?

@iesahin iesahin added the C: guide Content of /doc/user-guide label Oct 21, 2021
@jorgeorpinel jorgeorpinel removed the type: enhancement Something is not clear, small updates, improvement suggestions label Dec 22, 2022
@dberenbaum dberenbaum added the p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately. label Oct 16, 2023
@dberenbaum dberenbaum closed this as not planned Won't fix, can't repro, duplicate, stale Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: guide Content of /doc/user-guide p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately.
Projects
None yet
Development

No branches or pull requests

4 participants