Comments (4)
Hi, @lmdulz! Maybe you're right. What happens if you change the code?
Possibilities
- I'm generally doing stuff wrong
- older pydicom required something different
- older Python required something different
So many options!
I'll take a gander if I can find some time.
from dicognito.
While MultiValue
objects aren't list
s, they do act enough like list that there was no need to copy the individual values into a list, nor to serialize the results.
I don't really have enough interest to justify the time it would take to experiment with older versions of pydicom to see if this was not so in the past, so I'll just update the code and move on.
Thanks, @lmdulz.
from dicognito.
Unfortunately I was mixing up, how single-values are stored accessing them with pydicom and dcmjs (JS) (in dcmjs tag values are stored in a list, either one or multiple in a multivalue)
But I saw that you knew that and solved it in your commits - Thank you that you're still taking the time to update that!
from dicognito.
in dcmjs tag values are stored in a list, either one or multiple in a multivalue
I'm a little sad that pydicom didn't take a similar tack, but they probably had their reasons.
Cheerio.
from dicognito.
Related Issues (20)
- Fails to anonymize TimeOfLastCalibration
- Add option to write anonymized files to another directory HOT 2
- Fails to anonymize object with Issue Date of Imaging Service Request HOT 1
- Same patient names anonymize differently when formatted differently HOT 4
- Deflated files are corrupt when anonymized from the command line.
- Date/time offset is not always the same for a given seed
- Unable to anonymize dataset with encapsulated pixel data that contains embedded sequence delimiter HOT 2
- Anonymizing dataset with StationName but no Modality fails
- Add option to anonymize studies with StudyDateTime (and similar) set to now HOT 5
- Allow burned in annotations to cause a warning or failure to anonymize
- Support Python 3.10 HOT 1
- Fails to anonymize LEI file with Mitra global patient ID when using pydicom 2.2.x HOT 1
- How to map original to anonymized data in recursive mode? HOT 7
- Support fully-reproducible deidentification HOT 4
- Please fully document which attributes are removed/cleared/modified HOT 2
- Anonymizing according to DICOM Standard HOT 7
- Explicitly exclude DICOM tags from anonymization HOT 16
- Private creator 0031,0020 breaks anonymization
- Anonymization failure cites BytesIO object instead of input filename HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from dicognito.