Skip to content

refactor: remove obsolete notebook bootstrap #7

Description

@Jammy2211

Overview

Remove the obsolete %matplotlib inline plus pyprojroot notebook bootstrap from maintained AutoCTI workspace material. The public workspace's authored scripts move to the current autonerves.setup_notebook() convention and its paired notebooks are regenerated; non-legacy workspace-test scripts lose the inert bootstrap, while archived Euclid heritage remains verbatim.

This is a behavior-preserving workspace cleanup with no library or public API changes. It follows the completed standalone-comment cleanup in AutoGalaxy and AutoLens.

Plan

  • Replace the old five-line bootstrap in all 79 authored AutoCTI workspace scripts with the current one-line notebook setup convention.
  • Regenerate all 79 paired notebooks through PyAutoHands conversion tooling; never hand-edit notebook JSON.
  • Remove the old bootstrap from the 26 non-legacy autocti_workspace_test/imaging_ci/ scripts.
  • Preserve all 13 occurrences under autocti_workspace_test/legacy/ verbatim.
  • Validate notebook JSON, exact file scope, literal absence in non-legacy material, and the maintained CTI smoke surface.
Detailed implementation plan

Work Classification

Workspace

Affected Repositories

  • autocti_workspace (primary)
  • autocti_workspace_test

Branch Survey

Repository Current Branch Dirty?
./autocti_workspace main clean
./autocti_workspace_test main clean

No active worktree claims overlap either repository.

Suggested branch: feature/autocti-notebook-bootstrap

Worktree root: ~/Code/PyAutoLabs-wt/autocti-notebook-bootstrap/

Implementation Steps

  1. In the 79 matching autocti_workspace/scripts/**/*.py files, replace exactly:

    • # %matplotlib inline
    • the commented pyprojroot import/path setup
    • commented %cd and working-directory print

    with # from autonerves import setup_notebook; setup_notebook().

  2. Regenerate the corresponding 79 notebooks/**/*.ipynb files from the scripts with PyAutoHands. If the full generate.py autocti entry remains gated by the separately tracked Colab/release registry, use PyAutoHands' build_util.py_to_notebook conversion path for the exact pairs; do not modify release wiring or notebook JSON by hand.

  3. In the 26 matching non-legacy autocti_workspace_test/imaging_ci/**/*.py files, remove the obsolete five-line block. These integration/profiling scripts run from the repository root and do not need a notebook setup replacement.

  4. Do not edit autocti_workspace_test/legacy/; its 13 occurrences are preserved historical material.

  5. Validate all regenerated notebooks with a JSON parser and git diff --check; assert zero %matplotlib inline and zero pyprojroot references in non-legacy tracked .py/.ipynb files.

  6. Run the maintained autocti_workspace_test smoke runner and representative AutoCTI workspace script/notebook checks under the configured test-mode/cache environment.

Key Files

  • autocti_workspace/scripts/**/*.py — 79 authored workspace scripts.
  • autocti_workspace/notebooks/**/*.ipynb — 79 generated notebook counterparts.
  • autocti_workspace_test/imaging_ci/**/*.py — 26 non-legacy scripts.
  • autocti_workspace_test/legacy/** — explicitly excluded, unchanged archive.

Original Prompt

Click to expand starting prompt

ok, lets get rid of the five standalones and then tackle all the CTI stuff

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions