David de la Iglesia Castro

Results 309 comments of David de la Iglesia Castro

> This is what Facebook's Hydra does, and it really is intuitive and clean. We are currently exploring different ways to integrate with Hydra. This feature is part of the...

https://github.com/iterative/dvc/pull/8187 is adding support for using [Hydra syntax](https://hydra.cc/docs/advanced/override_grammar/extended/#sweeps) in `--set-param`, so: ``` dvc exp run -S 'a=1,2' --queue ``` Will put 2 experiments in the queue, that can be later...

> [iterative/dvc-render#8](https://github.com/iterative/dvc-render/issues/8) is an example where we would have to update the list of templates in a few places? I think we should just replace the [hardcoded names](https://github.com/iterative/dvc/blob/954de0cae44e31dfcf45630a2cc7e7f5acc7f00c/dvc/commands/plots.py#L225-L232) with something...

Two related issues on discord: https://discord.com/channels/485586884165107732/485596304961962003/932563413777924096 https://discord.com/channels/485586884165107732/485596304961962003/932663444329615420

Note that the last time I tackle this the conclusion was that `dvc remove` semantics were (more) unclear with this additional feature and decided to drop the P.R. (see https://github.com/iterative/dvc.org/pull/2357#issuecomment-828417049)

I think https://github.com/iterative/dvc/issues/6485 is a (long) related discussion

Was about to comment this in #5392 (haven't seen any other related issues). I think that the terms `Old` / `New` used in `diff` sometimes could be confusing, given that...

Two questions: A) Should `Change` be included by default ?? I might be biased but, in my personal experience (and current docs examples), the "Change" value is kind of awkward...

> @daavoo @jorgeorpinel Is there anything left to do on this issue? Showing `Change` in `dvc params diff` (If we agree that `Change` should be showed by default)

> Params are not always numeric, so I'm not sure what should be shown in `Change`. Maybe it's a boolean for non-numeric params? We are showing `Change` in the params...