Skip to content

Conversation

@stanriders
Copy link
Member

@stanriders stanriders commented Aug 7, 2024

@bdach
Copy link
Collaborator

bdach commented Aug 7, 2024

I'm well aware that people want this but please just let us try deploying that other PR first as a baseline to make sure everything doesn't just die from that alone. I dunno if I'd be willing to risk just dropping this change into production and seeing how hard it explodes - would definitely be in favour of a more careful rollout, throughput measurements included.

@stanriders
Copy link
Member Author

I'm well aware that people want this but please just let us try deploying that other PR first as a baseline to make sure everything doesn't just die from that alone. I dunno if I'd be willing to risk just dropping this change into production and seeing how hard it explodes - would definitely be in favour of a more careful rollout, throughput measurements included.

I'm not sure why you assume this needs to be deployed before thoroughly testing the deploy of the processing change, this PR can wait until that for sure

@pull-request-size pull-request-size bot added size/S and removed size/XS labels Aug 7, 2024
@bdach
Copy link
Collaborator

bdach commented Aug 7, 2024

I'm not sure why you assume this needs to be deployed before thoroughly testing the deploy of the processing change

I'm not. I'm trying to communicate what I think the process for this should be and manage expectations. For everyone's sake, not only yours. Other people read these threads and draw conclusions, not always correct ones.

@stanriders
Copy link
Member Author

Sure, I'd consider this change sort of a "second phase" of realtime calculation testing before ranking any other mods/mod settings as well

@pull-request-size pull-request-size bot added size/XS and removed size/S labels Aug 7, 2024
@smoogipoo smoogipoo changed the base branch from master to pp-dev December 18, 2024 14:09
@peppy peppy deleted the branch ppy:master February 26, 2025 16:17
@peppy peppy closed this Feb 26, 2025
@stanriders stanriders reopened this Feb 26, 2025
@stanriders stanriders changed the base branch from pp-dev to master February 26, 2025 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Pending Merge

Development

Successfully merging this pull request may close these issues.

5 participants