Skip to content

Fix broken links and setup errors in time series tutorials#121

Merged
petrasovaa merged 4 commits into
OSGeo:mainfrom
Valyrian-Code:fix/time-series-links-and-setup
Jun 1, 2026
Merged

Fix broken links and setup errors in time series tutorials#121
petrasovaa merged 4 commits into
OSGeo:mainfrom
Valyrian-Code:fix/time-series-links-and-setup

Conversation

@Valyrian-Code

Copy link
Copy Markdown
Contributor

Fixes 5 issues across two time series tutorials:

time_series_management_and_visualization.qmd

  • L270: t.register linked to grass83 manual (301 redirect) → updated to grass-stable
  • L297: g.gui.tplot linked to grass83 manual (301 redirect) → updated to grass-stable
  • L427: t.rast.univar linked to t.vect.univar.html (wrong tool) → fixed to t.rast.univar.html

time_series_aggregations.qmd

  • L48: undefined variable grassbin in hidden setup cell → replaced with string "grass" (consistent with all other tutorials in the series)
  • L55: wrong project path eu_laea/italy_LST_daily → corrected to italy_eu_laea/italy_LST_daily

The aggregations bugs are in a #| echo: false cell — invisible in the rendered tutorial but break execution in the downloaded Jupyter notebook.

Copilot AI review requested due to automatic review settings May 31, 2026 15:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates documentation links to use the stable GRASS manual URLs, fixes an incorrect link target, and corrects references in the time series aggregations tutorial.

Changes:

  • Replace grass83 versioned manual URLs with grass-stable URLs.
  • Fix t.rast.univar link that incorrectly pointed to t.vect.univar.
  • Replace grassbin variable with hardcoded "grass" string and update project path to italy_eu_laea/italy_LST_daily.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
content/tutorials/time_series/time_series_management_and_visualization.qmd Updates manual links to stable URLs and fixes an incorrect link target.
content/tutorials/time_series/time_series_aggregations.qmd Uses "grass" directly in subprocess call and updates the project path.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread content/tutorials/time_series/time_series_aggregations.qmd
Comment thread content/tutorials/time_series/time_series_aggregations.qmd Outdated

@petrasovaa petrasovaa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you!

@petrasovaa petrasovaa merged commit fd29f32 into OSGeo:main Jun 1, 2026
1 check passed
@Valyrian-Code Valyrian-Code deleted the fix/time-series-links-and-setup branch June 2, 2026 10:30
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.

3 participants