You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
MRdataset version: 0.1.dev862+g33355da (installed via the docs instructions)
Python version: 3.11
Operating System: Linux
Description
Attempting to run the code shown in Figure 3 of the JOSS manuscript.
What I Did
# get an example DICOM from pydicomimportpathlibimportpydicomdemo_dcm=pathlib.Path(pydicom.data.get_testdata_file("MR_small.dcm")).parent# from Figure 3fromMRdatasetimportimport_datasetds=import_dataset(str(demo_dcm), format='dicom')
gives
dicom.py:MRdataset:__init__:80: Unable to read config file /home/damien/reviews/MRdataset/venv_repo/lib/python3.11/site-packages/MRdataset/resources/mri-config.json
Traceback (most recent call last):
File "/home/damien/reviews/MRdataset/f3_err.py", line 9, in <module>
ds = import_dataset(str(demo_dcm), format='dicom')
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/damien/reviews/MRdataset/venv_repo/lib/python3.11/site-packages/MRdataset/common.py", line 95, in import_dataset
dataset = dataset_class(
^^^^^^^^^^^^^^
File "/home/damien/reviews/MRdataset/venv_repo/lib/python3.11/site-packages/MRdataset/dicom.py", line 81, in __init__
raise e
File "/home/damien/reviews/MRdataset/venv_repo/lib/python3.11/site-packages/MRdataset/dicom.py", line 78, in __init__
self.config_dict = read_json(Path(self.config_path))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/damien/reviews/MRdataset/venv_repo/lib/python3.11/site-packages/MRdataset/utils.py", line 207, in read_json
raise FileNotFoundError(f'File not found: {filepath}')
FileNotFoundError: File not found: /home/damien/reviews/MRdataset/venv_repo/lib/python3.11/site-packages/MRdataset/resources/mri-config.json
The text was updated successfully, but these errors were encountered:
It seems that the resources folder was not included while publishing the library on PyPI. As of now you may try out the code in Figure 3 in the tutorial notebook.
sinhaharsh
added a commit
to sinhaharsh/MRdataset
that referenced
this issue
Feb 9, 2024
1) Modify working directory so that pytest uses installed version of the library (in site-packages), and not that of Github repository
2) Include tests in pyproject.toml
Related to Open-Minds-Lab#29
openjournals/joss-reviews#6269
Description
Attempting to run the code shown in Figure 3 of the JOSS manuscript.
What I Did
gives
The text was updated successfully, but these errors were encountered: