Closed tjlane closed 7 years ago
So not in this, which (upon reflection) probably should be added before merge:
This PR aims to address #10
Currently done:
To be completed soon:
@chuckie82 coffset changes merged here resolves #16
OK, thanks. Could you show me an example for translate(0,0,Z) that I should be using?
Super simple:
from psgeom import camera
g = camera.Cspad.from_psana_file('ref_files/refgeom_psana.data')
print g.xyz
g.translate((0,0,-100000))
print g.xyz
I'll add a test for translate roundtrip.
@tjlane Let's merge this PR. It will be easier for me to test out newdets and distance all in one.
@chuckie82 sorry for the delayed response. I think we can do this. For my records, to be implemented:
New detectors pull request. I think this is good to go.
@chrisvam @chuckie82 please review.