controller is not work on windows

diambra / arena

DIAMBRA Arena: a New Reinforcement Learning Platform for Research and Experimentation

https://docs.diambra.ai

Other

315 stars 22 forks source link

controller is not work on windows #115

Open Yimi81 opened 3 months ago

Yimi81 commented 3 months ago

The input library for obtaining devices is outdated, making it impossible to obtain device information correctly under Windows. https://github.com/diambra/arena/blob/505328f4faa9a2807e56dd34b1d38e81d267a5cb/diambra/arena/utils/controller.py#L5

alexpalms commented 3 months ago

Hello @Yimi81,

can you please share additional details about:

The error you are receiving
The steps to replicate it
The environment you are using (OS, etc)

Yimi81 commented 3 months ago

python==3.9.18 diambra-arena==2.2.6

Run controller. py and enter 4 to trigger the following error:
Annotate self. logger. basicConfig (logging_level) and run controller. py again
At first, I encountered a problem in llm-coloseum, which was stuck in the following interface and could not respond to input. Therefore, I determined that the inputs library did not correctly retrieve the input device in Windows

I'm stuck on this interface. I pressed anything and there was no response

alexpalms commented 3 months ago

@Yimi81 The logging problem should have been solved by this commit: https://github.com/diambra/arena/commit/3d052667b41a966d755f5f611a6b67bf19006fd4

Can you update diambra-arena package via pip install -U diambra-arena, make sure you get 2.2.7 version and try again?

Yimi81 commented 3 months ago

I upgraded to diambra arena, but the problem remains the same. I am unable to configure the device both stuck

Yimi81 commented 3 months ago

press ctrl c, get:

alexpalms commented 3 months ago

@Yimi81 Thanks for the followup.

Two things: 1) Can you share the content of the python script you are executing? temp.py 2) This seems to be related with inputs library itself, which might be blocked by some security setup on your machine, as explained here: https://inputs.readthedocs.io/en/latest/user/install.html#windows-permissions

Yimi81 commented 3 months ago

temp.py is copy from controller.py(https://github.com/diambra/arena/blob/main/diambra/arena/utils/controller.py). Thanks, i will try it

alexpalms commented 3 months ago

Let us know if you manage to solve the problem, so far it has always worked fine, and it is very probably is a security problem, as the connection is hanging.