Skip to content

getnode and setnode#11503

Open
TripleToxic wants to merge 28 commits intoAnuken:masterfrom
TripleToxic:master
Open

getnode and setnode#11503
TripleToxic wants to merge 28 commits intoAnuken:masterfrom
TripleToxic:master

Conversation

@TripleToxic
Copy link
Copy Markdown
Contributor

@TripleToxic TripleToxic commented Dec 28, 2025

  • getnode instruction using index to retrieve a power or bridge node.
  • Sensor for total nodes/links for power node/bridge (cannot be combined because Phase Bridge exist).

update:

  • setnode instruction to modify connections of bridge links and power nodes (has a check for net.client() so hopefully it will not become a desync machine like setprop velocityX/Y

Edit: these instruction have been merged into sensor and control

If your pull request is not translation or serverlist-related, read the list of requirements below and check each box:

  • I have read the contribution guidelines.
  • I have ensured that my code compiles, if applicable.
  • I have ensured that any new features in this PR function correctly in-game, if applicable.

@Anuken
Copy link
Copy Markdown
Owner

Anuken commented Dec 30, 2025

This should be implemented as part of control and sense with different parameters. They don't need to be separate instructions, and they should be implemented in the blocks, instead of utilizing instanceof.

It would take a bit of refactoring to get sense working with parameters, however.

@TripleToxic
Copy link
Copy Markdown
Contributor Author

This should be implemented as part of control and sense with different parameters. They don't need to be separate instructions, and they should be implemented in the blocks, instead of utilizing instanceof.

It would take a bit of refactoring to get sense working with parameters, however.

hmmm, let me try this one out

@TripleToxic
Copy link
Copy Markdown
Contributor Author

done

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.

2 participants