-
Notifications
You must be signed in to change notification settings - Fork 280
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
fix: nondeterminism in renderer #1316
Conversation
Looks like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quick note about backwards compatibility before I review the rest.
I guess the |
± Registry diff
📊 PerformanceKeyNote that each bar component rounds up to the nearest 100ms, so each full bar is an overestimate by up to 400ms.
Data
|
Deploying with
|
Latest commit: |
1ba7088
|
Status: | ✅ Deploy successful! |
Preview URL: | https://11c074be.penrose-72l.pages.dev |
Branch Preview URL: | https://fix-seed.penrose-72l.pages.dev |
Codecov Report
@@ Coverage Diff @@
## main #1316 +/- ##
==========================================
- Coverage 61.30% 61.28% -0.03%
==========================================
Files 63 63
Lines 7826 7829 +3
Branches 1842 1845 +3
==========================================
Hits 4798 4798
- Misses 2922 2924 +2
- Partials 106 107 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks!
Description
Resolves #1312.
Implementation strategy and design decisions
Line.ts
, stop usinguuid.v4
for unique ID generationRender*
functions take inname
, a client-provided id for the diagram instance to be rendered. And names in the SVG are disambiguated from others instances by a unique instance id.Simple
andGridBox
take inname
too.uuid
fromcore
's dependenciesExamples with steps to reproduce them
Checklist
Open questions
Questions that require more discussion or to be addressed in future development: