Issue679CTAcquisitionTypeFiltering

Merged
#234 · Created  · Last updated

Merged pull request

Merged in issue679CTAcquisitionTypeFiltering (pull request #234)

0ff008c·Author: ·Closed by: ·2018-11-16

Description

This references issue #679

  • Added a filter for CT acquisition type. References issue #679.

  • Removed PX code that is not relevant to this issue. References issue #679.

  • Added check for presence of ct_acquisition_type to ensure correct data for charts. This fixes the problem I reported earlier. However, there remains an issue with hyperlinks from histogram bars that I need to address. References issue #679

  • Hyperlinks are now sorted. However, there remains an issue with acquisition-level charts - the data isn't correct at the moment. References issue #679

  • Acquisition-level charts are now behaving. I have some tidying to do tomorrow. References issue #679

  • Simplified chart code as all chart now use the same query data. References issue #679

  • Tidied up. Acquisisition level and request level charts are working, as are links from histgrams. Further testing of other chart types to do. References issue #679

  • Moved acquisition type filter to the bottom; forced widget to be checkboxes so that it is easy for a user to deselect all the options; styled the options to remove the leading bullet and indent. References issue #679

  • Added tests for CT export when ct_acquisition_type is set. References issue #679

  • Updated the way the query is constructed - I think this speeds things up, and could be used for the other modalities. References issue #679

  • Updated export code to ensure that the values of the ct_acquisition_type filter are passed as a list. References issue #679

  • Some optimisation of queries to help with speed. References issue #679. I'll post some timings in a minute.

  • Fixed problem where my exports stopped because of a missing value. References issue #679

  • Removing prefetch as I wasn't using it appropriately - replaced with values. References issue #679

  • Updating documents to reflect CT acquisition type filtering [skip ci]. References issue #679

  • Updated changes files. I want to force a build so that I can test the on-line docs. References issue #679.

  • Updated screenshot to include CT acquisition type restrictions [skip ci]. References issue #679

0 attachments

0 comments

Loading commits...