fix(export): add helper and expose new api on widget - #103
Conversation
f154bf2 to
2e597ae
Compare
| from trame_vtklocal.utils import exporter | ||
|
|
||
|
|
||
| def build_vtk_pipeline(): |
There was a problem hiding this comment.
@jspanchu it would be great to showcase a beautiful and complex scene instead of the cone. With Axis grid and more?
Are you interested to improve that before I merge this PR?
There was a problem hiding this comment.
Definitely. Will work on it. I’ve some ideas.
|
LGTM |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #103 +/- ##
==========================================
- Coverage 56.71% 52.00% -4.72%
==========================================
Files 12 13 +1
Lines 506 673 +167
==========================================
+ Hits 287 350 +63
- Misses 219 323 +104 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
0bfaa9d to
f1f1645
Compare
| function createClientSideObject(native, state) { | ||
| const objectId = nextClientId; | ||
| native.registerState({ | ||
| ...state, | ||
| Id: nextClientId++, | ||
| MTime: 1, | ||
| "vtk-object-manager-kept-alive": true, | ||
| }); | ||
| native.updateObjectsFromStates(); | ||
| return getVtkObject(objectId); | ||
| } |
There was a problem hiding this comment.
this is the best we can do for now without making a C++ change that unlocks the creation/destroy of objects in vtkRemoteSession
d22cce2 to
4fa4af0
Compare
No description provided.