Skip to content

small fix#37

Open
oriashush wants to merge 1 commit intomainfrom
tutorial1_fix
Open

small fix#37
oriashush wants to merge 1 commit intomainfrom
tutorial1_fix

Conversation

@oriashush
Copy link
Contributor

No description provided.

@oriashush oriashush self-assigned this Jul 25, 2024
@oriashush oriashush requested a review from yuvalrhino July 25, 2024 08:05
@oriashush oriashush assigned yuvalrhino and unassigned oriashush Jul 25, 2024
def dataset_dcm_to_jpg(dataset_df):
input_dir = '/input/dicom_data/'
output_dir = '/output/file_data/'
output_dir = '/output/0/file_data/'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that this is required - users can still write to /output/file_data if they have a single output dataset.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The error occurred while removing it:

Failed to import output datasets.Dataset 'Site 1 Dataset (v=3)': Failed to import output datasets. Error in prevalidation for the following fields:
JPG file: Field of type Filename requires file base path to be provided - please re-import and specify a file base path.

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