Azutte

Untitled

Dec 24th, 2016
176
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 2.84 KB | None | 0 0
  1. [20:10:10] <Azu> hey so I'm using worldedit-forge-mc1.10.2-6.1.4-SNAPSHOT-dist.jar and seeing a ton of issues with rotating logs/pillars in the clipboard, has this been noticed and possibly fixed since then?
  2. [20:11:18] <Azu> and by "a ton" I think I just mean "it doesn't do it correctly"
  3. [20:11:49] <Azu> copy some logs/pillars, rotate 90 degrees, paste, and they are not the correct variant
  4. [20:12:07] <Azu> well, right block, wrong resulting direction
  5. [20:30:47] <Azu> and I'm looking back at work I did like a year ago and seeing the same issue
  6. [20:36:34] * Tenebrae ([email protected]) has joined #sk89q
  7. [20:51:38] <@wizjany> uh
  8. [20:51:42] <@wizjany> what blocks specifically
  9. [20:59:15] <Azu> purpur pillars, some log that I can't remember if it was dark oak or spruce
  10. [21:00:14] <Azu> I've since spotted this with participation from you in it http://dev.enginehub.org/youtrack/issue/WORLDEDIT-3490
  11. [21:00:35] <Empire92> I think it's all logs, normal oak logs don't rotate as well
  12. [21:01:25] <Empire92> The logs have axis information in the blocks.json which could be converted to direction in the BundledBlockData class
  13. [21:04:04] <@wizjany> axis information isn't direction information
  14. [21:04:24] <Empire92> wizjany, it can be converted without issue though
  15. [21:04:36] <@wizjany> not without hardcoding names
  16. [21:05:12] <Empire92> Yes it can, just check if states.containsKey("axis"), and convert that, you don't need to know specific names
  17. [21:05:33] <@wizjany> "axis" <-- literally definition of harcoding
  18. [21:06:22] <Empire92> Another solution is to do it in your registry dumper
  19. [21:06:27] <@wizjany> still harcoding
  20. [21:06:30] <@wizjany> do you
  21. [21:06:32] <@wizjany> not understand
  22. [21:06:35] <@wizjany> the definition here
  23. [21:06:50] <@wizjany> if i have to manually write in the names of properties or their values to get a Vector from them
  24. [21:06:51] <Empire92> Yeah, sure I understand that is's hardcoding. I didn't read properly before responding dude
  25. [21:06:52] <@wizjany> it's hardcoding
  26. [21:07:17] <@wizjany> which means
  27. [21:07:26] <Empire92> You're arguing semantics, which is mostly unimportant. Are you willing for a fix that way or not?
  28. [21:07:33] <@wizjany> yes
  29. [21:07:38] <@wizjany> i mentioned that in my comments on that issue
  30. [21:07:40] <@wizjany> that was just linked
  31. [21:07:51] <@wizjany> "right, we'd somehow have to guess (or hardcode), based on the name of the properties, whether something like "axis" actually refers to rotation data, and then guess (or hardcode), based on the names of each value, what direction something like "up_x" refers to."
  32. [21:08:00] <Empire92> I can do a pr
  33. [21:13:06] * Nentify ([email protected]) Quit (Quit: Connection closed for inactivity)
  34. [21:13:21] <@wizjany> i wonder if our old unit test still works at all
  35. [21:13:26] <@wizjany> purpur pillars are new tho
Advertisement
Add Comment
Please, Sign In to add comment