Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: tdhopper/pythonplot.com
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: modernize
Choose a base ref
...
head repository: tdhopper/pythonplot.com
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 7 commits
  • 7 files changed
  • 2 contributors

Commits on Nov 4, 2025

  1. Fix default make target for Netlify deployment 

    Change default 'all' target from 'render s3_upload' to just 'render'
since deployment is now handled by Netlify via GitHub Actions.

- 'make' or 'make all' now just builds the site
- 'make deploy' added for manual S3 upload if needed
- S3 upload is legacy, kept for backward compatibility

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
    @tdhopper @claude
    tdhopper and claude committed Nov 4, 2025
    Configuration menu
    Copy the full SHA
    35f7c98 View commit details
    Browse the repository at this point in the history

  2. Add automatic dependency setup to Makefile 

    Use Make's dependency system to automatically install dependencies
before building. This follows standard Make patterns:

- Create .setup_done sentinel file after successful setup
- Make render and run_nb depend on .setup_done
- Running 'make' will now automatically run setup if needed
- Setup only runs once (or after 'make clean')
- Add .setup_done to .gitignore

Now users can just run 'make' and everything works.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
    @tdhopper @claude
    tdhopper and claude committed Nov 4, 2025
    Configuration menu
    Copy the full SHA
    27b8428 View commit details
    Browse the repository at this point in the history

  3. Switch to uv sync for proper dependency management 

    Replace `uv pip install -r requirements.txt` with `uv sync` to use
modern uv workflow with pyproject.toml:

- Use `uv sync` to create venv and install dependencies
- Use `uv run` prefix for all Python/Jupyter commands
- Remove build-system from pyproject.toml (not a package)
- Fix deprecation warning (tool.uv.dev-dependencies → dependency-groups)
- Add .venv/ and uv.lock to .gitignore

Benefits:
- Avoids "externally managed" Python installation errors
- Uses proper virtual environment isolation
- Follows modern uv best practices
- Automatic venv creation and management

Tested: `make` now successfully creates venv and installs all Python
dependencies. Only requires system R to be installed separately.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
    @tdhopper @claude
    tdhopper and claude committed Nov 4, 2025
    Configuration menu
    Copy the full SHA
    9d523fd View commit details
    Browse the repository at this point in the history

  4. Fix seaborn API compatibility for 0.13+ 

    Replace deprecated 'size' parameter with 'height' in seaborn calls:
- sns.lmplot(..., size=12) → height=12
- FacetGrid(..., size=10) → height=10

The 'size' parameter was renamed to 'height' in seaborn 0.11+
and removed entirely in 0.13+.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
    @tdhopper @claude
    tdhopper and claude committed Nov 4, 2025
    Configuration menu
    Copy the full SHA
    a55cc38 View commit details
    Browse the repository at this point in the history

  5. Fix all seaborn and Altair compatibility issues 

    Seaborn fixes (0.13+ compatibility):
- Fix remaining FacetGrid size -> height parameters (3 instances)
- Update all .pipe(sns.FacetGrid, size=X) calls

Altair fixes (5.0+ compatibility):
- Add vl-convert-python dependency for PNG export
- Update Altair renderer configuration to use 'png' mode with vl-convert
- Remove deprecated selenium-based PNG rendering

These changes ensure all plotting libraries work with their 2025 versions.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
    @tdhopper @claude
    tdhopper and claude committed Nov 4, 2025
    Configuration menu
    Copy the full SHA
    7f31ca9 View commit details
    Browse the repository at this point in the history

  6. Add Chrome installation for Kaleido v1 support 

    Kaleido v1+ requires Chrome for image rendering. Add automatic
Chrome installation to setup process:

- Update Makefile to run kaleido_get_chrome after uv sync
- Update GitHub Actions workflow to install Chrome
- Update pyproject.toml to note Kaleido v1+ requirement
- Update requirements.txt

This ensures Plotly static image export works with Kaleido v1.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
    @tdhopper @claude
    tdhopper and claude committed Nov 4, 2025
    Configuration menu
    Copy the full SHA
    e7639cc View commit details
    Browse the repository at this point in the history

  7. Add modernization summary document

    @tdhopper
    tdhopper committed Nov 4, 2025
    Configuration menu
    Copy the full SHA
    900bf71 View commit details
    Browse the repository at this point in the history
Loading