openworm / OpenWorm

Repository for the main Dockerfile with the OpenWorm software stack and project-wide issues
http://openworm.org
MIT License
2.54k stars 200 forks source link

Build.cmd & Run.cmd referencing wrong version? #345

Open Daasin opened 1 month ago

Daasin commented 1 month ago

Discussed in https://github.com/openworm/OpenWorm/discussions/344

Originally posted by **Daasin** May 3, 2024 and is there a way to get 0.9.4? Opening the files it seems to reference 0.9.4 but says 0.9.3 in windows terminal after building or running

Turns out build.cmd and run.cmd both referenced the older version

pgleeson commented 1 month ago

Hi @Daasin. Thanks for pointing this out. This is hopefully fixed in the pull requests https://github.com/openworm/OpenWorm/pull/339 and #346. Unfortunatley I've no way to test this on Windows, so please pull the latest from this repo and test it out.

Daasin commented 1 month ago

I managed to edit them manually with notepad and got it to download, unfortunately after 3x24 hour runs (accidents in the first two and last one failed to find output in ow sibernetics folder, not the first time I've had that issue however, I'll replicate from Linux another time) I've pretty much become exhausted by it atm 😅

But will check the source to see if it matches :)

pgleeson commented 1 month ago

Thanks for the update. It's all not very well tested on Windows unfortunately, simply because most of us work on Linux/Mac, but if you do have suggestions for improvments we're happy to incorporate them.

Daasin commented 1 month ago

Thanks for the update. It's all not very well tested on Windows unfortunately, simply because most of us work on Linux/Mac, but if you do have suggestions for improvments we're happy to incorporate them.

@pgleeson Does it run on Docker Engine in Linux?

And for Windows, Would love a way to utilise my GPU to accelerate it without completely passing through my GPU to the VM. Would make testing a lot faster, no? Docker Desktop seems to only have guides for NVIDIA GPU's which I found strange considering they're seen as more difficult to work with on Linux.

Daasin commented 1 month ago

Thanks for the update. It's all not very well tested on Windows unfortunately, simply because most of us work on Linux/Mac, but if you do have suggestions for improvments we're happy to incorporate them.

Also just realised the issue I faced is mirrored on #336 except it applies to 0.9.4 aswell