Skip to content

small fix - #37

Open
oriashush wants to merge 1 commit into
mainfrom
tutorial1_fix
Open

small fix#37
oriashush wants to merge 1 commit into
mainfrom
tutorial1_fix

Conversation

@oriashush

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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