Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Play visualization #317

Closed
wants to merge 6 commits into from
Closed

Play visualization #317

wants to merge 6 commits into from

Conversation

kimbring2
Copy link

@kimbring2 kimbring2 commented Apr 16, 2022

Hello,

I was inspired by the one of the NetHack projects of NeurIPS 2021.

They implemented the GUI to show the game state. I think it is helpful to play games as a human player.

I could add a code from that project to the NLE of this repo like the below image.

nle_visualization_v1 4

I will add more functions soon.

Thank you

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 16, 2022
@heiner
Copy link
Contributor

heiner commented Apr 21, 2022

Hey!

It might make sense to not include the glyph2tile array as data but use this instead: #288 (example usage).

I'll leave this to @condnsdmatters though.

@kimbring2
Copy link
Author

@heiner Thank you for your checking. I did not know the NLE already has the glyph2tile array. I will modify the code.

And I appreciate you forwarding my work to your colleague 💯

@cdmatters
Copy link
Contributor

Hi @kimbring2, thanks for your PR! Sorry, I've been out on leave the past week but I'm back today and will check this out in more detail this week. Thanks @heiner for flagging. :)

@cdmatters
Copy link
Contributor

Hey @kimbring2.

Thanks once again for your interest in NLE, and for this vizualiser that you've put together.

Looking at the PR, it seems to me that the main contribution of this PR is a modified play.py script, that effectively plays the game through your windowed display, with none of the code requiring any modification to the NLE environment, or the NetHack game.

While such a tool would undoubtedly be useful to some of the NLE community, I suggest it be open-sourced separately, perhaps hosted on your own GitHub account? We would be more than happy to point to such a tool on the NLE README.md (subject to us testing it first!), and that would also boost the discovery of the repo.

I think this is probably the best way for us to move forward, as it reduces code bloat for core NLE while also allowing you the flexibility to extend and expand the vizualiser as you see fit. It would also allow people with an interest in NLE to find your play script through this repo easily.

What do you think?

@kimbring2
Copy link
Author

kimbring2 commented Apr 28, 2022

@condnsdmatters Hello, Condnsdmatters. Thank you for your review.

It looks better idea to create a separate repository and refer to the README instead of adding it to the original NLE code directly like your opinion.

I host my GUI code to my repo. Please understand that the repo name is MMORPG because I am planning to extend the RL algorithm of NetHack to the more complex RPG game.

I think the NetHack is the best start environment for the MMORPG domain.

Thank you for making the awesome project.

Thank you again for your opinion 💯

@kimbring2 kimbring2 closed this Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants