Comments (16)
@jcohenadad sorry, I have someone testing this now. Thanks for your patience.
from dcm2bids.
After more investigation, we've noticed that the issue may not be present in SIEMENS but is present for GE and PHILIPS data.
from dcm2bids.
This should be corrected in 2.0.0 but I don't have PHILIPS data to test it
from dcm2bids.
@dasturge can you test it again, I've got some PHILIPS data as well but was not able to replicate your issue.
from dcm2bids.
@arnaudbore I should be able to give it a go in the next week or so, I'll post back with the results.
from dcm2bids.
@dasturge do you have any feedback? Can the issue be closed?
from dcm2bids.
@jcohenadad so unfortunately this doesn't seem to fix our issue.
However, I believe the problem is occurring due to some software mucking up the dicom metadata. I believe the code which does this was acquired directly from PHILIPS, but I'm checking with our collaborators to confirm.
from dcm2bids.
Can you make some dcm from Philips available so we can reproduce and fix it quickly (if possible) ?
from dcm2bids.
@arnaudbore if you sign a Data Use Certification for the ABCD study, we can provide you with a subject's dicoms where this error occurs: https://nda.nih.gov/abcd/request-access
I have confirmed that the GE data where we had the issue was due to a GE script which padded the series number, the PHILIPS data is apparently as-is.
from dcm2bids.
Not sure if someone would like to improve dcm2bids for GE scanner but I can confirm that for GE output, the issue is still there.
To be more precise, SeriesNumber = [3, 4, 11, 6, 5, 10] corresponding to the output of dcm2bids RunNumber = [1, 2, 3, 4, 5, 6]. In this example, SeriesNumber 1 and 2 are omitted, they were loc and anat images. I have not figured out the logic there but it seems the padding issue.
Any suggestions? I'd be happy to provide the data if needed.
from dcm2bids.
@CerenB if you can send me some data I will be happy to fix this issue.
from dcm2bids.
@arnaudbore thank you! it might be due to GE producing extra files and they are messing up the order, still not 100% sure. How can I share the data with you?
from dcm2bids.
@CerenB you can use https://wetransfer.com/.
from dcm2bids.
@CerenB can you test with this PR #68 ? I found something that may be the issue you were talking about.
from dcm2bids.
@CerenB can you test with this PR #68 ? I found something that may be the issue you were talking about.
thanks! it's true that my config.json has upper and lower cases but it's identical to the how Series/Protocol names are called in sidecar. But I am using the latest version of dcm2bids, that should have been resolved in this version?
from dcm2bids.
That PR #68 solved my issue with GE scanner run number organisation. Thank you!
from dcm2bids.
Related Issues (20)
- Should this line be pointing to one of the sidecar files? HOT 2
- Typos in some files of the scaffold raise errors on the validator
- Creating several 3D images instead of a single 4D image. HOT 2
- New features or enh for next release
- post_op HOT 6
- Provide a Docker image that includes pydeface HOT 4
- [BUG] dcmbids binary
- Processing order issue ? ```"IntendedFor" No id found for 'IntendedFor' value 'XX'``` HOT 4
- KeyError: 'datatype' HOT 1
- TypeError in Acquisition.init() when running dcm2bids HOT 1
- `sidecar_changes` do not fetch all matches of the referenced description
- Error: Too many NIFTI images with the name HOT 1
- compKeys default for run numbering doesn't seem like a good one HOT 1
- Subprocess Error Assistance HOT 3
- Allowing for post_op command's output to be printed to stdout HOT 2
- Script removes task information from filenames. HOT 8
- bids validator not working properly HOT 8
- Fix GHA
- Document new (and old) features
- Maybe a tab error 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 dcm2bids.