-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[PY3] Unit test CondCore/SiStripPlugins #28837
Comments
A new Issue was created by @silviodonato Silvio Donato. @Dr15Jones, @smuzaffar, @silviodonato, @makortel, @davidlange6, @fabiocos can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
assign db |
New categories assigned: db @ggovi you have been requested to review this Pull request/Issue and eventually sign? Thanks |
assign core |
New categories assigned: core @Dr15Jones,@smuzaffar,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks |
as far as I can see the problem is not due to |
naively we should not set PYTHONHOME by hand. Seems a missing dependency in the Buildfile? |
as far as I can see @smuzaffar did the same here: aff1edc ... |
@mmusich @ggovi
A unit test of CondCore/SiStripPlugins is crashing in Python3 since CMSSW_11_1_PY3_X_2020-01-28-2300 because of #28761 .
The problems is related to
boost::python
used in theCondCore/SiStripPlugins/test/testSiStripPayloadInspector.cpp
I tried to remove boost::python (see #28836) and the test is ok also in python3.
The problem arises from the method
PlotBase::setInputParamValues(const boost::python::dict& values)
that was integrated in CMSSW with #28648.@davidlange6 is it allowed to use boost::python? Is there a way to make it compatible to both python 2 and 3?
The text was updated successfully, but these errors were encountered: