Repository navigation
docs: restructure model developer getting started around existing and new models (CLIM-1478) - #830
Merged
Merged
Conversation
Contributor
Author
|
Overview of every problem found in the external modeler test, plus the ones found while fixing them, and how each is addressed. Status as of 5 November.
|
ivargr
marked this pull request as ready for review
November 5, 2026 08:22
|
No significant findings. This PR only changes docs. The |
knutdrand
approved these changes
Nov 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Restructures the model developer docs around whether you already have a model, following the external modeler test in CLIM-1389 and CLIM-1478. Getting Started is now one overview page that sends readers to: a new short "Set up your machine" page, a new "Bring your existing model" guide, and the MLproject tutorial, renamed "Build a new model from a template". Both guides follow three stages: run the model outside Chap, make it Chap-compatible with an
MLprojectfile andchap eval --run-config.as-chapkit, then package it as a Docker image with thecompose.ymland GitHub workflows now in themlproject_template_randmlproject_template_pythonrepos.Also: the existing-model guide recommends
--run-config.ignore-environmentandrenv_env/uv_envinstead ofdocker_env, says that train and predict scripts are a recommendation and the chapkit REST API is the only requirement, has a tip on using AI coding agents, and both guides list what the flow does not need (no chapkit install, no Chap server, nochapkit mlproject migrate). The comparison table moved from the tutorial tochapkit.md. "Development tools" moved from the contributor docs to the model developer docs and now points to the webinar guides. The "For experienced modellers" and "For newcomers" pages are removed. The overview says that a Chap-compatible model can be run both withchap evaland from the Modeling App, and "Running Your Own Model" links to the two guides.Review needed
Review needed: quick-merge
Test plan
make docs(strict) passes.chap eval --run-config.as-chapkit --run-config.ignore-environmentfrom an activated venv, thendocker compose up --buildandchap evalagainst the running service, for both templates. The R scripts outside Docker were not run, since this machine has no R.