Dave Berenbaum comments

Results 845 comments of


                                            Dave Berenbaum

data status: committed in index, not in cache entry shows up as modified

Thanks @skshetry! Any concerns with this or is it just a matter of figuring out the implementation?

data status: committed in index, not in cache entry shows up as modified

> There's another version of the commit that this issue is proposing: to consider both `index-commit` and `cache-commit` as one single commit, but only for the index, and that too...

flatten `deps` to facilitate more flexible templating

@daavoo Do you know if these scenarios work fine if not for YAML validation failures?

flatten `deps` to facilitate more flexible templating

If all the deps are a single list (`deps: ${those}`, as suggested in #8171), is additional work needed?

`exp run --set-param`: Doesn't support `-` in param name

My reading of the author's concerns appear to be limited to using `--` preceding a parameter name, not use of `-` anywhere in the parameter name. Are you reading it...

`exp run --set-param`: Doesn't support `-` in param name

@daavoo What do you think about adding this to the backlog since it is a regression in DVC?

`exp run --set-param`: Doesn't support `-` in param name

No, I meant contributing to hydra 😄 , but patching on our side is also fine.

`exp run --set-param`: Doesn't support `-` in param name

Got it. Let's follow in https://github.com/facebookresearch/hydra/issues/2363. Again, no problem with patching on our side for now if that's going to be quicker/easier.

`exp run --set-param`: Doesn't support `-` in param name

Should be fixed in the next hydra release

pull: how to only download data from a specified remote?

Discussed that first we should document the behavior better in push/pull, but we will also leave this open as a feature request.