mam91 / neat-genetic-mario

Update of Seth Bling's MarI/O
182 stars 62 forks source link

Fixes allowing it to function in the latest version of Bizhawk #22

Open ColtBry opened 1 year ago

ColtBry commented 1 year ago

Fixes a memory reading issue in getTile causing noise to be returned. Fixes pool loading that was caused by a change in lua engines in BizHawk version 2.1.0

kylechallis commented 1 year ago

I've opened a PR to help address my above suggested changes https://github.com/ColtBry/neat-genetic-mario/pull/1

ColtBry commented 1 year ago

Awesome nice catch, I forgot I was using a different state file than the default.

I'm also currently redoing the GUI drawing, and playing around with the ability to easily add new nodes to the network. Allowing for the network to have other game inputs (ex. powerup state). I'll probably get a chance to finish them up sometime in the next few days.

BrunoCollazo commented 9 months ago

Using this works all good, except that I have no GUI, any way to enable it?

ColtBry commented 9 months ago

sing this works all good, except that I have no GUI, any way to enable it?

The GUI overlay is currently disabled other than the circle. There isn't currently an option to toggle it in the config file, but if you go into mario-neat.lua it's just commented out. Look for all the spots there is "gui.drawText(), gui.drawBox(), and gui.drawLine() and uncomment those lines.

ColtBry commented 9 months ago

Using this works all good, except that I have no GUI, any way to enable it?

Oh i miss understood you. I assume you're talking about the training GUI that has the start and stop button. Are you getting an error in the script terminal? Ensure that you've created a saveState and name it "DonutPlains1.State" and save it in the neat-mario/state folder replacing the one currently in there.

BrunoCollazo commented 9 months ago

sing this works all good, except that I have no GUI, any way to enable it?

The GUI overlay is currently disabled other than the circle. There isn't currently an option to toggle it in the config file, but if you go into mario-neat.lua it's just commented out. Look for all the spots there is "gui.drawText(), gui.drawBox(), and gui.drawLine() and uncomment those lines.

Talking about that, didn't read that GUI was changed to the form and assumed it was your PR. Sorry and thanks

BrunoCollazo commented 9 months ago

So, now I have a problem loading a pool. NLua.Exceptions.LuaScriptException: [string "main"]:912: attempt to index local 'file' (a nil value) NLua.Exceptions.LuaScriptException: [string "main"]:1047: attempt to index local 'species' (a nil value)

And i see you removed the requirement to set a directory path, could this have something to do with it?