-
Notifications
You must be signed in to change notification settings - Fork 157
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
refactor!: Remove explicit geometry ID from source link #2356
refactor!: Remove explicit geometry ID from source link #2356
Conversation
This reduces the memory footprint. The access to the surface is now done using a delegate (which I typedef right next to Source Link itself). Components need to handle this individually.
Codecov Report
@@ Coverage Diff @@
## main #2356 +/- ##
==========================================
- Coverage 49.54% 49.52% -0.02%
==========================================
Files 453 453
Lines 25679 25677 -2
Branches 11811 11815 +4
==========================================
- Hits 12722 12716 -6
Misses 4583 4583
- Partials 8374 8378 +4
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
This allows python/pytest to catch them and report as failures, instead of tearing down the whole process. I don't think we care about the runtime implications of throwing exceptions here.
This reverts commit 8c742a0.
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.
Only one thing I would like to clarify, otherwise looks good!
Still some tests seem to rely on the old interface @paulgessinger |
This should hopefully finally be good. |
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 good!
RTD is stuck, I'm merging manually. |
This reduces the memory footprint. The access to the surface is now done using a delegate (which I typedef right next to Source Link itself). Components need to handle this individually.