Skip to content

Debugging engine inner edge IDs #7756

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 5 commits into from

Conversation

adamchalmers
Copy link
Contributor

@adamchalmers adamchalmers commented Jul 10, 2025

@gserena01 run

cargo nextest run -p kcl-lib --nocapture -E "test(tag_inner_face::kcl_test)"

to get this output:

ADAM: Curve -> Face ID map from engine
        ff644541-0515-5cbb-b057-003c3c1247eb -> Some(b04d04fa-99c4-5f05-a3f5-19f132230c6a)
        8a662fcd-9569-5482-8c42-aafff7b3f8d4 -> Some(0b12479f-fdc6-5717-9af0-ad5761ebabd9) (outer)
        6ee626a0-e8d9-51eb-a0e2-9975ea89d632 -> Some(ddd2fa0f-40a4-5040-8cf5-437440ba9fb8)
        89316223-2ea1-5731-9c72-0492bc30fbe2 -> Some(97da2c9d-1563-5f62-8b16-e3df01445dc1) (outer)
        f351621c-3247-5e9f-b8eb-d566e0802ef1 -> Some(3e16a80d-31d4-576f-aae8-1bf36b028e5a)
        daa2fbb0-8c3a-525f-bc28-3b5a77c8094b -> Some(d32d2839-ebf9-5d59-863a-8608a0b7334a) (outer)
ADAM: client knows about 3 outer paths
        8a662fcd-9569-5482-8c42-aafff7b3f8d4
        89316223-2ea1-5731-9c72-0492bc30fbe2 (seg01)
        daa2fbb0-8c3a-525f-bc28-3b5a77c8094b
ADAM: client knows about 3 inner paths
        f5cca6d8-813c-5e62-81d9-e4e8e17ede93
        2fb0a795-692e-5806-9d7e-a6d1b5ee79b7 (yoyo)
        38f0111a-8034-5e85-9ec1-07dbc9646b06
ADAM: Outer
        ADAM: engine reported this path
        ADAM: engine reported this path, tag = seg01
        ADAM: engine reported this path
ADAM: Inner
        ADAM: engine didn't report this path
        ADAM: engine didn't report this path, tag = yoyo
        ADAM: engine didn't report this path

Copy link

vercel bot commented Jul 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
modeling-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 15, 2025 10:24pm

Copy link

codspeed-hq bot commented Jul 10, 2025

CodSpeed Instrumentation Performance Report

Merging #7756 will not alter performance

Comparing achalmers/debugging-engine-inner-ids (d82ec6f) with main (b52ade2)

Summary

✅ 85 untouched benchmarks

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.

1 participant