Skip to content

Update links in doc, fix broken links and move to https when possible#2254

Merged
araffin merged 4 commits into
masterfrom
doc/update-links
May 11, 2026
Merged

Update links in doc, fix broken links and move to https when possible#2254
araffin merged 4 commits into
masterfrom
doc/update-links

Conversation

@araffin
Copy link
Copy Markdown
Member

@araffin araffin commented May 11, 2026

Description

Motivation and Context

  • I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (update in the documentation)

Checklist

  • I've read the CONTRIBUTION guide (required)
  • I have updated the changelog accordingly (docs/misc/changelog.md) (required).
  • My change requires a change to the documentation.
  • I have updated the tests accordingly (required for a bug fix or a new feature).
  • I have updated the documentation accordingly.
  • I have opened an associated PR on the SB3-Contrib repository (if necessary)
  • I have opened an associated PR on the RL-Zoo3 repository (if necessary)
  • I have reformatted the code using make format (required)
  • I have checked the codestyle using make check-codestyle and make lint (required)
  • I have ensured make pytest and make type both pass. (required)
  • I have checked that the documentation builds using make doc (required)

Note: You can run most of the checks using make commit-checks.

Note: we are using a maximum length of 127 characters per line

Copy link
Copy Markdown

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 updates various documentation and reference links across Stable-Baselines3 to fix broken URLs and prefer HTTPS where applicable, including some link target replacements.

Changes:

  • Updated many HTTP links to HTTPS (papers, docs, project references, and metadata files like CITATION/README).
  • Replaced several outdated external links (e.g., project repos, vendor docs) with newer canonical locations.
  • Minor doc guidance tweaks (e.g., Windows installer recommendation, pip extras quoting) and link hygiene in Sphinx/doc assets.

Reviewed changes

Copilot reviewed 24 out of 26 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
stable_baselines3/ddpg/ddpg.py Updated paper link in DDPG docstring to HTTPS.
stable_baselines3/common/sb2_compat/rmsprop_tf_like.py Updated linked lecture PDF to HTTPS.
stable_baselines3/common/noise.py Updated OU noise reference link to HTTPS.
README.md Updated JMLR URL to HTTPS and refreshed some maintainer/organization links.
NOTICE Updated OpenAI URL to HTTPS.
docs/modules/ppo.md Updated gSDE reference link (PR → issue) for inference note.
docs/modules/dqn.md Updated FQI link to a newer PDF location over HTTPS.
docs/modules/ddpg.md Updated DPG paper link to HTTPS.
docs/modules/a2c.md Updated OpenAI blog link and gSDE reference link (PR → issue).
docs/misc/projects.md Updated several external project repository links; removed some dead/outdated links.
docs/misc/changelog.md Added changelog entry for link fixes; updated a few reference URLs.
docs/make.bat Updated Sphinx URL to HTTPS.
docs/index.rst Updated JMLR URL to HTTPS.
docs/guide/tensorboard.md Updated PyTorch TensorBoard doc links and pinned an RL Zoo permalink.
docs/guide/sb3_contrib.md Updated RecurrentPPO implementation-details link.
docs/guide/rl.md Updated Berkeley course link to HTTPS.
docs/guide/install.md Updated Windows installer recommendation, pip command quoting, and proxy example.
docs/guide/imitation.md Updated imitation “first steps” documentation link.
docs/guide/export.md Updated Coral documentation link.
docs/guide/examples.md Updated external project links and permalinks (e.g., Isaac Lab wrapper).
docs/guide/checking_nan.md Updated an imputation article link.
docs/conf.py Updated Sphinx and commented intersphinx URLs to HTTPS.
docs/_static/img/colab.svg Changed SVG namespace URI scheme (needs correction).
docs/_static/img/colab-badge.svg Changed SVG + xlink namespace URI schemes (needs correction).
CONTRIBUTING.md Updated Stack Overflow link to HTTPS.
CITATION.bib Updated JMLR URL to HTTPS.

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

Comment thread docs/modules/a2c.md

:::{note}
Using gSDE (Generalized State-Dependent Exploration) during inference (see [PR #1767](https://github.com/DLR-RM/stable-baselines3/pull/1767)):
Using gSDE (Generalized State-Dependent Exploration) during inference (see [issue #1767](https://github.com/DLR-RM/stable-baselines3/issues/1767)):
Comment thread docs/modules/ppo.md

:::{note}
Using gSDE (Generalized State-Dependent Exploration) during inference (see [PR #1767](https://github.com/DLR-RM/stable-baselines3/pull/1767)):
Using gSDE (Generalized State-Dependent Exploration) during inference (see [issue #1767](https://github.com/DLR-RM/stable-baselines3/issues/1767)):
@araffin araffin merged commit 5f6003b into master May 11, 2026
4 checks passed
@araffin araffin deleted the doc/update-links branch May 11, 2026 15:32
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.

2 participants