Skip to content

Support engine execution for KCL commands#1711

Merged
adamchalmers merged 1 commit into
mainfrom
achalmers/export-remote
Jul 7, 2026
Merged

Support engine execution for KCL commands#1711
adamchalmers merged 1 commit into
mainfrom
achalmers/export-remote

Conversation

@adamchalmers

@adamchalmers adamchalmers commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Setting $ENGINE_EXECUTION to any non-empty string will execute KCL on the engine instead of locally.

This produces a significant speedup:

engine_exec_speedup
  • Engine execution takes 26 seconds
  • Local execution which takes 101 seconds
  • This means engine execution is 4x as fast!
  • Also incidentally uses 5x less memory (which wasn't part of the motivation, but is nice)

Measured against this test model.

@adamchalmers adamchalmers force-pushed the achalmers/export-remote branch from aacdcd2 to 938b7cc Compare July 7, 2026 01:43
@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 77.71739% with 82 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.75%. Comparing base (d038ace) to head (19aac63).

Files with missing lines Patch % Lines
src/context.rs 66.11% 82 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1711      +/-   ##
==========================================
+ Coverage   66.28%   66.75%   +0.47%     
==========================================
  Files          44       45       +1     
  Lines        8476     8843     +367     
==========================================
+ Hits         5618     5903     +285     
- Misses       2858     2940      +82     
Flag Coverage Δ
unittests 66.75% <77.71%> (+0.47%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@adamchalmers adamchalmers marked this pull request as ready for review July 7, 2026 03:43
@adamchalmers adamchalmers requested a review from jessfraz as a code owner July 7, 2026 03:43
Comment thread src/cmd_kcl.rs
@adamchalmers adamchalmers changed the title Achalmers/export remote Support engine execution for KCL commands Jul 7, 2026
Comment thread Cargo.toml Outdated
@adamchalmers adamchalmers force-pushed the achalmers/export-remote branch 6 times, most recently from ef66c23 to ea82e62 Compare July 7, 2026 21:24
This is around 20x faster on large models.
Set ENGINE_EXECUTION to any nonempty string to activate.
Off by default.
@adamchalmers adamchalmers force-pushed the achalmers/export-remote branch from ea82e62 to 19aac63 Compare July 7, 2026 21:25
@adamchalmers adamchalmers enabled auto-merge (squash) July 7, 2026 21:29
@adamchalmers adamchalmers merged commit 90dcf3d into main Jul 7, 2026
16 of 19 checks passed
@adamchalmers adamchalmers deleted the achalmers/export-remote branch July 7, 2026 21:57
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.

2 participants