-
Notifications
You must be signed in to change notification settings - Fork 20
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
Convert _cm.json to _cm.yml #257
Comments
I am ok with that. We may want to do some formatting of the YAML to make it a bit more readable but I think it can be done programmatically ... |
Sure Grigori. |
Hi @arjunsuresh @gfursin , are either of you working on or modifying scripts that include |
Hi @anandhu-eng. I am freezing our developments on flexaihq/cm4mlops until this change is done. However, I do believe we need to sync all branches (back and forth) as soon as possible: mlperf-inference <-> dev <-> main , otherwise we will have a total mess in a few weeks. We have enough time to test everything before main deadlines and move on with sub-projects ... |
I need to use and update mlperf-inference as well as dev but will also wait until we sync all branches and change json->yaml. Please tell me as soon as it's done to test everything and update my forks and branches ... Thank you very much for all those clean ups and syncs - very appreciated!!! |
@anandhu-eng - let's do #327 first and then do the json update ... |
Hi @gfursin , sure. |
Thank you very much @anandhu-eng !!! |
We should convert most/all of the _cm.json files to _cm.yml as having a consistent meta file format will make it easier for the development.
The text was updated successfully, but these errors were encountered: