Skip to content

Conversation

@BuckarooBanzay
Copy link
Member

Performance tests

  • vmanip (iterative, nested)
    • 80 ms for a 20x20x20 cube
  • vmanip (flood-fill)
    • TBD
  • get/setnode (flood-fill)
    • TBD
  • mapblock-based (flood-fill, breaks current behavior!)
    • TBD

@BuckarooBanzay BuckarooBanzay added the enhancement New feature or request label Jan 29, 2024
@BuckarooBanzay BuckarooBanzay self-assigned this Jan 29, 2024
})
--]]

minetest.register_chatcommand("test", {
Copy link
Member

Choose a reason for hiding this comment

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

Is this debug command supposed to be there 😉

local area = VoxelArea:new({MinEdge=e1, MaxEdge=e2})
local node_ids = manip:get_data()

for z=ipos1.z, ipos2.z do
Copy link
Member

Choose a reason for hiding this comment

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

A bit scared by those nested fors, but it looks like that the limits are very close to each other.

Just wondering why using this syntax because after you are adding/subtracting 1 coordinate-by-coordinate.

@BuckarooBanzay
Copy link
Member Author

@Athozus this is still a work-in-progress PR (or draft in github terms) the /test command will go away if this ever works properly (expect more of these in the future this is the way i develop/prototype right now)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants