Skip to content

Compress the logo and stop tracking bytecode - #46

Open
rogeriojorge wants to merge 2 commits into
mainfrom
agent/compress-logo
Open

rogeriojorge wants to merge 2 commits into
mainfrom
agent/compress-logo

Conversation

@rogeriojorge

@rogeriojorge rogeriojorge commented Sep 14, 2026 •

Copy link
Copy Markdown
Member

Compress the repository's only image and stop tracking bytecode. No code changes.

Change (6 binary files)

  • docs/SPECTRAX_logo.png: 4.67 MB -> 193 KB. The original was a 1576x1476 RGBA screenshot with metadata; now 800x749, 256-colour palette, no metadata. README shows it at 30% width.
  • Remove five tracked __pycache__/*.pyc files (already covered by .gitignore).

Evidence (tested SHA dac2ff6, rebased on main ab87385; local)

  • Visual: both images rendered side by side at 480 px and inspected; no visible difference except faint palette banding in the dark outer halo. Old image downscaled to 800 px vs new: mean |diff| 3.1/255, 99th percentile 15/255, max 49/255.
  • Docs: pip install -r docs/requirements.txt && sphinx-build -q -b html docs <out> succeeds; _images/SPECTRAX_logo.png is the 192559-byte file. The one warning (short title underline in getting_started.rst) is pre-existing.
  • python -m pytest -n 2 -q tests: 8 passed.

No dependencies on other PRs. Note: history still contains the 4.7 MB blob; only new clones' checkouts shrink.

@codecov

codecov Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The logo was a 4.7 MB 1576x1476 screenshot with embedded metadata; the
README shows it at 30% width. It is now 800 px wide with a 256-colour
palette and no metadata (193 KB). Tracked __pycache__ files are removed;
__pycache__/ is already ignored.
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.

1 participant