Skip to content

[workspace] Patch nanobind to allow list inputs for Eigens#24766

Open
jwnimmer-tri wants to merge 1 commit into
RobotLocomotion:masterfrom
jwnimmer-tri:nanobind-dense-caster-lists
Open

[workspace] Patch nanobind to allow list inputs for Eigens#24766
jwnimmer-tri wants to merge 1 commit into
RobotLocomotion:masterfrom
jwnimmer-tri:nanobind-dense-caster-lists

Conversation

@jwnimmer-tri

@jwnimmer-tri jwnimmer-tri commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Towards #21572.

The upstream feature request is wjakob/nanobind#327.


This change is Reviewable

@jwnimmer-tri jwnimmer-tri added priority: medium release notes: none This pull request should not be mentioned in the release notes labels Jul 21, 2026

@jwnimmer-tri jwnimmer-tri left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+a:@rpoyner-tri for feature review, please.

@jwnimmer-tri reviewed 5 files and all commit messages, and made 1 comment.
Reviewable status: LGTM missing from assignee rpoyner-tri(platform), needs at least two assigned reviewers (waiting on rpoyner-tri).

@jwnimmer-tri
jwnimmer-tri force-pushed the nanobind-dense-caster-lists branch from 359fff1 to 11fc2e0 Compare July 21, 2026 22:36

@rpoyner-tri rpoyner-tri left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@rpoyner-tri reviewed 5 files and all commit messages, and made 1 comment.
Reviewable status: needs at least two assigned reviewers (waiting on jwnimmer-tri).

@jwnimmer-tri

Copy link
Copy Markdown
Collaborator Author

+a:@SeanCurtis-TRI for platform review, please (load balancing).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: medium release notes: none This pull request should not be mentioned in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants