Skip to content

Commit

Permalink
Add ID field to DICOM query
Browse files Browse the repository at this point in the history
  • Loading branch information
@ryansanford
ryansanford committed Jul 11, 2016
1 parent 655df5d commit 2d2b7fe
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion reaper/dicom_reaper.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ def state_str(self, _id, state):
def instrument_query(self):
i_state = {}
scu_studies = None
scu_series = self.scu.find(scu.SeriesQuery(**scu.SCUQuery()))
scu_series = self.scu.find(scu.SeriesQuery(**scu.SCUQuery(**{self.id_field : ''})))
if scu_series is None:
return None
for series in scu_series:
Expand Down

2 comments on commit 2d2b7fe

@gsfr
Copy link
Member

@gsfr gsfr commented on 2d2b7fe Jul 11, 2016

Choose a reason for hiding this comment

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

Interesting. This should only be necessary, if you are using an id field that is not in the query template in scu.py, which wasn't my original idea, but I like it. How about we also add self.opt_field here (if not None) and trim the query template to the actually-required minimum?

@ryansanford
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had a similar thought. I'll add that change to the PR.

Please sign in to comment.