-
Notifications
You must be signed in to change notification settings - Fork 172
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
extended multi rotation #1358
extended multi rotation #1358
Conversation
… that that's possible and will save SO MANY ALLOCATIONS.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good to me.
It seems like the formatting got messed up in a lot of places here. I'd suspect spotless automatically breaking lines at a certain length, but not 100% sure.
I've added formatting suggestions in the places where it negatively affects readability.
src/main/java/com/gregtechceu/gtceu/api/machine/multiblock/MultiblockControllerMachine.java
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/client/renderer/machine/HPCAPartRenderer.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/client/renderer/machine/ChargerRenderer.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/client/renderer/machine/MaintenanceHatchPartRenderer.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/client/renderer/machine/OverlayTieredMachineRenderer.java
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/client/renderer/machine/PumpHatchPartRenderer.java
Outdated
Show resolved
Hide resolved
src/main/java/com/gregtechceu/gtceu/client/renderer/machine/SimpleGeneratorMachineRenderer.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Mikerooni <[email protected]>
Co-authored-by: Mikerooni <[email protected]>
Co-authored-by: Mikerooni <[email protected]>
Co-authored-by: Mikerooni <[email protected]>
Co-authored-by: Mikerooni <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
* fix machines not conducting redstone (might fix #1146) * try to fix #728 * implement extended multiblock rotation * Revert "try to fix #728" This reverts commit f27a5b9. * make upwards facing a blockstate property * make all extended-faceable multis allow all rotations as well * add KJS usability * make all machines use static face baking methods because I discovered that that's possible and will save SO MANY ALLOCATIONS. * try to fix render (didn't work) * start of fixing the machine overlay * more maths * fixes rotations for unformed machines * merge to 1.20.1 * lang handler fix * fix errors * fix overlay z-fighting * shit's fucked and idk why (it zfights when formed) * fix the dumbfuck models z-fighting * formatting part 1 Co-authored-by: Mikerooni <[email protected]> * formatting part 2 Co-authored-by: Mikerooni <[email protected]> * formatting part 3 Co-authored-by: Mikerooni <[email protected]> * formatting part 4 Co-authored-by: Mikerooni <[email protected]> * fuck it we ball Co-authored-by: Mikerooni <[email protected]> * spotless * disable spotless in build-on-push check by instead using assemble task --------- Co-authored-by: YoungOnion <[email protected]> Co-authored-by: Mikerooni <[email protected]>
it work yipppeeee
fixes #380