Skip to content

Changed the _get_blocks() method of block.Blocks()#23

Open
thelabcat wants to merge 2 commits into
waveform80:masterfrom
thelabcat:patch-1
Open

Changed the _get_blocks() method of block.Blocks()#23
thelabcat wants to merge 2 commits into
waveform80:masterfrom
thelabcat:patch-1

Conversation

@thelabcat
Copy link
Copy Markdown

A multi-block vector slice of World().blocks now properly returns block data instead of inevitably setting it to 0. Tested with Raspberry Jam.

A multi-block vector slice of World().blocks now properly returns block data instead of inevitably setting it to 0. Tested with Raspberry Jam.
@thelabcat
Copy link
Copy Markdown
Author

#22 is fixed by this change for Raspberry Jam.

@thelabcat
Copy link
Copy Markdown
Author

thelabcat commented Sep 2, 2022

Just discovered a major problem resulting from a misunderstanding of which of the separators (comma, and vertical bar) were used for separating block ID and data, and for separating individual blocks from each-other in the returned network transaction. Have edited a fix, successfully tested with Raspberry Jam.

@thelabcat
Copy link
Copy Markdown
Author

thelabcat commented Sep 2, 2022

Just tested on vanilla Minecraft Pi Edition v0.1.1-6 and this fix fully functional. It does not break the API (PTL😁).

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.

1 participant