Display "Requested procedure" on all filtered html pages

Issue #544 resolved
created an issue

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)

  1. Ed McDonagh

    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:

    • Study description
    • Procedure
    • Requested Procedure

    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!

  2. Ed McDonagh

    Added fields to search and display. Testing will be limited due to lack of procedure information in test DICOM. Refs #544

    → <<cset aab0ffab695c>>

  3. Ed McDonagh

    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>>

  4. David Platten reporter

    Having four items in the same cell and being consistent for all modalities is fine with me.

  5. Ed McDonagh

    Thanks @dplatten.

    Can you tell me what 'studyhist' in mod_filters.py ct_acq_filter introduced 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.

  6. Ed McDonagh

    Added order_by and distinct to avoid duplicates when filtering on event level data. Refs #544

    → <<cset 72f1b92df544>>

  7. David Platten 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.

  8. Ed McDonagh

    Modified mammo to return distinct studies when filtering on acquisition protocol Refs #544

    → <<cset 24d69d936520>>

  9. Ed McDonagh

    Added procedure to CT filtering and summary list display. Reordered filters to match other modalities Refs #544

    → <<cset 8ae8e9fe4beb>>

  10. Log in to comment