Skip to content

Issue 20: Improving the example script#63

Merged
SamuelBrand1 merged 10 commits intomainfrom
20-computed-examples
Feb 20, 2026
Merged

Issue 20: Improving the example script#63
SamuelBrand1 merged 10 commits intomainfrom
20-computed-examples

Conversation

@SamuelBrand1
Copy link
Copy Markdown
Collaborator

This pull request introduces several improvements to the documentation, and forecasting code in the NowcastAutoGP.jl project. The most significant changes include a new automated vignette generation workflow using Literate.jl, updates to documentation images and navigation, improvements to parallelism and progress reporting in forecasting functions, and minor API enhancements.

Documentation workflow and navigation:

  • Added docs/make_vignettes.jl to automate the generation of markdown vignettes from Literate.jl scripts, and updated documentation navigation to reference the new vignette locations. [1] [2] [3] [4]
  • Updated example images in README.md to use new paths reflecting the generated vignette outputs.

Dependency management:

  • Added new documentation dependencies: AutoGP, Literate, Parameters, and ProgressMeter to docs/Project.toml.

Forecasting code improvements:

  • Improved parallelization and progress reporting in forecast_with_nowcasts by using Threads.@spawn for concurrent scenario processing and a progress bar for user feedback. [1] [2]
  • Simplified the forecast and _forecast functions by removing the verbose argument and related logging, and by streamlining the HMC forecast sampling logic. [1] [2]

API enhancements:

  • Added GPModel as a re-exported constant for user convenience in NowcastAutoGP.jl.
  • Added new assertions in forecast_with_nowcasts for better input validation.

Add a generated Getting Started vignette and associated images, replace the old tutorial files, and update documentation build scripts and links. Changes include: adding docs/src/vignettes/getting-started.md and several PNG figures; removing docs/src/vignettes/tutorial.md and renaming docs/vignettes/tutorial.jl -> docs/vignettes/getting-started.jl; update README image paths and docs/src/index.md to point to the new vignette; adjust .gitignore to include docs/src/vignettes/*.png; simplify docs/Project.toml package path; and revise docs/make.jl and docs/make_vignettes.jl to generate CommonMark vignette markdown via Literate (execute = true) and include the vignette in the site pages. Also include a minor change to src/forecasting.jl.
@SamuelBrand1 SamuelBrand1 linked an issue Feb 19, 2026 that may be closed by this pull request
@SamuelBrand1 SamuelBrand1 requested a review from Copilot February 19, 2026 16:38
This is because the over nowcasts parallelism relies on Julia 1.11+
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR modernizes the “getting started” documentation pipeline (moving the tutorial to a Literate.jl-generated vignette) and updates core forecasting utilities to improve scenario batching, including parallel execution and progress reporting.

Changes:

  • Replace the Quarto tutorial with a Literate.jl source (docs/vignettes/getting-started.jl) and commit the generated markdown under docs/src/vignettes/.
  • Update documentation navigation/build config to point to the new vignette location and add required docs dependencies.
  • Refactor forecasting to remove forecast(...; verbose=...), adjust HMC sampling logic, and parallelize forecast_with_nowcasts via Threads.@spawn + a progress bar.

Reviewed changes

Copilot reviewed 12 out of 21 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/forecasting.jl Refactors forecast / _forecast and parallelizes forecast_with_nowcasts.
src/NowcastAutoGP.jl Re-exports GPModel for convenience.
docs/vignettes/tutorial.qmd Removes the Quarto-based tutorial source.
docs/vignettes/getting-started.jl Adds the Literate.jl vignette source.
docs/src/vignettes/tutorial.md Removes the previously committed generated tutorial markdown.
docs/src/vignettes/getting-started.md Adds the generated markdown output from Literate.jl.
docs/src/index.md Updates “Getting started” link to the new vignette path.
docs/make_vignettes.jl Adds a script to generate vignette markdown/images from Literate sources.
docs/make.jl Updates Documenter pages config to include the new vignette.
docs/Project.toml Adds documentation dependencies (e.g., Literate, Parameters, ProgressMeter).
README.md Updates image paths to the new generated vignette outputs.
Project.toml Bumps Julia compat to 1.11.
.gitignore Tracks vignette-generated PNGs under docs/src/vignettes/.

Comment thread src/forecasting.jl
Comment thread src/forecasting.jl Outdated
Comment thread src/forecasting.jl
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.08%. Comparing base (6da107c) to head (cc0e4a0).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
+ Coverage   94.64%   95.08%   +0.43%     
==========================================
  Files           5        5              
  Lines         112      122      +10     
==========================================
+ Hits          106      116      +10     
  Misses          6        6              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SamuelBrand1 SamuelBrand1 merged commit ad2744d into main Feb 20, 2026
10 checks passed
@SamuelBrand1 SamuelBrand1 deleted the 20-computed-examples branch February 20, 2026 14:15
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.

Improved example for docs

3 participants