-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[REve] Update to RenderCore release from github, deploy it as builtin tarball #11936
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
Conversation
… tarball. - Move some RenderCore specific classes and all shaders into RenderCore source tree. - Implement all existing Three.js functionality with RenderCore. - In RenderCore, further improve on existing functionality: - Textured points - arbitrary size and shape. - Mesh lines - arbitrary thickness. - Object and sub-object picking through color buffer with fragment depth detection. - Camera handling like in TEve, with ability to pick rotation center. C++ / Server Changes - REveManager: Make RenderCore the default rendering engine. - Stream interpreter errors and show them in client log - Add flag fIsRCore to REveManager. - Make REvePointSet secondary-selectable. - Add parameter for projection depth in REveDataCollection JS / Client Changes - Remove error MessageBox popup during client updates. - EveManager.js: Check change bits in ImportSceneBinary when scene changes are streamed. - EveScene.js: Optimize visibility changes. - EveScene.js: pass eve_element instead of object3d in cases where it simplifies the code. - EveScene.js, GlViewer classes: Cleamup highlite/selection management so it can be abstracted for RCore. - Separate viewer-side selection processing for Three / RCore. Tutorials / Demos Changes - event_demo.C: secondary select points, jet title include pT. - Tune marker sizes, colors, visual parameters.
Starting build on |
Build failed on ROOT-performance-centos8-multicore/cxx17. Warnings:
|
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 ok.
I test several tutorials - all them work.
Can we please have the warning fixed? |
Starting build on |
@Axel-Naumann Warning should be fixed now by my commit |
Build failed on windows10/cxx14. |
@phsft-bot build just on windows10/cxx14 |
Starting build on |
Move some RenderCore specific classes and all shaders into RenderCore source tree.
Implement all existing Three.js functionality with RenderCore.
In RenderCore, further improve on existing functionality:
The development branch has been squashed to hide redundant changes. Full history is available in PR [REve] Update to RenderCore release from github, deploy it as builtin tarball. osschar/root#41.
C++ / Server Changes
REveManager: Make RenderCore the default rendering engine.
Stream interpreter errors and show them in client log
Add flag fIsRCore to REveManager.
Make REvePointSet secondary-selectable.
Add parameter for projection depth in REveDataCollection
JS / Client Changes
Remove error MessageBox popup during client updates.
EveManager.js: Check change bits in ImportSceneBinary when scene changes are streamed.
EveScene.js: Optimize visibility changes.
EveScene.js: pass eve_element instead of object3d in cases where it simplifies the code.
EveScene.js, GlViewer classes: Cleamup highlite/selection management so it can be abstracted for RCore.
Separate viewer-side selection processing for Three / RCore.
Tutorials / Demos Changes
event_demo.C: secondary select points, jet title include pT.
Tune marker sizes, colors, visual parameters.