Skip to content

Conversation

@mrbean-bremen
Copy link
Member

This is a backport of 2 commits from the main branch:

  • update to DICOM 2025d
  • fixes to correctly evaluate nested sequences like in SR documents

- evaluate conditions in sequences lazily
  to make sure to check the correct dataset
- this also allows recursively defined sequence attributes

# Conflicts:
#	CHANGES.md
#	dicom_validator/tests/validator/test_iod_validator.py
#	dicom_validator/tests/validator/test_iod_validator_func_groups.py
#	dicom_validator/validator/iod_validator.py
#	dicom_validator/validator/validation_result.py
# Conflicts:
#	CHANGES.md
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 88.23529% with 2 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (version-0.7@60ca1ae). Learn more about missing BASE report.

Files with missing lines Patch % Lines
dicom_validator/validator/iod_validator.py 81.81% 2 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff               @@
##             version-0.7     #215   +/-   ##
==============================================
  Coverage               ?   89.62%           
==============================================
  Files                  ?       16           
  Lines                  ?     1726           
  Branches               ?        0           
==============================================
  Hits                   ?     1547           
  Misses                 ?      179           
  Partials               ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mrbean-bremen mrbean-bremen merged commit d7e4517 into pydicom:version-0.7 Oct 13, 2025
17 checks passed
@mrbean-bremen mrbean-bremen deleted the version-0.7.3 branch October 13, 2025 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants