Skip to content
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

Bug in vision case and Inconsistency in scaling observations #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

saiprabhakar
Copy link

@saiprabhakar saiprabhakar commented Nov 7, 2016

Fixed:
Bug: Calling image observation from names.
Inconsistency: In scaling physical parameters for self.vision=True and False.
As suggested by @Guan-Horng Liu

@saiprabhakar saiprabhakar changed the title Inconsistency in scaling observations Bug in vision case and Inconsistency in scaling observations Nov 8, 2016
@Learningto
Copy link

Learningto commented Dec 14, 2016

Hi,I run the code,the car can run normally just in that little period .After that period ,The agent encounters a collision.The result video in your blog is so confusion! Can you show the whole video? @yanpanlau

@@ -272,14 +272,14 @@ def make_observaton(self, raw_obs):
Observation = col.namedtuple('Observaion', names)

# Get RGB from observation
image_rgb = self.obs_vision_to_image_rgb(raw_obs[names[8]])
image_rgb = self.obs_vision_to_image_rgb(raw_obs[names['img']])

Choose a reason for hiding this comment

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

please change to
image_rgb = self.obs_vision_to_image_rgb(raw_obs['img'])

Copy link

Choose a reason for hiding this comment

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

please change to
image_rgb = self.obs_vision_to_image_rgb(raw_obs['img'])

Hello, how do I get the current image of the agent as the input of the neural network?

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.

4 participants