-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
training coordinates fix #68
Conversation
for more information, see https://pre-commit.ci
If more than one dataset has been passed for validation, the prediction callback automatic inference uses the first one, we warn the user about this behavior.
for more information, see https://pre-commit.ci
compute_main_extension=True, | ||
) | ||
logger.info( | ||
f"Main extension found: {train_coordinates.main_file_extension}" | ||
) | ||
else: | ||
train_coordinates.train_bundle = load_bundle( | ||
coordinates_dict.get("train_dataset") | ||
coordinates_dict.get("train_dataset"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't this become dataset
(instead of train_dataset
) in the new version? should we address this now or later?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope this is only the training coordinates file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mmm I'd like to be consistent, is it possible or does it break something?
@@ -23,3 +25,22 @@ but you can [add custom formats](/docs/getting-started/customizing-things/custom | |||
If your dataset is already formatted like this, great! Otherwise, this is the only bit where coding is required. | |||
You can either convert it yourself (via a python or bash script, whatever you're comfortable with), or you can register | |||
a [custom data reader](/docs/getting-started/customizing-things/custom-data-format/) to support your dataset format. | |||
|
|||
|
|||
## Datasets Organization |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"organization" here does not translate from Italian :D I'd say Datasets Folder Structure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Organizing? Because it does not have to be a folder.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yup, Organizing Datasets sounds a lot better
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok
…to bug-fix/training-coordinates
No description provided.