Skip to content

Conversation

@benjamaan476
Copy link
Contributor

Introduce a DEFAULT_BACKFACE_COLOR that is currently hard coded and sent to the engine on connection. This can be pulled in from user space with a follow-up effort

@benjamaan476 benjamaan476 requested review from a team as code owners December 10, 2025 13:40
@vercel
Copy link

vercel bot commented Dec 10, 2025

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

Project Deployment Preview Comments Updated (UTC)
modeling-app Ready Ready Preview Comment Dec 10, 2025 1:54pm

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 10, 2025

CodSpeed Performance Report

Merging #9290 will not alter performance

Comparing ben/default_backface (333cd5d) with main (5d11754)1

Summary

✅ 148 untouched
⏩ 92 skipped2

Footnotes

  1. No successful run was found on main (260e382) during the generation of this report, so 5d11754 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

  2. 92 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

Comment on lines -87 to +88
# [patch.crates-io]
# kcl-ezpz = { path = "../../ezpz/kcl-ezpz" }
# kittycad-modeling-cmds = { git = "https://github.com/KittyCAD/modeling-api.git", branch = "ben/chamfer_angle" }
[patch.crates-io]
kittycad-modeling-cmds = { git = "https://github.com/KittyCAD/modeling-api.git", branch = "ben/default_backface" }
Copy link
Contributor

Choose a reason for hiding this comment

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

We should remove this patch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah will do when the modeling-api is released with the required changes

@jacebrowning jacebrowning marked this pull request as draft December 11, 2025 18:49
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.

3 participants