MachineMuse / MachineMusePowersuits

Minecraft mod, take 2
236 stars 105 forks source link

fix "runClient" and set it's runtime ram to 3G. #879

Closed arnesacnussem closed 5 years ago

arnesacnussem commented 5 years ago

fix "runClient" and set it's runtime ram to 3G. except EnderIO...i really don't know how add those runtime dependency but you can add those two jar files to "run/mods" to make that mod works.

lehjr commented 5 years ago

There's a reason memory is set as high as it is.

arnesacnussem commented 5 years ago

@lehjr in fact , the default one is just not enough while run with EnderIO , 2G should work properly,but it constantly end with OOM. for me, i simply have that much ram...:joy:

lehjr commented 5 years ago

3GB is not enough when adding in all of the mods for testing.

arnesacnussem commented 5 years ago

maybe ,but......why not set that value yourself:fearful:

the default value is a little more than 1700M

lehjr commented 5 years ago

because I'm the one doing 99.9% of the work on the mod

lehjr commented 5 years ago

and when the server has a 5 minute load time, I can't be screwing around with out of memory errors

arnesacnussem commented 5 years ago

my poor English told me you have oom problem.... so now it solved :smile:.......?

lehjr commented 5 years ago

depends on what I'm testing. For larger test cases, I just build it and run it in MultiMC. I have one instance that I've had to give 9GB.

arnesacnussem commented 5 years ago

i only test codes with IDEA, now it's easier for me. (that's why i fix the gradle dependencies,also can use travis-ci to make nightly build)

lehjr commented 5 years ago

For small changes IDEA is enough, there are cases where more testing is needed, especially in the case of Galacticraft + addons, Advanced Rocketry, and a few others. Not to mention that some testing is needed outside of the development environment just to make sure things work outside of the development environment.

arnesacnussem commented 5 years ago

learn to microsoft:we only need issue tracker ...(and keep alive)

lehjr commented 5 years ago

Windows is a virus.

eyeonus commented 5 years ago

Agreed. A pernicious, invasive, and annoying one, at that.