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

Bump rust-g to 0.8.0 #68673

Closed
wants to merge 2 commits into from
Closed

Bump rust-g to 0.8.0 #68673

wants to merge 2 commits into from

Conversation

san7890
Copy link
Member

@san7890 san7890 commented Jul 22, 2022

About The Pull Request

According to tgstation/rust-g#101 (or tgstation/rust-g@76fd17a ), this version of rust-g makes Cellular Noise faster. We leverage the shit out of Cellular Noise in our cave generation in pretty much every type of mining map (especially IceBoxStation), so I am very keen on seeing this.

Here are some scuffed results from my local test on this claim, where you seem to see a not-bad 0.2 second reduction with the bumped DLL on my debug server:

image

image

Might also be nice to use the other types of noise one day. One day. Also contains some more cargo crates or whatever you wanna call it. Cool.

This is the first time I've bumped a dependency so please let me know if I am forgetting something critically important.

According to tgstation/rust-g#101 (or tgstation/rust-g@76fd17a ), this version of rust-g makes Cellular Noise faster. We leverage the shit out of Cellular Noise in our cave generation across all maps (especially IceBoxStation), so I am very keen on seeing this.

Here are some scuffed results from my local test on this claim, where you seem to see a not-bad 0.2 second reduction with the bumped DLL on my debug server:

Might also be nice to use the other types of noise one day. One day. Also contains some more cargo crates or whatever you wanna call it. Cool.
@san7890 san7890 added the Tools We pretend to be a real development community label Jul 22, 2022
@tgstation-server tgstation-server removed the Tools We pretend to be a real development community label Jul 22, 2022
@san7890
Copy link
Member Author

san7890 commented Jul 22, 2022

i hate machines

@san7890 san7890 marked this pull request as draft July 22, 2022 21:10
@san7890 san7890 marked this pull request as ready for review July 22, 2022 21:12
@san7890
Copy link
Member Author

san7890 commented Jul 22, 2022

wait never mind this doesn't work because #67474 couldn't get through in time

@san7890 san7890 closed this Jul 22, 2022
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.

2 participants