Conversation
|
@Big-Iron-Cheems You can remove all |
|
Welp, from what I see Ravel should support them fine for Java. Might try it again later to be sure. |
42b9af5 to
51005f3
Compare
|
yay |
Broken code is to be expected, check classes containing `TODO(Ravel)` in them.
Look for remaining `// TODO(Ravel)` and anything else I may have missed.
Not tested yet but it should work.
…boundAttackPacket
Note: there still are some broken mixins to handle
Marked via `TODO(Baritone)`
Also mark `26.1` as backwards compatible.
…date` From what I see the logic for handling flow push has been moved here.
51005f3 to
f3bd22a
Compare
|
Rebased to account for Sodium mixin + version change in master. |
Not sure if this is correct, but it makes sense. As with other changes needs testing in game later on.
Need to complete GameRendererMixin porting.
…hain warning In the future Gradle 10 will require this to be enabled: https://docs.gradle.org/current/userguide/toolchains.html#sec:provisioning
… opacity scope - route xray fluid alpha through new `Xray.getFluidAlpha(FluidState, BlockPos)` - add `fluid-opacity` enum setting to Xray (`none|water|lava|both`, default `both`) - apply fluid-specific alpha in vanilla and sodium fluid renderers - force translucent layer/material for partial alpha: - vanilla fluids via `FluidRendererMixin` `getBuilder(...)` arg rewrite - vanilla blocks via `ModelBlockRendererMixin` quad material layer rewrite - sodium blocks/fluids via existing material overrides - fix vanilla fluid vertex color path to use correct RGB values (remove `* 255`) - make xray block-entity cancellation alpha-aware (cancel only when alpha == 0) - remove obsolete `ItemBlockRenderTypesMixin` (legacy target removed in 26.1) - use correct color packing for Sodium lava color - ensured compatibility with Sodium via in game testing
|
Nice work at the moment |
- applies both to vanilla and Sodium rendering - add local xray field to simplify
|
Why ins't this merged? |
This comment was marked as resolved.
This comment was marked as resolved.
The PR isn't merged yet because:
In the meantime you (users and contributors) are free to test out this PR by downloading the artifacts from the latest entry of this actions list. Note that this requires a GitHub account. |
Was just curious, have you put anymore thought into testing if that Baritone PR would work temporarily? Was hoping to test it out if possible. |
|
I have not. Looking at this cabaletta/baritone#4990 PR, I believe it should work just fine. |
|
Ahh gotcha. Looking forward to seeing everything work together in 26.1.2 Thanks for the work you have done on this so far has been working great! |
Type of change
Description
Future 26.1 update.
Related issues
Closes #6247
How Has This Been Tested?
Builds and runs, some bugs have been found.
Most are fixed, notably
AirJumpdoesn't seem to work.Notably,
Baritonedependant code is commented out until they update to26.1.Checklist: