Remove stale unreferenced scripts and notebooks#745
Open
cailmdaley wants to merge 2 commits into
Open
Conversation
Drop six scripts and three 2021-22 notebooks with zero references repo-wide (docs, CI, scripts, src, example, auxdir all checked): - scripts/sh: canfar_selection.sh, curl_canfar_monitor_local.sh, missing_unique.sh, remove_duplicates.sh (remove_duplicates_tiles.sh is kept) - scripts/python: canfar_run_analyse.py, vignets2image.py - scripts/jupyter: CFIS_Footprint.ipynb, plot_spectro_areas.ipynb, shapepipe_tutorial_exploration_notebook.ipynb (wcs.ipynb and summary_run.ipynb are kept) Approved by Martin Kilbinger & Cail in review session 2026-06-11. Smoke tests are glob-based (tests/unit/test_shell_syntax.py, test_imports.py), so no test lists needed pruning; shell-syntax suite passes on the remaining scripts. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The documented curl_canfar_monitor.sh never existed under that name (the script was curl_canfar_monitor_local.sh, removed in this PR); the python canfar_monitor entry point supersedes it. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes nine files approved for deletion in the 2026-06-11 review session (Martin + Cail), each verified to have zero references repo-wide (bare filename and extension-less stem grepped across docs/, .github/, scripts/, src/, example/, auxdir/) before removal:
Scripts (superseded or orphaned, last touched 2024):
scripts/sh/canfar_selection.sh,scripts/sh/curl_canfar_monitor_local.sh— superseded by thecanfar_submit_job/canfar_monitorentry pointsscripts/sh/missing_unique.sh,scripts/sh/remove_duplicates.sh— no callers (remove_duplicates_tiles.shuntouched)scripts/python/canfar_run_analyse.py,scripts/python/vignets2image.py— no callersNotebooks (2021–22 exploration, unreferenced):
scripts/jupyter/CFIS_Footprint.ipynb,plot_spectro_areas.ipynb,shapepipe_tutorial_exploration_notebook.ipynb(wcs.ipynbandsummary_run.ipynbare load-bearing and untouched)Docs:
docs/source/canfar.md's monitoring section documentedcurl_canfar_monitor.sh, a name that never existed in git (the file wascurl_canfar_monitor_local.sh); it now points at thecanfar_monitorentry point that supersedes it. Independent review specifically hunted for references something still needed — this doc drift was the one find.Everything stays in git history;
example/cfis/defunct/remains the quarantine for configs.— Claude (Fable), on behalf of Cail
🤖 Generated with Claude Code