check for config.yaml or config.json? also one set of config files is yaml and other json. We should avoid this. data_config is json while config is yaml

microsoft / dstoolkit-mlops-v2

This repository contains the basic repository structure for machine learning projects based on Azure technologies (Azure ML and Azure DevOps).

Other

20 stars 10 forks source link

check for config.yaml or config.json? also one set of config files is yaml and other json. We should avoid this. data_config is json while config is yaml #107

Open lorrinferdinand opened 8 months ago

lorrinferdinand commented 8 months ago

          check for config.yaml or config.json? also one set of config files is yaml and other json. We should avoid this. data_config is json while config is yaml

_Originally posted by @ritesh-modi in https://github.com/microsoft/dstoolkit-mlops-v2/pull/97#discussion_r1520876026_

sbaidachni commented 8 months ago

@lorrinferdinand I don't think that we have a problem there. In real life json file as well as data pipeline will be deleted. We have it here just for people who would like to setup the template as is, and json file is a method to pass parameters to the script as a simplest hardcoded way that separates it from real configuration required by the template.