Merged
Conversation
5236b94 to
c757898
Compare
eae2801 to
e044a47
Compare
3 tasks
Merged
405045d to
0a8ccdc
Compare
0a8ccdc to
398a44b
Compare
398a44b to
a016243
Compare
9bd98b7 to
e462f62
Compare
cf57cb2 to
451489d
Compare
Draft
451489d to
ef552a4
Compare
- use GAT on Element type - add element_opt and element_by_name_opt
- 1-level deep copy of a DICOM value
- add DicomAttributeValue trait, use it as output of most methods in DicomObject - rename methods and add new ones in DicomObject - use GATs for the attribute value, item and pixel data types - reimplement DicomObject
- provide access to items and pixel data fragments - require DicomAttributeValue to impl DicomValueType - add a few tests
- rename from DicomAttributeValue - add `to_i32` and `to_u16`
- instead, meta attributes can be retrieved like any other attribute
- replace `to_dicom_value` with `to_primitive_value` - simplifies implementations and requires consumers to depend on other methods when working with sequences - remove method `meta` - treat meta information attributes like any other attribute, retrievable through the same methods - [core] add either crate - impl many DICOM traits to `either::Either` - implement DicomObject for FileMetaTable - reimplement DICOM traits for FileDicomObject so that users can retrieve either meta info or main data set info
- elide lifetime - fix formatting of doc comments
- elide lifetime parameter in impl DicomAttribute for FileMetaAttribute
- add methods `split_first` and `len` - fix a few doc comments
- add associated type `LeafAttribute<'_>` - opens the possibility of the attribute type being different when doing a deeper look-up - add method `DicomObject::at` - extend documentation to advise when to use `DicomObject - fix impl of DicomObject for &FileDicomObject<O> - it was not consulting like the non-ref counterpart
- rename AttributeSelector::len to AttributeSelector::num_steps, since semantics are sufficiently different (is_empty would have been meaningless) - deferefence first step instead of cloning it
- remove needless borrow
- show ellided lifetime in Cow
53d160e to
704056e
Compare
- fix a few minor mistakes and repetitions
- use `attr` instead of `get` so that they do not collide with other methods
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
A new iteration of the trait-based DICOM object abstraction, which may help uniformize DICOM object handling regardless of the underlying implementation.
Summary
Known caveats
MethodsRenamed them togetare nice and short, but they conflict withInMemDicomObject::getand do not work the same way. I might need to rename them again.attr.