Skip to content

feat(proposed-layers): move GPU rendering counterparts - #10985

Merged
ibgreen merged 3 commits into
masterfrom
codex/webgpu-layers
Oct 31, 2026
Merged

ibgreen merged 3 commits into
masterfrom
codex/webgpu-layers

Conversation

@ibgreen

@ibgreen ibgreen commented Oct 30, 2026 •

Copy link
Copy Markdown
Collaborator

Background

GPU-backed counterparts currently share a package with unrelated experimental effects.

Change List

  • Move path, polygon, solid polygon, and text GPU layers
  • Add direct @deck.gl/proposed-layers subpath exports
  • Update Arrow layer imports and private build configuration
  • Unit tests: verify layer subpath exports
  • Documentation: describe imports, ownership, and rendering paths

Verified: private build, lint, WebGPU Arrow tests, and node tests

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-30T21:25:00.610932Z
We couldn’t complete this review. Comment @codex review to try again.
7e2077e PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@greptile-apps

greptile-apps Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge; no actionable issues remain from this review.

Summary

Moves four GPU-backed layers into @deck.gl/proposed-layers, with separate import paths for each layer.

  • GPU layer counterparts now have dedicated named import paths.

No new actionable issues were found. No builds or tests were run during this review.

Reviews (3) · Last reviewed commit: "Rename webgpu layers to proposed layers" · Reviewed by Greptile

Comment thread vitest.config.ts Resolved

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 7e2077e. Configure here.

Comment thread vitest.config.ts Resolved
@coveralls

coveralls commented Oct 30, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 84.401%. remained the same — codex/webgpu-layers into master

@ibgreen ibgreen changed the title feat(webgpu-layers): move GPU rendering counterparts feat(proposed-layers): move GPU rendering counterparts Oct 31, 2026
@ibgreen
ibgreen merged commit 2abbc7a into master Oct 31, 2026
9 checks passed
@ibgreen
ibgreen deleted the codex/webgpu-layers branch October 31, 2026 11:05
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