-
Notifications
You must be signed in to change notification settings - Fork 0
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
add compile_method flag and add other framework artifact types #40
base: main
Are you sure you want to change the base?
Conversation
@fmigneault I added mlm:compile_method to the JSON Schema but this seems to have broken validation of
I went through and added
Any tips on what I need to change in the schema? I want it to be allowed under the asset role, if it is present I want it validated to be a string, but unlike |
Description
This adds a compile_method to the MLM Asset which identifies if the model artifact has related jit or aot compilation workflows applied to it (either when saving the artifact for aot or during runtime for jit).
It also reorganizes a lot of framework specific recommendations for describing model export formats to the best-practices.md . More descriptions for Tensorflow and Keras specific export formats are added as well.
Related Issue
Type of Change
Checklist
CONTRIBUTING.md
guide;make check
;Google
format for all the methods and classes that I used.