1Euro7Cent / Mrballou-drawbot

This is a drawbot that can draw in almost every drawing programm (skribbl.io, gartic phone ect)
GNU General Public License v3.0
95 stars 11 forks source link

Program can't use configurations in config.json #2

Closed 4c30 closed 3 years ago

4c30 commented 3 years ago

i have an uncommon screen resolution so i had to create a new layout for drawing on paint i created a config named "paintduplicate" using initialisation.exe but when i insert the name in the gui, the command prompt says "invalid platform"

the message only doesn't show when i use exactly "skribbl", "gartic", and "paint", but those are probably default settings and they don't work well with my screen

edit: i forgot to mention that i'm using the new .exe release, i have no experience with the source files i'm using python 3.9.4

here are some screenshots init json gui prompt

1Euro7Cent commented 3 years ago

I think I forgot to add to the README that you need to restart the server (startServer.bat) to apply the changes. It only reeds the config in boot up. and BTW I forgot to remove my settings in the config. I just removed them in the executable.zip. That is the reason why it works with skribbl, gartic and paint. So please remove everything in the config.json file that does not belong to your settings

4c30 commented 3 years ago

sorry, i have tried running startServer.bat, run.exe, and repeating these steps after rebooting the laptop but the issue remains. i have even tried emptying the json file, and the program still runs and accepts only skribbl, gartic, paint. it seems like it uses your settings no matter what.

also gui.json shows what i last typed in the gui, but everytime i start it, the text field begins with 'skribbl' and all the settings reset

1Euro7Cent commented 3 years ago

First the gui.json is only for sharing the data to the node server. But I might add that the GUI uses the settings to insert them again. Second that problem with the settings not loading is weird. I mean (if you don't done that right) close this window image and reopen that. If that isn't working ill make a quick fix to load the config every time I/you hit draw (tomorrow. it is very late in my country RN). I think it is an error in the config.json and not the script itself.

It maybe can be that problem that the files are in the wrong place e.g.: the config.json file not in the server folder

4c30 commented 3 years ago

i have deleted the folder and redownloaded, then followed your instructions on restarting the program but the problem is still there i don't think it's a problem of the script not updating the config, because even after leaving the config empty with "{ }" and restarting the program, it still uses the default settings even if it has nowhere to get them from. maybe the settings are saved somewhere inside the script itself? i have no idea though

1Euro7Cent commented 3 years ago

I am currently adding a debug version. When this is done you can please run the script with the debug mode on. it will look like this image I will notify you when I released that mode. And I added that the GUI loads the last saved stats instead of the default values

1Euro7Cent commented 3 years ago

Done. Please redownload the thing and run the server in debug mode please and please send screenshots of that

4c30 commented 3 years ago

ok, i have tried it and it works!

i first started testing an empty config.json:

debugempty guiempty promptempty

then i created a profile named newpaint, and the debug shows it correctly debugpaint

here is the result next to the prompt after finishing drawing (yes, i have to fiddle with the settings but it works great!) promptpaint guess i have been rickrolled

thanks for the quick help, feel free to ask me questions or to close the issue!

1Euro7Cent commented 3 years ago

NP. ill, thank you for creating an issue and not just throwing it away. I will add a button to be able to set the values to default BTW.

And feel free to create more issues when you have a problem. And if you have something that I could add let me know