Skip to content

[DOC] Clear stale ImportError output from data_unequal notebook - #3748

Open
AnkitKumarIISERB wants to merge 2 commits into
aeon-toolkit:mainfrom
AnkitKumarIISERB:doc/clear-stale-output-data-unequal-notebook
Open

[DOC] Clear stale ImportError output from data_unequal notebook#3748
AnkitKumarIISERB wants to merge 2 commits into
aeon-toolkit:mainfrom
AnkitKumarIISERB:doc/clear-stale-output-data-unequal-notebook

Conversation

@AnkitKumarIISERB

Copy link
Copy Markdown
Contributor

Closes #3385

The rendered documentation page was showing a stale ImportError traceback
from a previous notebook execution with an incompatible sklearn version.
The underlying code in _continuous_interval_tree.py has since been updated
to use validate_data from a compatible sklearn version.

This PR clears the cached cell output so the docs page no longer renders
the error traceback.

Copilot AI lite review requested due to automatic review settings August 16, 2026 12:30
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@aeon-actions-bot aeon-actions-bot Bot added documentation Improvements or additions to documentation examples Example notebook related labels Aug 16, 2026
@aeon-actions-bot

Copy link
Copy Markdown
Contributor

Thank you for contributing to aeon

I have added the following labels to this PR based on the title: [ documentation ].
I have added the following labels to this PR based on the changes made: [ examples ]. Feel free to change these if they do not properly represent the PR.

The Checks tab will show the status of our automated tests. You can click on individual test runs in the tab or "Details" in the panel below to see more information if there is a failure.

If our pre-commit code quality check fails, please run pre-commit locally and push the fixes to your PR branch.

Don't hesitate to ask questions on the aeon Discord channel if you have any.

PR CI actions

These checkboxes will add labels to enable or disable CI functionality for this PR. This may not take effect immediately, and a new commit may be required to run the new configuration.

  • Run pre-commit checks for all files
  • Run mypy typecheck tests
  • Run all pytest tests and configurations
  • Run all notebook example tests
  • Run numba-disabled codecov tests
  • Disable numba cache loading
  • Regenerate expected results for testing
  • Push an empty commit to re-run CI checks

Copilot AI 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.

Pull request overview

This PR updates the data_unequal example notebook to remove stale, cached execution output (an ImportError traceback) that was rendering on the documentation site.

Changes:

  • Cleared cached cell outputs and reset execution_count values in examples/datasets/data_unequal.ipynb.
  • Reordered/normalized some per-cell metadata blocks.

This pull request includes code written with the assistance of AI.
The code has not yet been reviewed by a human.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread examples/datasets/data_unequal.ipynb Outdated
Comment on lines +28 to +32
"outputs": [],
"source": [
"from aeon.utils.discovery import all_estimators\n",
"\n",
"all_estimators(type_filter=\"classifier\", tag_filter={\"capability:unequal_length\": True})"
Comment on lines 22 to 26
"metadata": {
"collapsed": false,
"ExecuteTime": {
"end_time": "2026-02-23T14:13:34.343799Z",
"start_time": "2026-02-23T14:13:19.438797Z"
}
@TonyBagnall

Copy link
Copy Markdown
Contributor

Thanks for looking at this. I don't think simply clearing the cached output fixes the issue though. Our docs use nbsphinx_execute = "never", so notebooks are not rerun during the documentation build

Could you rerun this notebook against the current aeon environment and commit the refreshed outputs instead? In particular, the all_estimators(...) cell should display the expected estimator list rather than have outputs: [].

It also looks like this change has cleared the outputs from all the other cells in the notebook, which we should retain.

@TonyBagnall TonyBagnall 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.

see comment

@AnkitKumarIISERB

Copy link
Copy Markdown
Contributor Author

Done! set up a Python 3.12 conda environment (numba crashes the kernel
on Python 3.13), reran the notebook, and committed the fresh outputs.
The all_estimators cell now shows the expected estimator list and all
5 code cells have fresh outputs. No ImportError or traceback remains.

Note: the failing CI check is examples/visualisation/plotting_estimators.ipynb
which is unrelated to this PR — data_unequal.ipynb passes in 7s.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation examples Example notebook related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOC] Example cell fails with ImportError and renders traceback on page (all_estimators import)

3 participants