Skip to content

ref: remove outdated make_output_plots.py and related docs#263

Merged
ParticularlyPythonicBS merged 1 commit intounstablefrom
ref/removing_make_output_plots
Mar 5, 2026
Merged

ref: remove outdated make_output_plots.py and related docs#263
ParticularlyPythonicBS merged 1 commit intounstablefrom
ref/removing_make_output_plots

Conversation

@ParticularlyPythonicBS
Copy link
Member

@ParticularlyPythonicBS ParticularlyPythonicBS commented Mar 5, 2026

The script is outdated and unsupported in Temoa v4.0.

Summary by CodeRabbit

  • Documentation

    • Removed guides, examples, and the "Output Graphs" section that explained how to generate output visualizations.
  • Chores

    • Removed the output-graph generation utility and its command-line workflow; capacity, flow, and emissions plotting instructions and tooling are no longer available.

@ParticularlyPythonicBS ParticularlyPythonicBS added the Maintenance Code quality fixes and deprecation management label Mar 5, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: d2c76a95-47b3-494e-b7c8-befc9962e1a0

📥 Commits

Reviewing files that changed from the base of the PR and between 06935c7 and fb98145.

📒 Files selected for processing (3)
  • docs/source/Documentation.rst
  • docs/source/visualization.rst
  • temoa/data_processing/make_output_plots.py
💤 Files with no reviewable changes (3)
  • docs/source/Documentation.rst
  • docs/source/visualization.rst
  • temoa/data_processing/make_output_plots.py

Walkthrough

The output plotting feature was removed: the plotting module (make_output_plots.py) and associated documentation sections describing CLI usage and matplotlib-based output graphs were deleted. No new code or public APIs were introduced.

Changes

Cohort / File(s) Summary
Documentation
docs/source/Documentation.rst, docs/source/visualization.rst
Removed "Output Graphs" subsections, command-line examples, usage instructions, figures, and descriptive text for the make_output_plots.py plotting workflow.
Output Plotting Module
temoa/data_processing/make_output_plots.py
Removed entire plotting utility: OutputPlotGenerator class (constructor, data extraction, processing, plotting helpers, and generate_plot_for_* methods), CLI entrypoint generate_plot, and public type aliases (DbRow, PlotData, Color, ColorMapFunc).

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: removing an outdated script and its documentation, which matches the changeset exactly.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ref/removing_make_output_plots

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ParticularlyPythonicBS ParticularlyPythonicBS force-pushed the ref/removing_make_output_plots branch from 06935c7 to fb98145 Compare March 5, 2026 19:54
@ParticularlyPythonicBS ParticularlyPythonicBS merged commit e918b1b into unstable Mar 5, 2026
12 checks passed
@ParticularlyPythonicBS ParticularlyPythonicBS deleted the ref/removing_make_output_plots branch March 5, 2026 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintenance Code quality fixes and deprecation management

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant