Skip to content

Commit

Permalink
Fixing the access to APEs
Browse files Browse the repository at this point in the history
  • Loading branch information
Luca committed Apr 3, 2017
1 parent 8b88938 commit 5f8c0d9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Alignment/MuonAlignment/python/geometryXMLparser.py
Expand Up @@ -57,7 +57,7 @@ class Alignable:
def pos(self):
return self.x, self.y, self.z
def covariance(self):
return (self.xx, self.xy, self.xz, self.xa, self.xb, self.xc), (self.xy, self.yy, self.yz), (self.xz, self.yz, self.zz)
return (self.xx, self.xy, self.xz, self.xa, self.xb, self.xc), (self.xy, self.yy, self.yz, self.ya, self.yb, self.yc), (self.xz, self.yz, self.zz, self.za, self.zb, self.zc), (self.xa, self.ya, self.za, self.aa, self.ab, self.ac), (self.xb, self.yb, self.zb, self.ab, self.bb, self.bc), (self.xc, self.yc, self.zc, self.ac, self.ac, self.cc)

class DTAlignable:
def index(self):
Expand Down

1 comment on commit 5f8c0d9

@lpernie
Copy link
Contributor

@lpernie lpernie commented on 5f8c0d9 Jun 8, 2017

Choose a reason for hiding this comment

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

Dear Experts (@ghellwig),
I do not understand the purpose of this line:

I believe I should initialize the parameter as done this, but I generally use this file in a workflow that is external to cmssw, and that line result in an exception. So the only way I have to use to use geometryXMLparser is to comment that line. Am I missing something? Should not "survey" be false by definition?

Please sign in to comment.