Documentation for model interdependencies#275
Conversation
|
@gtagle & @panchicore Nik suggested that for feature work I start tagging in the feature owner to confirm if the documentation is accurate if he can't get to it fast enough! Let me know if you two can help/have any questions. :) |
There was a problem hiding this comment.
Thank you for addressing these docs that were missed during our product development cycle! Yes, let's start asking the originators for these changes to review the docs for accuracy. My comments are more of an add-on from the docs perspective.
One item to discuss: I’m a bit confused by what should go under “Working with the model inventory” and “Managing the model inventory” in these PR changes. There’s overlap between topics in these sections — I’d maybe get rid of one of these?
For example, you could have “Register models” as a top-level topic, followed by “Managing the model inventory” with the all the other topics under it, including those from “Working with the model inventory”, and then get rid of that “Working with …” heading.
site/guide/model-inventory/configure-model-interdependencies.qmd
Outdated
Show resolved
Hide resolved
site/guide/model-inventory/configure-model-interdependencies.qmd
Outdated
Show resolved
Hide resolved
To me, the existing working with the model inventory section is more of a "Get started" sort of idea, and the "Managing" is more advanced set up. 🤔 I personally like the separation of these two ideas, but I'm in agreement that maybe the seconds can be renamed? I know that the "Working with..." topic is linked A LOT elsewhere, including the |
Co-authored-by: Nik Richers <[email protected]>
Co-authored-by: Nik Richers <[email protected]>
PR SummaryThis pull request introduces several enhancements and bug fixes to the model inventory documentation within the ValidMind platform. The key changes include:
Test Suggestions
|
nrichers
left a comment
There was a problem hiding this comment.
To me, the existing working with the model inventory section is more of a "Get started" sort of idea, and the "Managing" is more advanced set up. 🤔
It looks like the short descriptions now explain this better after your last commit. LGTM, though we might simply want to name the first topic "Getting started with ..." at some point. ![]()
PR SummaryThis pull request introduces several enhancements to the model inventory documentation within the project. The key changes include:
Test Suggestions
|

Internal Notes for Reviewers
Managing the model inventory
model-inventory/configure-model-interdependencies.qmdmodel-inventory/manage-inventory-custom-fields.qmdunder here as wellConfigure model interdependencies
Net-new page, explaining what model interdependencies are within ValidMind, and how to view and work with them.