4 ms·

Thirteen year old singularity bug fixed

31 points by beckthompson 2y ago

8485501956 2y ago
[flagged]
maxbond 2y ago
steps = 3//Yes this is right So, this comment ought to explain their reasoning and it doesn't, which is a shame - but isn't it concerning that this person was warning us that there was a reason this value was 3 and not 2, and that no one in the PR seems to know what that reason is and whether it's still relevant? Who were you, mport2004? What did you see?
- jorvi 2y ago
  Seems likely it’s an off-by-one error deeper in the code, and they decided to just work around it? Reminds me of Chesterton’s Fence: you shouldn’t replace an old, out of place seemingly-useless fence unless you understand the cause for it being there.
- maxbond 2y ago
  > ...and that no one in the PR seems to know what that reason is and whether it's still relevant? Upon rereading I feel I was a but uncharitable here, with more context it's clear the reviewers don't need to look at this too closely because the blast radius is effectively zero, and I imagine the commiter did ask themselves this question and probably prioritized the aesthetic of the terse meme PR over a complete explanation (and fair enough).
exo-pla-net 2y ago
This is a fix for a 13 year old bug from SpaceStation 13, a multiplayer computer game. The bug fix is for an obsolete engine for the game. It's a fix, but it's completely pointless. A meme fix. More broadly, it's probably not why you clicked on this link. Hopefully I've saved you some time. (Or, if you're of a certain perverse mindset, hopefully I've introduced you, so that you can dive in.)
ourmandave 2y ago
It only seemed like 13 years, because the closer you get to a singularity...
stoltzmann 2y ago
I disagree, the containment field should be a minimum of 4x4. This was done to prevent engineers from being too greedy and not giving the singulo the space it deserves. Also it gives an additional layer of safety that gave you a moment to act and shut down the PA when it grew from 3 to 4 size. Putting that in a 3x3 containment will eat the field emitters and the singulo will be loose. Having it require a 4x4 field in the first place was a safer option.
- Ikalpo 2y ago
  IDK what server you’re in that has even number singularity sizes. Without this fix, a stage 2 (3x3) singularity will still escape a 4x4 containment field 50% of the time.
deleted 2y ago
[deleted]
Ikalpo 2y ago
hey its me ur pr author Yeah I basically sat on this fix for a year because of that comment, then the topic of singulos got brought up again in the server I’m in so I decided to just send it.

31 points by beckthompson 2y ago

8485501956 2y ago

[flagged]

steps = 3//Yes this is right So, this comment ought to explain their reasoning and it doesn't, which is a shame - but isn't it concerning that this person was warning us that there was a reason this value was 3 and not 2, and that no one in the PR seems to know what that reason is and whether it's still relevant? Who were you, mport2004? What did you see?

jorvi 2y ago
Seems likely it’s an off-by-one error deeper in the code, and they decided to just work around it? Reminds me of Chesterton’s Fence: you shouldn’t replace an old, out of place seemingly-useless fence unless you understand the cause for it being there.
maxbond 2y ago
> ...and that no one in the PR seems to know what that reason is and whether it's still relevant? Upon rereading I feel I was a but uncharitable here, with more context it's clear the reviewers don't need to look at this too closely because the blast radius is effectively zero, and I imagine the commiter did ask themselves this question and probably prioritized the aesthetic of the terse meme PR over a complete explanation (and fair enough).

exo-pla-net 2y ago

This is a fix for a 13 year old bug from SpaceStation 13, a multiplayer computer game. The bug fix is for an obsolete engine for the game. It's a fix, but it's completely pointless. A meme fix. More broadly, it's probably not why you clicked on this link. Hopefully I've saved you some time. (Or, if you're of a certain perverse mindset, hopefully I've introduced you, so that you can dive in.)

ourmandave 2y ago

It only seemed like 13 years, because the closer you get to a singularity...

stoltzmann 2y ago

I disagree, the containment field should be a minimum of 4x4. This was done to prevent engineers from being too greedy and not giving the singulo the space it deserves. Also it gives an additional layer of safety that gave you a moment to act and shut down the PA when it grew from 3 to 4 size. Putting that in a 3x3 containment will eat the field emitters and the singulo will be loose. Having it require a 4x4 field in the first place was a safer option.

Ikalpo 2y ago
IDK what server you’re in that has even number singularity sizes. Without this fix, a stage 2 (3x3) singularity will still escape a 4x4 containment field 50% of the time.

deleted 2y ago

[deleted]

Ikalpo 2y ago

hey its me ur pr author Yeah I basically sat on this fix for a year because of that comment, then the topic of singulos got brought up again in the server I’m in so I decided to just send it.