-
-
Notifications
You must be signed in to change notification settings - Fork 107
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
Fixes to Schematic Command relating to Entities #2623
base: dev
Are you sure you want to change the base?
Conversation
…Now functions as expected for any valid yaw value. Update entity rotation mechanism and entity yaw anytime schematic is flipped or rotated, instead of calculating when pasting "how much entity has rotated since creation" then applying. Remove an offset compensation for hanging entities being weird. During testing this has not caused any issues, unsure if this was an issue previously due to incorrect entity rotations.
@@ -104,7 +104,7 @@ public void run(ReplaceableTagEvent event) { | |||
// This takes an optional duration as "fake_duration" for how long the fake blocks should remain. | |||
// | |||
// The "create" and "paste" options allow the "entities" argument to be specified - when used, entities will be copied or pasted. | |||
// At current time, entity types included will be: Paintings, ItemFrames, ArmorStands. | |||
// At current time, entity types included will be: Paintings, ItemFrames, GlowItemFrames, ArmorStands, and DisplayEntities. |
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.
Is DisplayEntities
implemented?
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.
Ah good point... Display Entities were already included in the types to be copied so the meta update for it was just to document what already existed. However after adding display entities to the test schematic I'm seeing them getting updated in unexpected ways, losing transformation data, and text_displays losing text.
After disabling the thrown exception on EntityHelper.setStepHeight
(not all entity mechanisms are applied otherwise) it does actually spawn display entities, however rotation transformation is still completely lost - wondering if this might be a v1.20.6 issue? I'm not able to spawn in display entities with pitch/yaw ever applied either.
Want me to remove the DisplayEntities note for the moment?
Have you tested this very very very thoroughly with item frames in weird positions? There's a lot of obscure hacks in the original code that exist entirely because of incredibly specific edge-case behaviors with item frames |
I'm pretty confident in the transformations for item_frame, glow_item_frame, and armor_stand. Attached are the script and schematic I've used during testing, which has a bunch of item frames and armor stands in all sorts of positions. The script pastes 50 copies of the schematic, applying a random transformation to the same schematic between each paste. I did catch a comment in the |
Fixes Schematic Command relating to Entities
Notable potential issues