Conversation
35cd69b to
8478562
Compare
Code ReviewFound 1 issue that needs to be addressed: Issue: Broken sidebar navigation linkFile: Problem: The sidebar file still references Impact: Without this change, the Quarto build will fail or produce a broken navigation entry in the sidebar. Current code (line 13): - developer/supported-models.qmdShould be: - developer/supported-model-frameworks.qmdReferences:
Note: All other internal references (in |
- Rename supported-models.qmd to supported-model-frameworks.qmd - Add alias for old URL to preserve links - Restructure content into clearer sections: - Framework support with wrapper classes and install commands - Test input requirements (predict vs predict_proba) - Custom model wrappers (FunctionModel, PipelineModel) - GenAI and LLM support - RAG evaluation with RAGAS - Python and dependency compatibility - Add Mermaid diagrams for class hierarchy and test input flow - Update references in faq-integrations.qmd and overview-model-documentation.qmd
8478562 to
2db869e
Compare
PR SummaryThis PR modifies documentation to update references from an old supported models page to a new, more comprehensive Supported Model Frameworks page. Specifically, the changes include:
These changes improve the clarity and relevance of the documentation, aligning the content with current terminology and the expanded scope of supported frameworks. Test Suggestions
|
Pull Request Description
What and why?
Renames "Supported Models" to "Supported Model Frameworks" and restructures content:
predict()vspredict_proba()is needed)FunctionModel,PipelineModel)Adds alias to preserve existing links to
supported-models.html.Addresses sc-5014.
How to test
/developer/supported-model-frameworks.html/developer/supported-models.htmlWhat needs special review?
Dependencies, breaking changes, and deployment notes
Release notes
Restructured the "Supported Models" documentation page as "Supported Model Frameworks" with clearer guidance on test requirements, framework wrappers, and GenAI/RAG support. Learn more
Checklist