-
assigned issue to
Ed McDonagh
Display "Requested procedure" on all filtered html pages
Issue #544
resolved
This is only displayed on the CT and radiographic pages at the moment. I would find it useful on the fluoroscopy page, and figure we may as well add it to the mammo one too.
Comments (20)
-
-
Added fields to search and display. Testing will be limited due to lack of procedure information in test DICOM. Refs
#544→ <<cset aab0ffab695c>>
-
Named summary list filter urls so that they can be used in testing. Added tests for RF - currently failing for acquisition filter as study is listed more than once. Refs
#544→ <<cset cc2de56e8508>>
-
reporter
Having four items in the same cell and being consistent for all modalities is fine with me.
-
Thanks @dplatten.
Can you tell me what 'studyhist' in
mod_filters.py ct_acq_filterintroduced in commit d47a9d9a738cc refers to?I'm in the process of recreating the fix for filtering on event level data (acquisition protocol) causing multiple repeats in the summary list.
-
Added order_by and distinct to avoid duplicates when filtering on event level data. Refs
#544→ <<cset 72f1b92df544>>
-
reporter
studyhist is set to 1 in urls attached to histogram data bars of plots of DLP and CTDIvol per study description. The code that puts them there is in ctfiltered.html on lines 567 and 666.
I think that it is set to ensure that when a user clicks on a histogram link from one of the above-mentioned plots that the resulting data is filtered by the appropriate study description.
-
Thanks
-
Added Procedure to rf detail page. Refs
#544→ <<cset 57262aa7cf78>>
-
Modified mammo to return distinct studies when filtering on acquisition protocol Refs
#544→ <<cset 24d69d936520>>
-
Added Procedure to DX. Need to add tests. Refs
#544→ <<cset 2425dec3221b>>
-
Minor corrections Refs
#544→ <<cset 46c3cc90dedf>>
-
Start of test file for dx filters Refs
#544→ <<cset cad77f3781c2>>
-
Added procedure to CT filtering and summary list display. Reordered filters to match other modalities Refs
#544→ <<cset 8ae8e9fe4beb>>
-
Adde procedure to CT detail page Refs
#544→ <<cset 240b6e19ee73>>
-
Removed extra blank line Refs
#544→ <<cset ac9fbb6e410d>>
-
New CT summary view filtering tests Refs
#544→ <<cset 77c88b14cfa0>>
-
Added CT procedure and requested procedure filtering tests Refs
#544→ <<cset 29065276f419>>
-
Updated changes Refs
#544→ <<cset fe318cc11343>>
-
- changed status to resolved
Merged in issue544procedurenames (pull request #120)
Fixes
#544Approved-by: David Platten dplatten@gmail.com Approved-by: Ed McDonagh ed@mcdonagh.org.uk
→ <<cset f5e7be18f2b1>>
- Log in to comment
I'd like to broaden this out to ensure consistency across modalities based on changes made for issue
#507.Each modality should display in the filtering list the following:
Each of these should be searchable, along with acquisition protocol.
I'd appreciate anyone's thoughts on whether having four items in the same cell (including accession number) is too much. You can see the effect in the mammo tab on testing or dev demo sites.
@tcdewit, @luuko, @dplatten, anyone else!