openai / procgen

Procgen Benchmark: Procedurally-Generated Game-Like Gym-Environments
https://openai.com/blog/procgen-benchmark/
MIT License
1.01k stars 209 forks source link

Suggestion: Remove Space in between Folder Names #87

Open YukunJ opened 1 year ago

YukunJ commented 1 year ago

It caught my attention that when trying to use Procgen as a third party dependency, when building in a sandbox Bazel, the empty space in between Procgen Asset's folder cause some problems. It might be a super nice space left there.

For example,

/data/assets/kenney-abstract/Players)/[space folders] /data/assets/kenney/Players)/[space folders]

Would that be possible for maintainer to consider replace those space with _ as common practices? Not sure why those spaces are there at first place