Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v0.3.0 #7

Merged
merged 19 commits into from
Jul 14, 2024
Merged

v0.3.0 #7

merged 19 commits into from
Jul 14, 2024

Conversation

AViewFromTheTop
Copy link
Contributor

@AViewFromTheTop AViewFromTheTop commented Oct 3, 2023

If anything else is needed or if you want me to try adding anything else, just let me know
I'm open to this PR being edited as well, it's just a basis for compat with Wilder Wild and Particle Tweaks

CURRENT CHANGES (edited by @Tschipcraft):

  • Support for Minecraft 1.21 ⚔️ (closes #13, #14, #15)
  • Added config file and settings menu when installed alongside MidnightLib (closes #4)
  • Added entity interaction physics to bubbles (configurable)
  • Added biome dependent tint to all bubble types (configurable)
  • Fixed barrels opened by other players on servers not emitting bubbles (closes #11) (@AViewFromTheTop)
  • Adjusted current down bubbles to pop on top of magma blocks rather than inside them (closes #9)
  • Added varied pitch and volume to bubble pop and bubble swirl sounds (closes #8) (@AViewFromTheTop)
  • Barrel bubbling is now reliable on the client and will no longer bubble when a chunk with an already opened Barrel is loaded (@AViewFromTheTop)
  • Fixed checking incorrect side of barrels for bubble particles
  • Changed the default behavior to not maintain bubble particle velocity when popping (configurable) (@AViewFromTheTop)
  • Tweaked explosion bubbles and used better variable names
  • Restructured bubble particle logic and used better variable types
  • Adjusted barrel bubble spawn locations
  • Moved bubble pop logic to a separate utility class to reduce code duplication
  • Simplified water checks to use the already provided helper function in world or level
  • Fixed literal new line in fabric.mod.json after building (closes #12)

@AViewFromTheTop
Copy link
Contributor Author

Just letting you know btw, Math.cos and Math.sin are ever so slightly faster than their MathHelper variants, even when they're casted to another variable
In this case the regular Math.cos/sin methods wouldn't even need to be casted so I just swapped them out

@Tschipcraft
Copy link
Owner

Hello, thank your for actually contributing to Make Bubbles Pop!
For me university just started so I sadly have little to no free time available to check the pull request and make a new release at least the next few days

@Tschipcraft
Copy link
Owner

Just letting you know btw, Math.cos and Math.sin are ever so slightly faster than their MathHelper variants, even when they're casted to another variable In this case the regular Math.cos/sin methods wouldn't even need to be casted so I just swapped them out

Thanks, I didn't know that

@AViewFromTheTop
Copy link
Contributor Author

Yeah same here college kinda sucks

@Tschipcraft Tschipcraft self-assigned this Oct 8, 2023
@Tschipcraft Tschipcraft added the enhancement Changes for an existing features label Oct 8, 2023
@Tschipcraft Tschipcraft linked an issue Oct 8, 2023 that may be closed by this pull request
@AViewFromTheTop
Copy link
Contributor Author

AViewFromTheTop commented Oct 29, 2023

actually im gonna luna-fy the mod lol (or aviewfromthetop-ify)

@AViewFromTheTop AViewFromTheTop changed the title MBP Compat Booleans Random Changes I Like Oct 29, 2023
@AViewFromTheTop AViewFromTheTop changed the title Random Changes I Like Small PR Oct 29, 2023
@Tschipcraft
Copy link
Owner

ohh, nice!
I'm just going to use this pr as a base and add my own commits to it

@Tschipcraft
Copy link
Owner

Actually, I like the pop bubbles not retaining their velocity more as well... maybe I'll just ditch the option
Any idea how I could implement the custom network packet on Forge?

- Fixed maintain velocity option not having an effect
- Made BarrelEntityOnUse an abstract class
- Added OnlyIn annotation in BarrelBubbler on Forge
- Added comments
@AViewFromTheTop
Copy link
Contributor Author

Actually, I like the pop bubbles not retaining their velocity more as well... maybe I'll just ditch the option
Any idea how I could implement the custom network packet on Forge?

I unfortunately know literally nothing about Forge other than how to register some things lol

@Tschipcraft
Copy link
Owner

Damn, rip

- Tweaked explosion bubbles and used better variable names
- Updated GitHub workflow
- Updated CurseForge badge in README
- Added CurseForge direct messages as a way of contacting me in the LICENSE
- Added a config file and menu when installed alongside MidnightLib (closes Tschipcraft#4)
- Restructured bubble particle logic and used better variable types
- Added experimental multiplayer support for barrel bubbles (even on Forge - Tschipcraft#11)
- Adjusted barrel bubble spawn locations
- Updated GitHub workflow to upload build artifacts
- Adjusted current down bubbles to pop on the top of magma blocks rather than inside them (closes Tschipcraft#9)
- Fixed barrel bubbles not appearing outside the dev environment on forge (Tschipcraft#11)
- Added alt texts to images in README
- Updated GitHub workflow to use setup-gradle caching instead
@Tschipcraft Tschipcraft changed the title Small PR v0.3.0 Feb 18, 2024
@AViewFromTheTop
Copy link
Contributor Author

AViewFromTheTop commented Feb 24, 2024

@Tschipcraft is it ok if I change the mod version to 1.20.1 and add FrozenLib+Cloth Config config compat to this?
FrozenLib allows for config syncing between client and server (modifiable with annotations) but we haven't released a version with that below 1.20.1.

Edit: I just remembered I only know how to do Fabric stuff

@AViewFromTheTop
Copy link
Contributor Author

Why though? If the mod checks for FLib and e.g. a Minecraft version above or equal 1.20.1 before initializing the config class for FLib (and therefore call the library) it should work fine? If it doesn't call the problematic code, it doesn't crash, unless FLib itself calls some parts of it somehow even when it's not registered

That should work, I kinda forgot you could call for the MC version-

@AViewFromTheTop
Copy link
Contributor Author

Wilder Wild has chest bubbling? Damn rip my features 🫠

I found out about Make Bubbles Pop a couple months after I added it lmao
it doesn't have all the other bubble stuff though

@Tschipcraft
Copy link
Owner

I found out about Make Bubbles Pop a couple months after I added it lmao it doesn't have all the other bubble stuff though

Ah well, it's a pretty common idea, Effective also added chest bubbling after myself a while ago

@Tschipcraft
Copy link
Owner

Btw, any particular reasons why you switched from storing block positions to block entities in the array list in the barrel bubbles, and why from an override to an inject that cancels the method in BubblePop, wouldn't a redirect be better if overrides should be avoided?

@AViewFromTheTop
Copy link
Contributor Author

Btw, any particular reasons why you switched from storing block positions to block entities in the array list in the barrel bubbles, and why from an override to an inject that cancels the method in BubblePop, wouldn't a redirect be better if overrides should be avoided?

It prevents block entities from other dimensions potentially bubbling (though im not 100% but it’s just to be safe)
As for the redirect, that can cause mod compat issues
Simply injecting and canceling does the same but saves you from dealing with potential issues with other mods

- Added biome dependent tint to all bubble types, including config entries to control intensity and enablement of this feature
- Moved bubble pop logic to a global utility class to reduce code duplication
- Added varied bubble pop volume (closes Tschipcraft#8)
- Reordered and reformatted config entries
- Added tooltips when hovering over config entries
- Added option to disable bubble swirl sound playing when opening chests or barrels underwater
- Updated GitHub workflow
@Tschipcraft
Copy link
Owner

I tested this out a bit, saving only the BlockPos does prevent containers in other dimensions in the same location to bubble, at least from my testing. I'm gonna change this back to be consistent with the Forge version, solely because saving the BlockPos there is easier lol

- Fixed some players affecting bubbles when returning a high upwards y velocity despite standing still (They return the constant y velocity of 0.419875. This seems to happen to players if they have been flying in creative mode before while loaded on the client)
- Reverted to saving only block positions for barrel bubbles on Fabric for consistency with the Forge version
- Fixed literal new line in fabric.mod.json after building (closes Tschipcraft#12)
- Fixed opened double chests playing bubble swirl sound even if it's turned off in the config
- Simplified water checks to use the already provided helper function in world or level
- Added varied volume and pitch to bubble swirl sound when opening containers
- Reworded some config entries and log output
- Updated to Gradle 8.7, Loom 1.6 and ForgeGradle 6.+
- Updated mappings
- Increased pack format to 41 for mc1.20.5
@R00tB33rMan
Copy link

R00tB33rMan commented Jun 5, 2024

This weirdly fixes 1.20.5/6 support. Cannot identify any bugs since the latest changes you've made 3 months ago with a performance-based modpack.

@Tschipcraft
Copy link
Owner

I mean yeah, v0.3.0 is basically done. I just need to backport it to pre 1.19.4 versions still. I'll have time in approx. 1 month when the semester ends

@ViTulyankin
Copy link

Works fine even in 1.21 version

@AViewFromTheTop
Copy link
Contributor Author

My apologies for not contributing for a while, life got in the way and I’ve been working on lots of bugfixes for Wilder Wild.
Once I get Particle Tweaks on 1.21 I’ll try to see what I can do here, if anything
It just kinda sucks that I don’t know forge

@Tschipcraft
Copy link
Owner

Ya, no worries. I know the feeling. I want to push out v0.3.0 in the next few weeks before any new features, since it's been almost a year since the last update, but after that I'd still be interested in the FLib integration (and/or any other additions/fixes). Oh and again thanks for contributing in the first place <3

@Tschipcraft
Copy link
Owner

Tschipcraft commented Jul 7, 2024

Works fine even in 1.21 version

Forge version sadly doesn't work in 1.21 yet

Edit: Works now

Tschipcraft added a commit that referenced this pull request Jul 11, 2024
- Added new forge-1.21 branch for Minecraft 1.20.6-1.21x forge builds
- Fixed forge build script not working in Minecraft 1.20.6+ (closes #13, #14, #15, progress for finalizing v0.3.0 #7)
Tschipcraft added a commit that referenced this pull request Jul 11, 2024
- Added config file and settings menu when installed alongside MidnightLib (#4)
- Added entity interaction physics to bubbles
- Added biome dependent tint to all bubble types
- Fixed barrels opened by other players on servers not emitting bubbles (#11) (Thanks @AViewFromTheTop)
- Adjusted current down bubbles to pop on top of magma blocks rather than inside them (#9)
- Added varied pitch and volume to bubble pop and bubble swirl sounds (#8)
- Barrel bubbling is now reliable on the client and will no longer bubble when a chunk with an already opened Barrel is loaded (Thanks @AViewFromTheTop)
- Changed the default behavior to not maintain bubble particle velocity when popping (Thanks @AViewFromTheTop)
- Tweaked explosion bubbles and used better variable names
- Restructured bubble particle logic and used better variable types
- Adjusted barrel bubble spawn locations
- Moved bubble pop logic to a separate utility class to reduce code duplication
- Simplified water checks to use the already provided helper function in world or level
- Fixed literal new line in fabric.mod.json after building (#12)
- Updated to Gradle 8.7, Loom 1.6 and ForgeGradle 6.+
Tschipcraft added a commit that referenced this pull request Jul 11, 2024
- Added config file and settings menu when installed alongside MidnightLib (#4)
- Added entity interaction physics to bubbles
- Added biome dependent tint to all bubble types
- Fixed barrels opened by other players on servers not emitting bubbles (#11) (Thanks @AViewFromTheTop)
- Adjusted current down bubbles to pop on top of magma blocks rather than inside them (#9)
- Added varied pitch and volume to bubble pop and bubble swirl sounds (#8)
- Barrel bubbling is now reliable on the client and will no longer bubble when a chunk with an already opened Barrel is loaded (Thanks @AViewFromTheTop)
- Changed the default behavior to not maintain bubble particle velocity when popping (Thanks @AViewFromTheTop)
- Tweaked explosion bubbles and used better variable names
- Restructured bubble particle logic and used better variable types
- Adjusted barrel bubble spawn locations
- Moved bubble pop logic to a separate utility class to reduce code duplication
- Simplified water checks to use the already provided helper function in world or level
- Fixed literal new line in fabric.mod.json after building (#12)
- Updated to Gradle 8.7, Loom 1.6 and ForgeGradle 6.+
- Added media for changelog
- Updated version range for forge
- Updated GitHub workflow to include version range
@Tschipcraft Tschipcraft merged commit d108e5e into Tschipcraft:master Jul 14, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment