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

Update model to be a single object on kitfile #34

Merged
merged 1 commit into from
Feb 23, 2024
Merged

Update model to be a single object on kitfile #34

merged 1 commit into from
Feb 23, 2024

Conversation

gorkem
Copy link
Contributor

@gorkem gorkem commented Feb 22, 2024

Updates the build and export to adjust to single object.

DataSets []DataSet `json:"datasets,omitempty"`
Models []TrainedModel `json:"models,omitempty"`
ManifestVersion string `json:"manifestVersion"`
Kit ModelKit `json:"modelkit,omitempty"`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure on modelkit as a field, as model kit already refers to the OCI artifact. This is just a metadata field.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the earlier package was a reference to the data related to the whole package. So this is reference to model kit which is the whole package

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense 👍

pkg/cmd/dev/cmd.go Outdated Show resolved Hide resolved
Updates the build and export to adjust to single object.
@gorkem gorkem marked this pull request as ready for review February 22, 2024 22:35
@gorkem gorkem requested a review from amisevsk February 22, 2024 22:44
@gorkem gorkem merged commit 249f02d into main Feb 23, 2024
1 check passed
@gorkem gorkem deleted the single-model branch February 23, 2024 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants