Skip to content

Increase GLB export board texture resolution from 512 to 1024#3049

Open
mohan-bee wants to merge 2 commits intotscircuit:mainfrom
mohan-bee:fix-res
Open

Increase GLB export board texture resolution from 512 to 1024#3049
mohan-bee wants to merge 2 commits intotscircuit:mainfrom
mohan-bee:fix-res

Conversation

@mohan-bee
Copy link
Copy Markdown
Contributor

@mohan-bee mohan-bee commented Apr 7, 2026

GLB exports were forcing boardTextureResolution: 512, which made board textures lose detail; switching to 1024 fixes the missing export content.

https://discord.com/channels/1233487248129921135/1490585777149575168

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 7, 2026

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

Project Deployment Actions Updated (UTC)
runframe Ready Ready Preview, Comment Apr 7, 2026 11:22am

Request Review

comment

comment-rm
Copy link
Copy Markdown
Contributor

@rushabhcodes rushabhcodes left a comment

Choose a reason for hiding this comment

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

can you add before and after for proof in the comment

@mohan-bee
Copy link
Copy Markdown
Contributor Author

i have just added my circuit code in runframe for reproduction:

Before:

before.mov

After:

after.mov

@mohan-bee mohan-bee requested a review from rushabhcodes April 7, 2026 12:58
Copy link
Copy Markdown
Contributor

@rushabhcodes rushabhcodes left a comment

Choose a reason for hiding this comment

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

nice

@rushabhcodes rushabhcodes requested a review from seveibar April 7, 2026 14:32
@seveibar seveibar requested a review from imrishabh18 April 7, 2026 14:52
@seveibar
Copy link
Copy Markdown
Contributor

seveibar commented Apr 7, 2026

We used 512 to fix some user boards that wouldn't export i think CC @imrishabh18 can approve if it probably won't be an issue

@github-actions
Copy link
Copy Markdown
Contributor

This PR has been automatically marked as stale because it has had no recent activity. It will be closed if no further activity occurs.

@mohan-bee
Copy link
Copy Markdown
Contributor Author

@imrishabh18 ready for review !

Copy link
Copy Markdown
Member

@imrishabh18 imrishabh18 left a comment

Choose a reason for hiding this comment

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

I think this would should be okay now, as there have been other performance improvements on rendering, so increasing the texture resolution won't effect much

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