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
DM-42643: Pre-process data into catalog format for analysis_tools #37
Conversation
964e512
to
95ae809
Compare
|
||
def repackRunStats(self, runStats): | ||
# for runStats in runStats: | ||
raise NotImplementedError("Repack needs to be defined by subclasses.") |
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.
Are repackExpStats and repackRunStats used anywhere? It looks like it is just repackDetStats.
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.
They are not, but I wanted to be able to access them in the future if they became useful.
95ae809
to
a4eaa7d
Compare
def runQuantum(self, butlerQC, inputRefs, outputRefs): | ||
inputs = butlerQC.get(inputRefs) | ||
|
||
inputs["detectorDims"] = [exp.dataId.byName() for exp in inputRefs.detectorStats] |
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.
This should be dict(exp.dataId.required)
instead of exp.dataId.byName()
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.
That explains the warnings I'm seeing. I'm going to wait for the next ticket to fix this, as I have been struggling to get this ticket to pass Jenkins for the merge.
def runQuantum(self, butlerQC, inputRefs, outputRefs): | ||
inputs = butlerQC.get(inputRefs) | ||
|
||
inputs["detectorDims"] = [exp.dataId.byName() for exp in inputRefs.detectorStats] |
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.
Also here
No description provided.