VP1HepVis: Assertion / error message fix

We were tripping the assertion in SbPolyhedronPolygonXSect::Internals::setData because it could be called with three points (n=3). But that case seems to be handled ok, so change the assertion to n>2.

We also got a bunch of errors from SbPolyhedronCons::SbPolyhedronCons because the tracking geometry can produce zero-height cones. Just ignore those cases.

This merge request affects 1 package:

• graphics/VP1/VP1HEPVis

This merge request affects 1 file:

• graphics/VP1/VP1HEPVis/src/SbPolyhedron.cxx

Adding @rbianchi as watcher

added EventDisplay master review-pending-level-1 labels

CI Result SUCCESS (hash 60e3e36a)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 38675]

Looks good. Approved

Sergei L1

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 796f18c4

added sweep:ignore label