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

Do not allow items to be inserted into the Matter Condenser when the output slot is full #8335

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

62832
Copy link
Member

@62832 62832 commented Jan 18, 2025

Closes #8311.

@Mithi83
Copy link
Contributor

Mithi83 commented Jan 18, 2025

I was tempted to tackle this bug too but ultimately didn't because this changes the semantics of the matter condenser a bit. The old implementation was something along the line of it being a fancy trash can which could potentially give you something in return if you can manage to pull it out fast enough. The emphasis was on trash can with a small perk, i.e. its main feature was to void items as fast as they could be put inside of it.

Your PR changes that to something like this: The matter condenser is primarily used for generating a resource (matter balls or singularities) from trash. This can now stall if extraction is too slow to prevent the build up and this could propagate down the entire supply line and thus lead so some unintended side effects.

I've not taken a closer look at the code yet, but have you considered the cases that the matter condenser is set to trash can mode and may or may not have any storage? Will that stall eventually as well?

@62832
Copy link
Member Author

62832 commented Jan 18, 2025

I've not taken a closer look at the code yet, but have you considered the cases that the matter condenser is set to trash can mode and may or may not have any storage? Will that stall eventually as well?

If the Matter Condenser is set purely to its trashing mode then there's never an output to consider that would even stall; getOutput() always returns an empty ItemStack in that case and canAddOutput() always returns true as a result.

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.

Matter Condenser deletes items above storage cell limit if output is full
2 participants