ReikaKalseki / Reika_Mods_Issues

The issue tracker for all of my mods - RotaryCraft, its addons, ChromatiCraft, and everything else.
46 stars 13 forks source link

ReactorCraft: Nuclear Waste Disposal Drum not stacking similar waste #839

Closed Fatmice closed 8 years ago

Fatmice commented 8 years ago

ReactorCraft 1.7.10V13b Per title 2016-05-19_01 55 26 2016-05-19_01 55 29

ReikaKalseki commented 8 years ago

Both waste types work the same. I remember changing this a long time ago.

Fatmice commented 8 years ago

Interesting, so you changed it again. =P

ReikaKalseki commented 8 years ago

"Again"? I remember this being changed because stacking the waste amplified the radiation/heat effects of the parent tiles.

Fatmice commented 8 years ago

Well, if stacking is not desired, I can tell you it is still stacking. You just have to remove some for things to filter down and as they do they will start to stack.

Stacking the waste also lowers the chance for them to decay since decay chance is not evaluated per item in a stack but only per stack.

ReikaKalseki commented 8 years ago

Wait...it is still stacking, and ignoring IInventory rules?

Fatmice commented 8 years ago

Yep, would you like a picture? :smile:

Stack is no more than sixteen items though, like before.

ReikaKalseki commented 8 years ago

Wtf...yes, post an image.

Fatmice commented 8 years ago

Here's some. To get them to stack you just have to remove some waste to make room. As they filter down, they will start to stack. 2016-05-19_23 24 19 2016-05-19_23 25 01

Eventually, you end up with this 2016-05-19_23 21 13 2016-05-19_23 21 19

This is opposite of what you want. The behavior that you want is of a spent fuel container where stacking is not possible. Though to be fair, the waste storage container was for long term. Waste in these containers do not decay significantly during game play if at all, even under acceleration. At their current storage size, they are too small. I suggest you at least quadruple their storage to make up for their lack of stacking...The long-lived wastes build up rather quickly and with only 12 slots per drum you need a lot of drums very fast. With each drum eating 10us, 100 of them would be eating 1ms. This is not a stretch of the imagination. No wonder people like to store your waste in the ME hard drives or void them all together. It's more tick friendly, no hard feelings.

The alternative is something that I've suggested before. Make the waste useful, i.e. nuclear batteries, or transmute them with neutron beam bombardment generated by accelerator driven system. What's all that power you make in the fusion reactor for? Use it to drive a synchrotron to generate a proton beam and smash it against heavy nuclei source to make the neutron beam to transmute long-lived waste with a chance of turning into something that's more likely to decay within the lifetime of the game. This latter is not only more fun but feels less overbearing.

The alternative, since I highly doubt you will be willing to do all of that work, is make tile accelerator work super efficient against waste with long half lives. Instead of a flat 256x, make it scale against the waste such that all wastes that go into the storage drums will decay at the same rate as Cesium-137. This means the longer the half-lives, the more effective the tile accelerators.

ReikaKalseki commented 8 years ago

I cannot reproduce the stacking behavior unless the stack being put in is of a size greater than one. Also, the code seems to imply that they were meant to stack after all, at least to 16.

And I cannot performantly combine different stacks of similar waste. That means iterating over the entire inventory for every slot in the inventory (144 comparisons per tick).

Fatmice commented 8 years ago

Well it originally would stack to 16 for each slot. At least that is how I remembered it about V7 for 1.7.10.

ReikaKalseki commented 8 years ago

And it will if you place it manually, or use item automation. But feeding cannot do so performantly.