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

second attempt. ports spatial gridding #22539

Closed

Conversation

wonderinghost
Copy link
Contributor

Document the changes in your pull request

ports over the spatial grid and fixes from tg. yell at me when this is ready to source those that made it

read the original pr but essentially optimizes radio signals to be super fast

Why is this good for the game?

late but more optimizations are good. also the final piece to port over modsuits and integrated circuits

Testing

TBD. but if cool with the original then take that

Changelog

🆑
rscadd: ports the spatial grid subsystem
tweak: systems that communicate over the radio have been refitted for grid
experimental: This is experimental.
/:cl:

@Yogbot-13 Yogbot-13 added DME Edit This PR affects the yogstation.DME file Feature This adds new content to the game Tweak This PR contains changes to the game. labels Aug 18, 2024
@Tsar-Salat
Copy link

Tsar-Salat commented Aug 20, 2024

@wonderinghost Just added spatial grid for bee, lmk if you need help or anything, it was kind of annoying there lol.

You dont mark or credit any of the authors, so I can't tell if you grabbed all the fixes or not.

Here is my port, if needed for reference: BeeStation/BeeStation-Hornet#10730

The TG people have kinda sucky testing evidence (and their script didnt even work) I found, so I recommend testing get_hearers() before and after the pr in Tracy, and just measuring the speed increase.

@wonderinghost
Copy link
Contributor Author

Thanks

@wonderinghost
Copy link
Contributor Author

restarting branch, been inactive for too long

@wonderinghost wonderinghost deleted the Spatial_gridding2 branch October 27, 2024 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DME Edit This PR affects the yogstation.DME file Feature This adds new content to the game Tweak This PR contains changes to the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants