Skip to content

Conversation

@gserena01
Copy link
Contributor

@gserena01 gserena01 commented Oct 9, 2025

@vercel
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 Nov 13, 2025 10:41pm

@codspeed-hq
Copy link

codspeed-hq bot commented Oct 9, 2025

CodSpeed Performance Report

Merging #8525 will improve performances by 10.25%

Comparing serena/cloning-set-surfaces (ccec5bd) with main (e4e4c7e)1

Summary

⚡ 1 improvement
✅ 138 untouched
⏩ 92 skipped2

Benchmarks breakdown

Mode Benchmark BASE HEAD Change
Simulation mock_execute_mike_stress_test_program 356.3 ms 323.1 ms +10.25%

Footnotes

  1. No successful run was found on main (6d90865) during the generation of this report, so e4e4c7e 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
@gserena01 gserena01 merged commit 6ed08d9 into main Nov 14, 2025
79 of 81 checks passed
@gserena01 gserena01 deleted the serena/cloning-set-surfaces branch November 14, 2025 16:39
adamchalmers pushed a commit that referenced this pull request Nov 14, 2025
* update snapshot

* update surfaces of clones

* fmt and clippy

* unfix clipppy sryyy

* Fix lint

* fmt

* udpate according to suggestions

* udpate according to suggestions

* fix excessive if let

* stopping point

* update close fn

* update samples

* clippy fix

* update snapshots, clean up optioonal handling

* fmt

* stopping poiint in case i mess things up

* this version works?

* revert kcl file

* fmt

* overwrite sls and saturn tests

* clippy fix

* clippy fix

* only use map if its there

* fmt

* add warning

* add warning

* cleanup

* Update snapshots

* revert

* Update rust/kcl-lib/src/std/extrude.rs

Co-authored-by: Jonathan Tran <[email protected]>

* make install

---------

Co-authored-by: gserena <[email protected]>
Co-authored-by: Adam Sunderland <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Jonathan Tran <[email protected]>
Co-authored-by: Jace Browning <[email protected]>
adamchalmers pushed a commit that referenced this pull request Nov 14, 2025
* update snapshot

* update surfaces of clones

* fmt and clippy

* unfix clipppy sryyy

* Fix lint

* fmt

* udpate according to suggestions

* udpate according to suggestions

* fix excessive if let

* stopping point

* update close fn

* update samples

* clippy fix

* update snapshots, clean up optioonal handling

* fmt

* stopping poiint in case i mess things up

* this version works?

* revert kcl file

* fmt

* overwrite sls and saturn tests

* clippy fix

* clippy fix

* only use map if its there

* fmt

* add warning

* add warning

* cleanup

* Update snapshots

* revert

* Update rust/kcl-lib/src/std/extrude.rs

Co-authored-by: Jonathan Tran <[email protected]>

* make install

---------

Co-authored-by: gserena <[email protected]>
Co-authored-by: Adam Sunderland <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Jonathan Tran <[email protected]>
Co-authored-by: Jace Browning <[email protected]>
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.

5 participants