Skip to content

Conversation

@0hmX
Copy link
Contributor

@0hmX 0hmX commented Nov 18, 2025

0hmX added 8 commits November 17, 2025 23:51
- Always show 3D visualization by default
- Remove Hide 3D/Show 3D toggle button
- Remove Rebuild 3D button
- Remove Root and Output checkboxes (always enabled)
- Remove Dataset Information and Usage Instructions sections
- Keep Obstacles toggle and advanced controls (opacity, wireframe, etc.)
- Clean up component props and state management
Adds a toggle for orthographic camera view in the CapacityNode3dDebugger component.
- Introduces an 'isOrthographic' state and a corresponding checkbox.
- Modifies ThreeBoardView to dynamically switch between PerspectiveCamera and OrthographicCamera.
- Adjusts camera positioning and resize handling for the orthographic view, providing a top-down initial perspective.
@vercel
Copy link

vercel bot commented Nov 18, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
trace-capacity-visualizer Ready Ready Preview Comment Nov 18, 2025 7:55am

@@ -0,0 +1,126 @@
# CapacityNode3dDebugger Implementation Summary
Copy link
Contributor

Choose a reason for hiding this comment

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

remove

@@ -0,0 +1,951 @@
import {
Copy link
Contributor

Choose a reason for hiding this comment

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

this is ok for now but this file needs to be broken up (next PR, i would definitely use Repomix + ChatGPT Pro to do the breakup because agents tend to do refactors very poorly, after Pro gives you instructions, just copy and paste those instructions to a coding agent like claude)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

bun.lock Outdated
@@ -0,0 +1,583 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

this shouldn't be commited

Copy link
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

make sure to read over your PRs

@seveibar
Copy link
Contributor

the format check might be failing because it requires the following in the package.json scripts:

"format": "biome format --write .",
"format:check": "biome format ."

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

return m
}, [layerNames])

const layerCount = layerNames.length || srj?.layerCount || 1

Choose a reason for hiding this comment

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

P1 Badge Honor SRJ layerCount when sizing the scene

layerCount is computed as layerNames.length || srj?.layerCount || 1, so as soon as there is at least one node, any SRJ-specified layer count is ignored. If the SRJ describes more layers than are present in the node layer names (e.g., nodes labeled only “top” with srj.layerCount of 4 or obstacles with zLayers > 0), layerCount collapses to 1, causing obstacles on higher layers to be skipped by the z >= layerCount check and the root box/camera fit to use the wrong stack height. Prefer using the SRJ layerCount (or the max of both) so multi-layer boards render correctly.

Useful? React with 👍 / 👎.

@seveibar seveibar merged commit 5d22662 into main Nov 18, 2025
4 checks passed
@tscircuitbot
Copy link


Thank you for your contribution! 🎉

PR Rating: ⭐⭐⭐
Impact: Major

Track your contributions and see the leaderboard at: tscircuit Contribution Tracker


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.

4 participants