Skip to content
This repository has been archived by the owner on Dec 11, 2024. It is now read-only.

Adding validation check for meta files #39

Open
arjunsuresh opened this issue Jan 26, 2023 · 3 comments
Open

Adding validation check for meta files #39

arjunsuresh opened this issue Jan 26, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@arjunsuresh
Copy link

CM scripts can fail with unexpected errors if there is a syntax error in _cm.json/_cm.yaml files. It'll be good to add a quick syntax check for these files.

@arjunsuresh arjunsuresh added the enhancement New feature or request label Jan 26, 2023
@gfursin
Copy link

gfursin commented Jan 26, 2023

Interesting. I think scripts will report an error in the _cm.json/_cm.yaml file if there is a syntax error? Do you mean that we should we make error more user-friendly? Or you want some extra syntax check? We can discuss it during the conf-call ... Thanks!

@arjunsuresh
Copy link
Author

Hi @gfursin , actually I'm referring to the internal semantics specific to CM like the type of tags being a string, env being a dict, deps being a list, adr being a dict etc. Sure, we can discuss this.

@gfursin
Copy link

gfursin commented Jan 28, 2023

Oh, yes, that's very important! The idea of CK/CM is to enable very quick prototyping of ideas while gradually adding tests and safety including information about types. There are several ways of doing it. Let's discuss it at one of our future conf-calls. I agree that it becomes important now. Thanks a lot!

@arjunsuresh arjunsuresh transferred this issue from mlcommons/ck Sep 18, 2024
@arjunsuresh arjunsuresh transferred this issue from mlcommons/cm4mlops Dec 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
Archived in project
Development

No branches or pull requests

2 participants