Skip to content

Conversation

gserena01
Copy link
Contributor

@gserena01 gserena01 commented Oct 9, 2025

Copy link

vercel bot commented Oct 9, 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 Oct 14, 2025 5:56pm

Copy link

codspeed-hq bot commented Oct 9, 2025

CodSpeed Performance Report

Merging #8525 will not alter performance

Comparing serena/cloning-set-surfaces (6a9839f) with main (bf9d019)1

Summary

✅ 139 untouched
⏩ 92 skipped2

Footnotes

  1. No successful run was found on main (d70d6b4) during the generation of this report, so bf9d019 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.

@gserena01 gserena01 marked this pull request as ready for review October 9, 2025 22:41
@gserena01 gserena01 requested review from a team as code owners October 9, 2025 22:41
exec_state: &mut ExecState,
args: &Args,
edge_id: Option<Uuid>,
clone_id_map: Option<&HashMap<Uuid, Uuid>>, // old sketch id -> new sketch id
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think the new param is ever passed in.

Path::Arc { .. }
| Path::TangentialArc { .. }
| Path::TangentialArcTo { .. }
// TODO: (bc) fix me
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems like forgery to sign this as Ben, but I won't stop you 🙂

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