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

Update to 1.21.3/1.21.4 #399

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Update to 1.21.3/1.21.4 #399

wants to merge 5 commits into from

Conversation

Xtendera
Copy link

@Xtendera Xtendera commented Dec 22, 2024

Description

I am attempting to update this addon to 1.21.3. There is a lot of changes so bare with me while I attempt to fix everything.

Related Issue

#397

Motivation and Context

This project is a bit backlogged, I totally get it and I will try to lessen the work for maintainers.

How Has This Been Tested?

I will individually test some features once it starts working, but I will ask for other people's help to make sure everything works properly and will make changes as needed.

Builds

Testing builds for 1.21.3 are now available.

Build 1:
Artifact

Roadmap (Not all changes may be committed yet, this is specifically for my own tracking)

  • Update Gradle and Dependencies
  • CenterCommand.java
  • KickCommand.java
  • InteractionScreen.java
  • GameRendererMixin.java
  • AntiCrash.java
  • ExtraElytra.java
  • PacketFly.java
  • ArrowDmg.java
  • AutoCraft.java
  • BoatPhase.java
  • ChorusExploit.java
  • CoordLogger.java
  • FullFlight.java
  • Jetpack.java
  • KnockbackPlus.java
  • NewChunks.java
  • Ore.java

⚠️Rendering.java (See below)

  • ShieldBypass.java

⚠️SkeletonESP.java (See below)

  • SoundLocator.java
  • GiveUtils.java
  • RejectUtils.java
  • WorldUtils.java

⚠️ Issues with SkeletonESP and Shaders

In Minecraft 1.21.2, Minecraft's rendering system was rewritten, breaking some of the code used in many modding projects, including this one. There is work being done to transition old code over to the new system, but it will take some time. For now, expect that some parts of the SkeletonESP Module and the Shaders feature of the Rendering module will be broken until further notice.

Don't expect it to work anytime soon.
@Xtendera
Copy link
Author

By the way, my changes for AutoCraft.java are quite weird, if someone could verify that it logically makes sense and lines up with previous logic that would be great.

Had to revert gradle to 8.10 due to unforseen issues.
@Mleczyk
Copy link

Mleczyk commented Dec 22, 2024

OMG U GOT IT ❤️❤️

@Xtendera
Copy link
Author

image
We're in boys.

On first glance everything seems to be working fine. Please lmk of any issue you encounter.

@Mleczyk
Copy link

Mleczyk commented Dec 22, 2024

ILYSM ❤️❤️

@crazymoose7
Copy link
Contributor

Without opening IDE
it seems line 47 in WorldUtils might be able to be replaced with : mc.player.setSneaking(false);
Rendering.java have you tried mc.getShaderLoader().loadPostEffect() and pass in the ID and DefaultFramebufferSet.STAGES ?

Willtest and take a look at autocraft soon.

@Mleczyk
Copy link

Mleczyk commented Dec 25, 2024

also AutoLogin wont save passwords after register with the create strong password option

@Xtendera
Copy link
Author

Without opening IDE it seems line 47 in WorldUtils might be able to be replaced with : mc.player.setSneaking(false); Rendering.java have you tried mc.getShaderLoader().loadPostEffect() and pass in the ID and DefaultFramebufferSet.STAGES ?

Willtest and take a look at autocraft soon.

I will also look into it soon. Yes line 47 of WorldUtils is overcomplicated lol. I have not tried the getShaderLoader thing, but I will see if that works hopefully today.

also AutoLogin wont save passwords after register with the create strong password option

Will also look into this 👍🏽

@Xtendera
Copy link
Author

@crazymoose7 Okay after some testing the loadPostEffect method does not seem to work in it's current state, it is causing a black screen on launch. I will attempt to move it around (maybe even by mixining some other methods that use it) to see if it could just be a placement thing. I also tried MAIN_ONLY and MAIN_AND_ENTITY_OUTLINE and neither work properly.

@crazymoose7
Copy link
Contributor

@Xtendera I think something worth noting is the different shaders in the shaders/post directory, while not directly connected to this issue. The current enum hasnt been updated to reflect changes.
Did the log output provide anything ?

@vanvac
Copy link

vanvac commented Jan 9, 2025

will this pr be migrated to 1.21.4 or finish the 1.21.3 update and then work on 1.21.4?

@Xtendera
Copy link
Author

Xtendera commented Jan 9, 2025

@Xtendera I think something worth noting is the different shaders in the shaders/post directory, while not directly connected to this issue. The current enum hasnt been updated to reflect changes. Did the log output provide anything ?

Sorry for the late response. Nope, the log output was completely clear, and it seemed that the on screen buttons also worked, just nothing was actually rendered.

will this pr be migrated to 1.21.4 or finish the 1.21.3 update and then work on 1.21.4?

1.21.4 is super easy I will work on that soon. It was the 1.21.2 update that really caused issues. Of course renderer and skeleton might still stay broken/

@areume669
Copy link

How do I download this??

@vanvac
Copy link

vanvac commented Jan 13, 2025

https://github.com/AntiCope/meteor-rejects/actions/runs/12456969030 download the artifact file from this github action and unzip for the 1.21.3 version please note the some render related things are broken due to changes with 1.21.2-3

@Xtendera Xtendera changed the title Update to 1.21.3 Update to 1.21.3/1.21.4 Jan 13, 2025
@fortune428
Copy link

fortune428 commented Jan 14, 2025

when loading the world,the game will crashes,I dont know the reason
game version is1.21.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants