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.

Sign up for GitHub

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

Jump to bottom

Bug: Cumulus crashing on servers. #7

Closed
bconlon1 opened this issue Jul 28, 2023 · 1 comment · Fixed by #8
Closed

Bug: Cumulus crashing on servers. #7

bconlon1 opened this issue Jul 28, 2023 · 1 comment · Fixed by #8
Assignees
@bconlon1
Labels
feat/system This issue is associated with a system. type/bug The issue describes a bug or error in the project. type/crash The issue describes a problem which crashes the game or kicks the player from a server.

Comments

@bconlon1
Copy link
Contributor

No description provided.

@bconlon1 bconlon1 added feat/system This issue is associated with a system. type/bug The issue describes a bug or error in the project. type/crash The issue describes a problem which crashes the game or kicks the player from a server. labels Jul 28, 2023
@bconlon1 bconlon1 self-assigned this Jul 28, 2023
@Jonathing
Copy link
Contributor

L

@Jonathing Jonathing mentioned this issue Jul 28, 2023
Merged
bconlon1 pushed a commit that referenced this issue Jul 28, 2023
@Jonathing
fix: Only initialize Cumulus on client (#8)
8b9062b 
closes #7

Uses `DistExecutor.unsafeRunForDist()` to only call initialization code on client, and merely send a log message on server.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bconlon1 bconlon1

Labels
feat/system This issue is associated with a system. type/bug The issue describes a bug or error in the project. type/crash The issue describes a problem which crashes the game or kicks the player from a server.
Projects
Cumulus
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Only initialize Cumulus on client Jonathing/Cumulus
2 participants
@Jonathing @bconlon1