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
Move MTD geometry test from expanded to filtered view #25217
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25217/7229 |
A new Pull Request was created by @fabiocos (Fabio Cossutti) for master. It involves the following packages: Geometry/MTDCommonData @civanch, @Dr15Jones, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @kpedro88 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test not testing anything, the test modules have been probed with the standalone configuration provided in the Geometry/MTDCommonData package |
The tests are being triggered in jenkins. |
@ianna please let me know whether this is what you had in mind, in this case I do not see a specific attribute to select without changing the logic of the code |
@@ -17,9 +17,9 @@ | |||
#include "Geometry/Records/interface/IdealGeometryRecord.h" | |||
|
|||
#include "DetectorDescription/Core/interface/DDValue.h" | |||
#include "DetectorDescription/Core/interface/DDCompactView.h" | |||
#include "DetectorDescription/Core/interface/DDExpandedNode.h" |
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.
@fabiocos - is it needed?
|
||
edm::LogInfo("TestMTDNumbering") << "Top Most LogicalPart = " << fv.logicalPart(); | ||
|
||
typedef DDFilteredView::nav_type nav_type; |
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.
using nav_type = DDFilteredView::nav_type;
|
||
edm::LogInfo("TestMTDNumbering") << "Top Most LogicalPart = " << fv.logicalPart(); | ||
|
||
typedef DDFilteredView::nav_type nav_type; | ||
typedef std::map<nav_type,int> id_type; |
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.
using id_type = std::map<nav_type,int>;
@@ -18,9 +18,9 @@ | |||
#include "Geometry/Records/interface/IdealGeometryRecord.h" | |||
|
|||
#include "DetectorDescription/Core/interface/DDValue.h" | |||
#include "DetectorDescription/Core/interface/DDCompactView.h" | |||
#include "DetectorDescription/Core/interface/DDExpandedNode.h" |
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.
is it needed?
Comparison job queued. |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
Comparison Summary:
|
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25217/7234 |
Pull request #25217 was updated. @civanch, @Dr15Jones, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @kpedro88 can you please check and sign again. |
+1 |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
Comparison Summary:
|
@kpedro88 this is the code cleaning and move to filtered view of two standalone test classes not used in any production workflow |
+upgrade |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
This PR addresses a comment of @ianna about using only filtered views for geometry navigation.
As in the case of these tests the whole geometry tree structure is scanned, a pass through filter is used.