Skip to content
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

Make the EOH store astral arrays internally #2660

Merged
merged 2 commits into from
Jul 14, 2024

Conversation

GDCloudstrike
Copy link
Member

@GDCloudstrike GDCloudstrike commented Jun 13, 2024

This PR makes the EOH store astral arrays internally to prevent them from being moved around between EOHs.
They can be extracted by right clicking the controller with a wire cutter, and are saved to item NBT so they dont get voided if you (accidentally) break the controller.
Existing setups should not be affected, as astral arrays have to be in the input bus anyways, so they will just be removed from the bus and added to the internal counter after updating.

@GDCloudstrike GDCloudstrike marked this pull request as draft June 13, 2024 19:26
@GDCloudstrike GDCloudstrike marked this pull request as ready for review June 13, 2024 19:34
@HoleFish
Copy link
Contributor

I would like a EIG-style of UI if possible...

@GDCloudstrike
Copy link
Member Author

I would like a EIG-style of UI if possible...

Yeah a GUI would be better but I unfortunately wasn't able to get it to work. If you or anyone else wants to implement it then feel free to close this PR in favor of the GUI.

@GDCloudstrike GDCloudstrike requested a review from a team June 16, 2024 18:26
Copy link
Contributor

@NotAPenguin0 NotAPenguin0 left a comment

Choose a reason for hiding this comment

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

Shouldn't the astral arrays also be saved/loaded to NBT data in saveNBTData and loadNBTData?

@Dream-Master
Copy link
Member

@GDCloudstrike

@Dream-Master Dream-Master merged commit 6e005f0 into master Jul 14, 2024
2 checks passed
@Dream-Master Dream-Master deleted the internal_astral_arrays branch July 14, 2024 13:19
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.

4 participants