Skip to content

[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

Merged
merged 2 commits into from
Dec 21, 2022

Conversation

osschar
Copy link
Contributor

@osschar osschar commented Dec 20, 2022

  • 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.
  • 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.

… 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.
@phsft-bot
Copy link

Starting build on ROOT-debian10-i386/soversion, ROOT-performance-centos8-multicore/cxx17, ROOT-ubuntu18.04/nortcxxmod, ROOT-ubuntu2004/python3, mac12/noimt, mac11/cxx14, windows10/cxx14
How to customize builds

@phsft-bot
Copy link

Build failed on ROOT-performance-centos8-multicore/cxx17.
Running on olbdw-01.cern.ch:/data/sftnight/workspace/root-pullrequests-build
See console output.

Warnings:

  • [2022-12-20T07:29:41.094Z] /data/sftnight/workspace/root-pullrequests-build/root/graf3d/eve7/src/REvePointSet.cxx:64:1: warning: base class ‘class ROOT::Experimental::REveSecondarySelectable’ should be explicitly initialized in the copy constructor [-Wextra]

Copy link
Member

@linev linev left a 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.

@Axel-Naumann
Copy link
Member

Can we please have the warning fixed?

@phsft-bot
Copy link

Starting build on ROOT-debian10-i386/soversion, ROOT-performance-centos8-multicore/cxx17, ROOT-ubuntu18.04/nortcxxmod, ROOT-ubuntu2004/python3, mac12/noimt, mac11/cxx14, windows10/cxx14
How to customize builds

@linev
Copy link
Member

linev commented Dec 20, 2022

@Axel-Naumann Warning should be fixed now by my commit

@phsft-bot
Copy link

Build failed on windows10/cxx14.
Running on null:C:\build\workspace\root-pullrequests-build
See console output.

@Axel-Naumann
Copy link
Member

@phsft-bot build just on windows10/cxx14

@phsft-bot
Copy link

Starting build on windows10/cxx14
How to customize builds

@Axel-Naumann Axel-Naumann merged commit 1b1d677 into root-project:master Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants