Skip to content

[WIP][PyROOT] Move interoperability functions to ROOT.Numpy namespace#5252

Closed
stwunsch wants to merge 5 commits into
root-project:masterfrom
stwunsch:numpy-asrvec
Closed

[WIP][PyROOT] Move interoperability functions to ROOT.Numpy namespace#5252
stwunsch wants to merge 5 commits into
root-project:masterfrom
stwunsch:numpy-asrvec

Conversation

@stwunsch

@stwunsch stwunsch commented Mar 30, 2020

Copy link
Copy Markdown
Contributor

The new namespace allows us to import libraries more lazily. Added a test that we load exactly the libraries that we expect.

@stwunsch
stwunsch requested a review from etejedor March 30, 2020 17:05
@stwunsch stwunsch self-assigned this Mar 30, 2020
@phsft-bot

Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos7-multicore/default, ROOT-fedora29/python3, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1015/cxx17, windows10/cxx14
How to customize builds

@phsft-bot

Copy link
Copy Markdown

Build failed on ROOT-debian10-i386/cxx14.
See cdash See console output.

Failing tests:

@hageboeck hageboeck left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shallow review OK, but leave for expert.

This allows us to inject dependencies associated with AsRVec lazily.
This allows us to inject TMVA and its dependencies fully lazily.
@phsft-bot

Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos7-multicore/default, ROOT-fedora29/python3, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1015/cxx17, windows10/cxx14
How to customize builds

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos7-multicore/default, ROOT-fedora29/python3, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1015/cxx17, windows10/cxx14
How to customize builds

@stwunsch stwunsch changed the title [WIP][PyROOT] Move AsRVec to ROOT.Numpy [WIP][PyROOT] Move interoperability functions to ROOT.Numpy namespace Mar 31, 2020
@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos7-multicore/default, ROOT-fedora29/python3, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1015/cxx17, windows10/cxx14
How to customize builds

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

Build failed on ROOT-performance-centos7-multicore/default.
See cdash See console output.

Failing tests:

@phsft-bot

Copy link
Copy Markdown

Build failed on windows10/cxx14.
See cdash See console output.

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos7-multicore/default, ROOT-fedora29/python3, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1015/cxx17, windows10/cxx14
How to customize builds

@phsft-bot

Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos7-multicore/default, ROOT-fedora29/python3, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1015/cxx17, windows10/cxx14
How to customize builds

@root-project root-project deleted a comment from phsft-bot Apr 1, 2020
@phsft-bot

Copy link
Copy Markdown

@hageboeck hageboeck left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shallow review looks ok. Maybe the commit message of #5 is a bit too generic if you don't know what the 4 previous commits were.

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

Build failed on ROOT-performance-centos7-multicore/default.
See cdash See console output.

Failing tests:

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos7-multicore/default, ROOT-fedora29/python3, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1015/cxx17, windows10/cxx14
How to customize builds

@phsft-bot

Copy link
Copy Markdown

Build failed on windows10/cxx14.
See cdash See console output.

Errors:

  • [2020-04-01T09:53:36.366Z] CMake Error at cmake/modules/SearchRootCoreDeps.cmake:52 (if):
  • [2020-04-01T09:53:36.366Z] CMake Error at C:/build/workspace/root-pullrequests-build/rootspi/jenkins/root-build.cmake:941 (message):

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

Build failed on ROOT-performance-centos7-multicore/default.
See cdash See console output.

Failing tests:

@stwunsch

stwunsch commented Apr 2, 2020

Copy link
Copy Markdown
Contributor Author

@phsft-bot build

@phsft-bot

Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos7-multicore/default, ROOT-fedora29/python3, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1015/cxx17, windows10/cxx14
How to customize builds

@phsft-bot

Copy link
Copy Markdown

Build failed on ROOT-performance-centos7-multicore/default.
See cdash See console output.

Failing tests:

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos7-multicore/default, ROOT-fedora29/python3, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1015/cxx17, windows10/cxx14
How to customize builds

@phsft-bot

Copy link
Copy Markdown

Build failed on windows10/cxx14.
See cdash See console output.

Errors:

  • [2020-04-02T13:19:50.809Z] CMake Error at cmake/modules/SearchRootCoreDeps.cmake:52 (if):
  • [2020-04-02T13:19:51.337Z] CMake Error at C:/build/workspace/root-pullrequests-build/rootspi/jenkins/root-build.cmake:941 (message):

@phsft-bot

Copy link
Copy Markdown

@phsft-bot

Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos7-multicore/default, ROOT-fedora29/python3, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1015/cxx17, windows10/cxx14
How to customize builds

@phsft-bot

Copy link
Copy Markdown

@stwunsch

stwunsch commented Apr 3, 2020

Copy link
Copy Markdown
Contributor Author

Going to be replaced by #5285. I keep this one a little longer around to figure out which libraries we load on mac.

@stwunsch stwunsch closed this Apr 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants