Skip to content
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

Open
arjunsuresh opened this issue Sep 19, 2024 · 8 comments
Open

Convert _cm.json to _cm.yml #257

arjunsuresh opened this issue Sep 19, 2024 · 8 comments

Comments

@arjunsuresh
Copy link
Contributor

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.

@gfursin
Copy link
Contributor

gfursin commented Sep 19, 2024

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 ...

@arjunsuresh
Copy link
Contributor Author

Sure Grigori.

@anandhu-eng
Copy link
Contributor

Hi @arjunsuresh @gfursin , are either of you working on or modifying scripts that include _cm.json? If so, I can open the PR after your work is completed.

@gfursin
Copy link
Contributor

gfursin commented Oct 3, 2024

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 ...

@gfursin
Copy link
Contributor

gfursin commented Oct 3, 2024

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!!!

@gfursin
Copy link
Contributor

gfursin commented Oct 3, 2024

@anandhu-eng - let's do #327 first and then do the json update ...

@anandhu-eng
Copy link
Contributor

Hi @gfursin , sure.

@gfursin
Copy link
Contributor

gfursin commented Oct 3, 2024

Thank you very much @anandhu-eng !!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

No branches or pull requests

3 participants