cc-tweaked / CC-Tweaked

Just another ComputerCraft fork
https://tweaked.cc
882 stars 206 forks source link

Example GPS constellation in GPS guide had sub-optimal setup of hosts. #1681

Open Wojbie opened 5 months ago

Wojbie commented 5 months ago

Explanation of Issue

Example constellation in doc/guides/gps_setup.md GPS guide had sub-optimal setup of hosts. It results in devices on same y level as constellation to get ambiguous positions errors as close as 500 blocks away from it. This is because 3 out of 4 computers have same y level. Issue intermittent depending on exact distances.

Example of issue (Provided by user who discovered it):

image5

https://github.com/cc-tweaked/CC-Tweaked/assets/5893536/ee199685-7b3b-4b6a-baee-b5b525d34ffb

Possible solution 1

Possible solution would be to change from example constellation to one commonly used before this guide was made (yes i know labels in this image as misleading). But it is harder to build and explain. 2P3XL

Possible solution 2

We could brainstorm alternative constellation that would have increased reliability. Also if someone smarter then me was able to math exactly why first picture returned ambiguous position we would have more data what is happening here.

SquidDev commented 5 months ago

There was a similar issue in #1530 and a doc update in #1542 which I've still not merged (sorry Lupus, I entirely forgot!). I'm not thrilled about how the GPS tutorial reads right now - there's too many asides in it - but maybe it's better to pull that PR in for now anyway.

It is just sufficient to move the hosts further apart from each other. I hadn't actually considered alternative shapes - the one you're proposing will also fail at a certain distance, but I don't know when the drop-off is.

Lupus590 commented 5 months ago

My goal with the constellation was to have something that was easy to build even for someone who couldn't see the image.

Maybe we should have a note about a more optimised setup?

Edit: Thinking about this, could it be fixed by moving one of the 3 computers at the bottom to be at the same y level as the computer at the top?

I'm not thrilled about how the GPS tutorial reads right now - there's too many asides in it

Yeah, maybe that should be footnotes if they are worth keeping at all.

Wojbie commented 5 months ago

I have completely missed that this issue was reported before.

My goal with the constellation was to have something that was easy to build even for someone who couldn't see the image.

I definitely agree that your shape is simpler to explain and build.. What about increasing example suggested size as noted in #1530 between computers from 4 blocks (and only 3 blocks upward) to be 5 blocks apart in all axis? This would increase the accuracy in common ranges while still keeping it a small build. Even just moving upper computer to 4 block from 3 seems to help with this issue from my test.

Edit: Thinking about this, could it be fixed by moving one of the 3 computers at the bottom to be at the same y level as the computer at the top?

It might also increase accuracy to move one of arm computers upward yea... but i think just increasing proposed size as SquidDev mentioned would solve this while keeping array shape simple to understand.

Lupus590 commented 5 months ago

I would offer to rewrite the guide but life is very busy for me lately and I've not been immersed in CC for a while.

If I can find time and motivation, I'll rewrite the entire guide and probably incorporate #1542 into it.