MachineMusePowersuits icon indicating copy to clipboard operation
MachineMusePowersuits copied to clipboard

1.7.10 Improvements

Open kmecpp opened this issue 6 years ago • 37 comments

Multiple players have reported losing all energy in their power armor after teleporting

Also I don't think creative players should not lose energy or gain heat

I can submit a PR soon-ish if you're no longer supporting 1.7.10

kmecpp avatar Jan 28 '19 01:01 kmecpp

Heat and drain are left on for development reasons as these things are often debugged in creative mode. Since this version is no longer under active development, they can probably be disabled for creative mode. Teleportation drain is an old bug related to the sprint assist module and was fixed in 1.10.2. Backporting the fix should be pretty simple. I'll look at it tomorrow.

lehjr avatar Jan 28 '19 03:01 lehjr

Thanks. Might be best to have a config option for creative heat drain in the updated versions then

kmecpp avatar Jan 28 '19 04:01 kmecpp

Not really a priority at the moment since the only supported version is still alpha and under development.

lehjr avatar Jan 28 '19 15:01 lehjr

Fixing the power drain issue was relatively easy. The creative "bypass" is another matter due to the implementation of multiple power systems and multiple points are where power drain is called, all lacking a player parameter. Without a player parameter, there is no way to check if the player is in creative mode.

lehjr avatar Jan 28 '19 20:01 lehjr

ElectricItemUtils.drainPlayerEnergy is about the only place this can be implemented. Keep in mind this isn't likely to be 100% effective. The story for the heat system is similar. I've implemented a creative check on both of those where they can be applied, but again, it's not likely to be 100% reliable.

lehjr avatar Jan 28 '19 21:01 lehjr

Understood. I think anything to make it generally less annoying would be quite helpful.

Also thanks for fixing the teleporting issue! Will you post the changes soon?

kmecpp avatar Jan 30 '19 15:01 kmecpp

probably later today. I was looking into fixing a SmartMoving bug, but unfortunately the mod author's MinecraftForums.net account and forums posts fell victim to the purge where roughly 1 million users and 10 million posts got deleted due to the take over by Fandom. That alone wouldn't be an issue, but the only documentation and source for the mods were posted there.

lehjr avatar Jan 30 '19 15:01 lehjr

Have you tried the wayback machine?

http://web.archive.org/web/20181120050755/https://www.minecraftforum.net/forums/mapping-and-modding-java-edition/minecraft-mods/1274224-smart-moving

eyeonus avatar Jan 30 '19 21:01 eyeonus

Also, here's the DropBox for the last version, which has the source code inside the .zip in the "development" sub-folder.

https://www.dropbox.com/sh/w2qf6kej868412r/AAA5SuRdaftT2p0CPJ7R3xmLa?preview=MC+1.8.9+-+Smart+Moving+16.3.zip

And the Dropbox folder that has, apparently, every version of SmartMoving ever.

https://www.dropbox.com/sh/w2qf6kej868412r/AAA5SuRdaftT2p0CPJ7R3xmLa

eyeonus avatar Jan 30 '19 21:01 eyeonus

Thanks. Unfortunately, the whole thing is SmartMoving SmartRenderer RenderPlayerAPI PlayerAPI

For 1.12.2, I did fix the model, but untested due to coremod signing requirements and no compiled version being available.

lehjr avatar Jan 30 '19 21:01 lehjr

The wayback machine is very useful.

(SmartRenderer's source is in with the SmartMoving files, in the same "development" sub-folder.)

RenderPlayerAPI: http://web.archive.org/web/20180512055645/https://www.minecraftforum.net/forums/mapping-and-modding-java-edition/minecraft-mods/1283669-render-player-api

https://www.dropbox.com/sh/2nfhgvo4nh037xp/AADeghcvD5uON-c-uFuGUL6Ja

PlayerAPI: http://web.archive.org/web/20180927134817/https://www.minecraftforum.net/forums/mapping-and-modding-java-edition/minecraft-mods/1277996-player-api

https://www.dropbox.com/sh/wfd5j8p3hp6rhmd/AACRgjxNT176S1cma_MijD_Fa

eyeonus avatar Jan 30 '19 21:01 eyeonus

Thanks. I'm not entirely sure yet, but I'm relatively certain the issue is in one of the 4 and not in MPS. So I could probably release what I have and if I find a fix, release that separately, though there isn't any official channels for that

lehjr avatar Jan 30 '19 22:01 lehjr

Divisor does still have a CurseForge account, so maybe a pm sent via that?

https://minecraft.curseforge.com/members/Divisor

eyeonus avatar Jan 30 '19 22:01 eyeonus

Yeah, forgot about that. That might work. There seems to be some recent activity on that account too.

lehjr avatar Jan 30 '19 22:01 lehjr

It's too bad Divisor never GitHub'd his stuff....

eyeonus avatar Jan 30 '19 22:01 eyeonus

I know, it would have probably gotten more fixes for issues.

lehjr avatar Jan 30 '19 22:01 lehjr

I've half a mind to take his stuff and GitHub it myself, but I have no desire to work on it, so that'd be a bit pointless. I mean, I could give contributor access to other people, but then again, other people could just make their own fork....

eyeonus avatar Jan 30 '19 22:01 eyeonus

It's one of those things where it's not really worth the time unless you're both willing and able to support it. I'm going to try to fix this one issue, but I want no part in maintaining that project.

lehjr avatar Jan 30 '19 23:01 lehjr

Fixing the power drain issue was relatively easy.

Was the fix for this ever commited?

kmecpp avatar Aug 20 '19 00:08 kmecpp

Fixing the power drain issue was relatively easy.

Was the fix for this ever commited? Should have been, but who knows, Either way, 1.17.10 is years old and I'm not working on it. I already have my hands full with 1.14.4.

lehjr avatar Aug 20 '19 01:08 lehjr

Could you just check your 1.7.10 branch and see if you have the uncommited patch sitting there?

That comment where you said you fixed it was from Jan 27 and the latest commit to the branch is Jan 17

kmecpp avatar Aug 20 '19 01:08 kmecpp

Whether it is or isn't won't change whether or not I work on it or not. If it didn't get committed then the code is gone.

lehjr avatar Aug 20 '19 01:08 lehjr

Looks like you're in luck after all since it was part of the backup I made before wiping my primary OS.

lehjr avatar Aug 20 '19 01:08 lehjr

Awesome, thank you!

kmecpp avatar Aug 20 '19 01:08 kmecpp

And that's the end of 1.7.10 support from me.

lehjr avatar Aug 20 '19 01:08 lehjr

Im sorry if this sounds dumb, but was it ever released?

MalTeeez avatar Nov 25 '19 08:11 MalTeeez

I noticed the commit in 1.7.10 but saw a direct release - Im not good in building it myself

MalTeeez avatar Nov 25 '19 09:11 MalTeeez

was probably released months ago, but keep in mind that 1.17.10 is not the only version of the mod, so I can't keep track of them all.

lehjr avatar Nov 25 '19 13:11 lehjr

Where could it have been released, I looked on both Curse Pages and the release Section - Is there something else?

MalTeeez avatar Nov 25 '19 14:11 MalTeeez

No that's it. Unfortunately, setting up a 1.7.10 build environment is not trivial, especially for this mod. And I simply do not have the time at the moment. This is a team of 1, and I'm already working on 1.14.4 and 1.12.2. I cannot split my time any further than that.

lehjr avatar Nov 25 '19 14:11 lehjr