Ladysnake / Effective

A Minecraft Quilt client-side mod adding ambient and environmental interaction effects to enhance immersion.
Other
203 stars 66 forks source link

Update 1.19.4 #199

Closed SzczurekYT closed 9 months ago

SzczurekYT commented 1 year ago

Hello I'm back, with an update to 1.19.4 for Effective 2.x All the changes I made are grouped into few big commits that have a description explaining what and why it was changed. (Except from few small ones where I missed something). I recommend taking a look a them and reading the descriptions.

I tested everything from the new showcase video and made sure it works. There is only one thing that is broken - ghast fireball ultrakill, the white particles don't show up, the screen only freezes for these few ticks. I have no idea how to fix, but it's probably something related to changes made to LodestoneLib in 1.19.3/4 version. (The 1.19.3 branch supports 1.19.4, and arathain said he will be updating it to 1.20 soon). Maybe you will know? You probably should take look at ScreenParticleParryRenderEnforcer, as the class it targets got a rewrite and the mixin it's probably not correct now.

MasterMM007 commented 1 year ago

This doesn't work with Fabric right?

SzczurekYT commented 1 year ago

This doesn't work with Fabric right?

No, it's just an update to 1.19.4, it's still on quilt. It's a bit funny that now most of the mods is developed for fabric, but still most people run quilt because of the few mods that use it.

a-fan9823 commented 1 year ago

how do you package this?

Kavin7000 commented 1 year ago

Are you planning on updating to 1.20.1?

SzczurekYT commented 1 year ago

Currently probably not, as I don't think I will be updating to 1.20 soon, so I don't have any motivation.

KiloPetraGames commented 1 year ago

Greetings! Is there any way that this can be modified to work in 1.19.3?

ViTulyankin commented 1 year ago

@SzczurekYT, is there any way to build mod by myself? (Dejavu?) Or maybe you can make a release? :)

SzczurekYT commented 1 year ago

@SzczurekYT, is there any way to build mod by myself? (Dejavu?) Or maybe you can make a release? :)

Since everyone is asking, here is a build. effective-1.7+1.19.4.zip You need to rename it from .zip to .jar (because github doesn't allow uploading jars in comments). @a-person1908 you have been waiting for this too.

SzczurekYT commented 1 year ago

Greetings! Is there any way that this can be modified to work in 1.19.3?

@KiloPetraGames You can try, it may work without any changes, but I don't guarantee that.

ViTulyankin commented 1 year ago

Since everyone is asking, here is a build. effective-1.7+1.19.4.zip

But... Isn't it 1.7 version of this mod?

SzczurekYT commented 1 year ago

Since everyone is asking, here is a build. effective-1.7+1.19.4.zip

But... Isn't it 1.7 version of this mod?

Uh.. This feels so wrong. I think it's just a wrong number, it might get lost somewhere during porting, but I need to make sure.

KiloPetraGames commented 1 year ago

Greetings! Is there any way that this can be modified to work in 1.19.3?

@KiloPetraGames You can try, it may work without any changes, but I don't guarantee that.

Dang, doesn't seem to work. Is there a tutorial somewhere on how to build this? Where I can then swap out the game/dependency versions?

SzczurekYT commented 1 year ago

Greetings! Is there any way that this can be modified to work in 1.19.3?

@KiloPetraGames You can try, it may work without any changes, but I don't guarantee that.

Dang, doesn't seem to work. Is there a tutorial somewhere on how to build this? Where I can then swap out the game/dependency versions?

Open an issue on my repo and send the error, I can try to help you there, so we don't clutter this PR.

doctor4t commented 1 year ago

I'll test this and merge it into a new 1.19.4 if it's good. Thanks for the contribution!

SzczurekYT commented 1 year ago

I'll test this and merge it into a new 1.19.4 if it's good. Thanks for the contribution!

Great, most of the things should work. Some dependency stuff changed recently, so make sure to pull the commit I added now. I also can't build the project, as an error shows up Failed to transform mod metadata file, but might be just me, as gradle sometimes can get a bit tricky, and It worked before.

KiloPetraGames commented 1 year ago

Greetings! Is there any way that this can be modified to work in 1.19.3?

@KiloPetraGames You can try, it may work without any changes, but I don't guarantee that.

Dang, doesn't seem to work. Is there a tutorial somewhere on how to build this? Where I can then swap out the game/dependency versions?

Open an issue on my repo and send the error, I can try to help you there, so we don't clutter this PR.

Apologies! I can't see an issues tab on your Repo, is there somewhere else we could communicate?

SzczurekYT commented 1 year ago

Greetings! Is there any way that this can be modified to work in 1.19.3?

@KiloPetraGames You can try, it may work without any changes, but I don't guarantee that.

Dang, doesn't seem to work. Is there a tutorial somewhere on how to build this? Where I can then swap out the game/dependency versions?

Open an issue on my repo and send the error, I can try to help you there, so we don't clutter this PR.

Apologies! I can't see an issues tab on your Repo, is there somewhere else we could communicate?

Oops, I forgot it's disabled for forks by default, try now.

Spike-has-no-idea-how-to-code commented 1 year ago

Hey, I can’t build this either ( although I am completely unfamiliar on everything about this platform,Git, and Gradle) I learned how to do a build from a comment on a pull request for the Auditory mod on here, and I would like to use the updated 2.1 for 1.19.4, but I’m stuck with your only release for 1.7 because it fails when I try to build it. Maybe someone could tell me what I’m supposed to do? Maybe what worked for auditory won’t work with this, and that’s what’s stopping me. Anyway, please help but no rush

SzczurekYT commented 1 year ago

Hey, I can’t build this either ( although I am completely unfamiliar on everything about this platform,Git, and Gradle) I learned how to do a build from a comment on a pull request for the Auditory mod on here, and I would like to use the updated 2.1 for 1.19.4, but I’m stuck with your only release for 1.7 because it fails when I try to build it. Maybe someone could tell me what I’m supposed to do? Maybe what worked for auditory won’t work with this, and that’s what’s stopping me. Anyway, please help but no rush

Looks like the build error fixed itself, I saw gradle downloading new quilt loom versions, so probably quilt loom had a bug.

Treetrain1 commented 1 year ago

Looks like the build error fixed itself, I saw gradle downloading new quilt loom versions, so probably quilt loom had a bug.

Yes, Quilt Loom 1.3.2 fixed the Failed to transform mod metadata file error it was due to Loom only ever checking for a QMJ5 (which isnt fully implemented) or an FMJ

SzczurekYT commented 1 year ago

Looks like the build error fixed itself, I saw gradle downloading new quilt loom versions, so probably quilt loom had a bug.

Yes, Quilt Loom 1.3.2 fixed the Failed to transform mod metadata file error it was due to Loom only ever checking for a QMJ5 (which isnt fully implemented) or an FMJ

Thanks for the details.

doctor4t commented 1 year ago

Been a while since I looked at this. Are the issues you mentionned originally fixed?

SzczurekYT commented 1 year ago

Been a while since I looked at this. Are the issues you mentionned originally fixed?

I was absent too. The only issue that I mentioned, is that there is no white flash on ghast ultrakill. It still isn't fixed, and I don't know how to fix that, so you have to do that yourself.

ViTulyankin commented 1 year ago

I was absent too. The only issue that I mentioned, is that there is no white flash on ghast ultrakill. It still isn't fixed, and I don't know how to fix that, so you have to do that yourself.

I've tested 1.20 version - water splashes are still pretty broken https://github.com/Ladysnake/Effective/pull/181#issuecomment-1589319597

SzczurekYT commented 1 year ago

I was absent too. The only issue that I mentioned, is that there is no white flash on ghast ultrakill. It still isn't fixed, and I don't know how to fix that, so you have to do that yourself.

I've tested 1.20 version - water splashes are still pretty broken #181 (comment)

I see, yes, that's still an issue. I'll try to fix it soon.

doctor4t commented 1 year ago

Personally still can't build this, as it seems to be failing to find the Lodestone commit you put as the version.

SzczurekYT commented 1 year ago

Personally still can't build this, as it seems to be failing to find the Lodestone commit you put as the version.

To test this I did a fresh clone of the 2.x branch with git clone --single-branch -b 2.x-1.19.4 https://github.com/SzczurekYT/Effective and it built successfully without any issues. Maybe some cache issues?

SzczurekYT commented 1 year ago

Personally still can't build this, as it seems to be failing to find the Lodestone commit you put as the version.

Oh wait, I yeeted my gradle cache and now I have this issue too.

SzczurekYT commented 1 year ago

Looks like the build on jitpack failed https://jitpack.io/com/github/Arathain/LodestoneLib-Quilt/70a03b920e/build.log

doctor4t commented 1 year ago

I love jitpack, I need to get arathain to start publishing Lodestone Quilt on the Ladysnake maven

SzczurekYT commented 9 months ago

Can you please make your next minecraft video about kidnapping Arathain until he publishes his libs to some kind of maven repo? Recently I wanted to go back to this PR and finish off it, but broken Jitpack quickly made sure I leave the thing again.

Pyrofab commented 9 months ago

Can relate, ended up publishing the latest version to our repo myself.

I have merged this PR as part of the 1.20.1 update, huge thanks for this.

Pyrofab commented 9 months ago

Also I haven't worked on the mod enough to know how to fix the issue you mentioned (if it's still there), so if you have an idea a PR would be very appreciated =)