Skip to content

DEVAI-147: Define catalog structure for model catalogs in RHDH#2

Merged
johnmcollier merged 8 commits intomainfrom
devai147
Sep 25, 2024
Merged

DEVAI-147: Define catalog structure for model catalogs in RHDH#2
johnmcollier merged 8 commits intomainfrom
devai147

Conversation

@johnmcollier
Copy link
Contributor

This PR updates the model-catalog readme to include the model catalog structure, that was previously documented in https://docs.google.com/document/d/1ArB8w-TawZ_DuMbZoukk4inVsAgqHntB8yN2lttwjyo/edit?usp=sharing.

The readme includes a table outlining the metadata for the model servers and models that are stored in the catalog

  • Note: Fields in the document that were not implementable OOTB in backstage were left out

Signed-off-by: John Collier <jcollier@redhat.com>
@johnmcollier johnmcollier requested a review from a team September 24, 2024 19:38
@johnmcollier johnmcollier reopened this Sep 24, 2024
Signed-off-by: John Collier <jcollier@redhat.com>
Copy link
Contributor

@Jdubrick Jdubrick left a comment

Choose a reason for hiding this comment

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

Just a couple small comments! Overall I like the bulk of the content and think this is good

johnmcollier and others added 5 commits September 24, 2024 16:34
Signed-off-by: John Collier <jcollier@redhat.com>
Signed-off-by: John Collier <jcollier@redhat.com>
Co-authored-by: Jordan Dubrick <jdubrick@redhat.com>
Signed-off-by: John Collier <jcollier@redhat.com>
@johnmcollier
Copy link
Contributor Author

@Jdubrick Review comments addressed if you don't mind re-reviewing when you have time, thanks!

Copy link
Contributor

@Jdubrick Jdubrick left a comment

Choose a reason for hiding this comment

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

lgtm

Co-authored-by: Jordan Dubrick <jdubrick@redhat.com>
@johnmcollier johnmcollier merged commit e4c739a into main Sep 25, 2024
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