Skip to content

Added projector sable compat - #132

Closed
sculpted-code wants to merge 205 commits into
PC-Logix:main-MC1.21.1from
sculpted-code:main-MC1.21.1
Closed

sculpted-code wants to merge 205 commits into
PC-Logix:main-MC1.21.1from
sculpted-code:main-MC1.21.1

Conversation

@sculpted-code

@sculpted-code sculpted-code commented Oct 24, 2026 •

Copy link
Copy Markdown

Bug fixes:

  • now players can interact with projection if it is on sublevel

New featurest:

  • projection now stops on closest world or sublevel hit
image

DadamaldaDad and others added 30 commits October 6, 2026 17:05
@sculpted-code

Copy link
Copy Markdown
Author

I've moved some logic to SableCompat util. Didn't noticed util at first

@sculpted-code sculpted-code reopened this Oct 25, 2026
return SableCompanion.INSTANCE.distanceSquaredWithSubLevels(level, first, second);
}

public static double distanceRectilinear(Level level, Vec3 first, Vec3 second) {

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, i should've named it distanceTo...

@CaitlynMainer

Copy link
Copy Markdown
Member

So this PR can stay open, but all of the files you're modifying no longer exist locally. I've got ~1,120 files pending a big commit that moves about 95% (so far) of the mod from Scala to Java. Once that's done, I'll review this, and you can either update it to Java, or I can take your Scala and merged it into the Java rewrite.

I appreciate the PR, it's a neat feature.

@sculpted-code

Copy link
Copy Markdown
Author

Yes, i would love to move code to Java, considering size of changes pr makes, it should be easy

But while i am on it, i could try fixing all Sable related issues, holoscreens and holograms(wich do not render correctly even in world) as an example

@sculpted-code

sculpted-code commented Oct 25, 2026 •

Copy link
Copy Markdown
Author

Apparently, you can't draft pr with gh mobile

@sculpted-code sculpted-code reopened this Oct 25, 2026
@sculpted-code
sculpted-code marked this pull request as draft October 25, 2026 15:25
@CaitlynMainer

Copy link
Copy Markdown
Member

Ok, the Java rewrite is done, so you're good to go whenever.

@CaitlynMainer
CaitlynMainer force-pushed the main-MC1.21.1 branch 2 times, most recently from 591bd0e to e20e05a Compare October 26, 2026 02:17
@CaitlynMainer

Copy link
Copy Markdown
Member

Manually merged in

CaitlynMainer added a commit that referenced this pull request Oct 26, 2026
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.

5 participants