KrisCris / Palworld-Pal-Editor

Palworld Pal Editor, the tool for Adding, Deleting, and Modifying Pals (and unlocking Viewing Cage for multiplayer) for Palworld Game Save, supports running on Docker, GUI, WebUI, and Cli.
https://www.nexusmods.com/palworld/mods/995
GNU General Public License v3.0
106 stars 11 forks source link

Not working in Linux dedicated headless environment #3

Closed sebagallo closed 4 months ago

sebagallo commented 4 months ago

Using docker-compose method in linux headless environment

palworld-editor  | Installing collected packages: darkdetect, customtkinter, palworld-pal-editor
palworld-editor  | Successfully installed customtkinter-5.2.2 darkdetect-0.8.0 palworld-pal-editor-0.2.1
palworld-editor  | Running command: python -i -m palworld_pal_editor --mode "web" --lang "en-US" --port "12345" --path "/mnt/gamesave" --password ""
palworld-editor  | 2024-03-04 14:42:11 [INFO] Logs written to /app/src/palworld_pal_editor/logs
**palworld-editor  | /app/src/palworld_pal_editor/cli.py:113: SyntaxWarning: invalid escape sequence '\g'**
palworld-editor  |   """
palworld-editor  | 2024-03-04 14:42:11 [ERROR] Exception caught on __main__: Object of type property is not JSON serializable
palworld-editor  | >>> 

I guess this happens because inside that logger.info method, the string contains at the last line \g after removing it, palworld-editor | /app/src/palworld_pal_editor/cli.py:113: SyntaxWarning: invalid escape sequence '\g' disappears.

BUT I still get the palworld-editor | 2024-03-04 14:42:11 [ERROR] Exception caught on main: Object of type property is not JSON serializable right after the info message.

KrisCris commented 4 months ago

Can you pull the latest changes and post the log?

sebagallo commented 4 months ago

While you were working on it, I managed to make it work now by myself. I think it was probably a lack of config.json and/or not passing the password env var? well, one of the two, cant be arsed to go back at testing this.

Also not passing the password actually passes it as 'null', and the frontend will require it.

KrisCris commented 4 months ago

Alright, thanks for letting me know! I think I've fixed it.