Apply product feedback to dvc.yaml snippets

[mattseddon]

Changes based on feedback in #4194 (comment)

Demo

Screen.Recording.2023-07-11.at.10.19.40.am.mov

[mattseddon]

[F] After removing metrics params where the only difference between this and the snippet above. I didn't think it made sense to keep it around.

[dberenbaum]

Thanks @mattseddon!

params still doesn't look right to me. From https://dvc.org/doc/user-guide/project-structure/dvcyaml-files#parameters, it should fit one of these patterns:

    params:
      - threshold # track specific param (from params.yaml)
      - nn.batch_size
      - myparams.yaml: # track specific params from custom file
          - epochs
      - config.json: # track all parameters in this file

The items under params: can't be simply filenames. Each item in the list must be either a parameter name or a filename dict with parameter values (or an empty dict to include all parameter values from the file).

[mattseddon]

Thanks @mattseddon!

params still doesn't look right to me. From https://dvc.org/doc/user-guide/project-structure/dvcyaml-files#parameters, it should fit one of these patterns:

    params:
      - threshold # track specific param (from params.yaml)
      - nn.batch_size
      - myparams.yaml: # track specific params from custom file
          - epochs
      - config.json: # track all parameters in this file

The items under params: can't be simply filenames. Each item in the list must be either a parameter name or a filename dict with parameter values (or an empty dict to include all parameter values from the file).

updated to give the simplest example

[codeclimate]

Code Climate has analyzed commit 12fad3b and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (85% is the threshold).

This pull request will bring the total coverage in the repository to 95.1% (0.0% change).

View more on Code Climate.