search

duckietown / gym-duckietown

Self-driving car simulator for the Duckietown universe
http://duckietown.org
Other
52 stars 19 forks source link

Fullyobs nav #103

Closed bhairavmehta95 closed 5 years ago

bhairavmehta95 commented 5 years ago

Enables top down rendering only for the HUMAN with env.render(mode='top_down')

Addresses #101

bhairavmehta95 commented 5 years ago

This will need to be merged after #100.

@AndreaCensi this is the top down mode you were asking for by the way. I unfortunately forked this branch from the branch where I had all of the changes for dynamic duckiebots, but the two should be merged into master within a few hours / after some reviews.

AndreaCensi commented 5 years ago

@bhairavmehta95 thanks - I merged the other one.

AndreaCensi commented 5 years ago

@bhairavmehta95 I am unsure about the conflicts - can you take a look?

Also, is it possible to have both images generated? If so, we can produce the two videos at the same time.

bhairavmehta95 commented 5 years ago

Sure I'll fix the conflicts. And yeah I can do that.

@maximecb can you show me how you got your other simulator to do PiP (agent view and the bigger view?)

bhairavmehta95 commented 5 years ago

I'll merge this for now, leave the Picture-in-picture views for a different PR / set of issues, since it has to do with the overarching theme of "Rendering", so it might be time to rethink it altogether (i.e Hierarchy of renderers as previously suggested in #84)