Skip to content

Conversation

@HelenaFP
Copy link

Sample output:

image

):
self._network_settings = network_settings

self._observation_size = observation_size
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this needed?


class ClassificationEnvironment(Environment):
"""

Copy link
Contributor

Choose a reason for hiding this comment

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

Fill this in

Copy link
Contributor

Choose a reason for hiding this comment

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

We should not need a separate script for classification. Why can't this be run through the existing main script?

Copy link
Contributor

Choose a reason for hiding this comment

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

New environment needs to be added to the docs

dataloader_train = DataLoader(self._dataset_train, batch_size=1, shuffle=True)
dataloader_test = DataLoader(self._dataset_test , batch_size=1, shuffle=False)
return dataloader_train, dataloader_test

Copy link
Contributor

Choose a reason for hiding this comment

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

Should raise for not implemented here

Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure this is necessary to include, would favor removing it

Co-authored-by: Jordan Ott <jordanott365@gmail.com>
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