Skip to content

Conversation

@SamTov
Copy link
Member

@SamTov SamTov commented Oct 10, 2025

Summary by CodeRabbit

  • New Features

    • Official support for Python 3.12 (tests now run on 3.12).
  • Chores

    • Updated dependencies: added mitsuba; relaxed minimum versions for open3d (≥0.17.0) and numpy (≥1.26.4).
    • Modernized CI workflows and linters to latest stable actions with improved permissions and checks.

@coderabbitai
Copy link

coderabbitai bot commented Oct 10, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

CI workflows updated: actions upgraded to v4, pull_request trigger and read permissions added, Python set to 3.12 for lint and tests, and explicit isort/flake8 steps added. Dependencies relaxed for open3d/numpy and mitsuba added in requirements.txt.

Changes

Cohort / File(s) Summary of Changes
CI workflows
.github/workflows/lint.yaml, .github/workflows/pytest.yaml
Upgrade actions/checkout and setup-python to v4; add pull_request trigger and permissions: contents: read; set Python 3.12 for isort/flake8 and test matrix; add explicit isort --check-only and flake8 command steps.
Dependencies
requirements.txt
Relax open3d (>=0.17.0) and numpy (>=1.26.4) constraints; add mitsuba; retain other pins.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

Thump-thump go my reviewer feet,
New lint and tests are crisp and neat.
Python hopped to 3.12’s glade,
Requirements trimmed, new shade of jade.
With mitsuba in the meadow bright,
I nibble green checks by moonlight.
(_/)> ✅

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch play-with-workflos

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1f2157a and 88edeab.

📒 Files selected for processing (3)
  • .github/workflows/lint.yaml (2 hunks)
  • .github/workflows/pytest.yaml (1 hunks)
  • requirements.txt (1 hunks)

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.

@SamTov SamTov merged commit ab58aea into main Oct 10, 2025
8 of 9 checks passed
@SamTov SamTov deleted the play-with-workflos branch October 10, 2025 12:47
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.

2 participants