Skip to content
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.

Sign up for GitHub

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

Jump to bottom

PIXL pipeline: Testing for "other" imaging modalities being dropped #193

Open
stefpiatek opened this issue Nov 20, 2023 · 0 comments · May be fixed by #594
Open

PIXL pipeline: Testing for "other" imaging modalities being dropped #193

stefpiatek opened this issue Nov 20, 2023 · 0 comments · May be fixed by #594
Milestone
VOXL

Comments

@stefpiatek
Copy link
Contributor

stefpiatek commented Nov 20, 2023 •
edited
Loading

Definition of Done / Acceptance Criteria

Tests exist with different imaging modalities (metadata of messages set to be X-ray, CT, MRI, ultrasound) as inputs, ensure that modalities that are not being processed are dropped out during de-identification in pixl_dcmd

Testing

No response

Documentation

No response

Dependencies

No response

Details and Comments

Current testing doesn't test for imaging modalities being dropped if they are not accepted by PIXL, should tests that address this so we can have certainty that the pipeline doesn't transfer more data than it should
@stefpiatek stefpiatek transferred this issue from another repository Dec 18, 2023
@stefpiatek stefpiatek added this to the 100-days milestone Dec 18, 2023
@stefpiatek stefpiatek changed the title PIXL pipeline: end to end testing for other modalities being dropped PIXL pipeline: Testing for "other" imaging modalities being dropped Jan 15, 2024
@stefpiatek stefpiatek modified the milestones: 100-days, VOXL Jan 29, 2024
@stefpiatek stefpiatek mentioned this issue Mar 4, 2024
Open
11 tasks
@tomaroberts tomaroberts linked a pull request Jan 20, 2025 that will close this issue
Open
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
VOXL
Development

Successfully merging a pull request may close this issue.

Adds test to check modalities not specified in project config are dropped SAFEHR-data/PIXL
3 participants
@jeremyestein @stefpiatek @HChughtai