• Joined on 2025-01-14
Lixfel commented on pull request SteamWar/SteamWar#34 2025-04-16 20:16:38 +02:00
Update WorldEdit CUI to RBlockDisplay

Das alles O(Spieler^2)? Unabhängig was gerade los ist (d.h. auch auf dem Event oder Spectateserver mit 70 Mann?). Vllt. doch die alte Lösung (nur auf den Besitzer-Spieler-Wand schauen)?

Lixfel commented on pull request SteamWar/SteamWar#34 2025-04-16 20:16:38 +02:00
Update WorldEdit CUI to RBlockDisplay

Wird doch eh sekündlich gemacht? Muss mMn. nicht noch häufiger aktualisiert werden (vor allem sprengt das die alte Partikellösung, die ja immer noch aktiv ist).

Lixfel commented on pull request SteamWar/SteamWar#34 2025-04-16 20:16:38 +02:00
Update WorldEdit CUI to RBlockDisplay

? Warum das?

Lixfel commented on pull request SteamWar/SteamWar#34 2025-04-16 20:16:38 +02:00
Update WorldEdit CUI to RBlockDisplay

?

Lixfel commented on pull request SteamWar/SteamWar#34 2025-04-16 20:16:38 +02:00
Update WorldEdit CUI to RBlockDisplay

Die CAA-Namen sind jetzt nicht soo pralle (und ist nicht eh fast alles AA in MC?). Würde eher was wie WireframeCube und/oder Line vorschlagen.

Lixfel commented on pull request SteamWar/SteamWar#34 2025-04-16 20:16:38 +02:00
Update WorldEdit CUI to RBlockDisplay

Einmal im Konstruktor anlegen?

Lixfel commented on pull request SteamWar/SteamWar#34 2025-04-16 20:16:38 +02:00
Update WorldEdit CUI to RBlockDisplay

Dieses Methodenpattern ist doch sehr repetetiv. Könnte man da evtl. eine verallgemeintere Variante abstrahieren?

Lixfel commented on pull request SteamWar/SteamWar#34 2025-04-16 20:16:38 +02:00
Update WorldEdit CUI to RBlockDisplay

Sicher, dass super nicht schon tickt? (Tick hier überhaupt notwendig, weil es hier ja eh immer nur null from und to geben kann?

Lixfel commented on pull request SteamWar/SteamWar#34 2025-04-16 20:16:38 +02:00
Update WorldEdit CUI to RBlockDisplay

Warum besteht eine Linie aus 3 Teilen?

Lixfel suggested changes for SteamWar/SteamWar#34 2025-04-16 20:16:38 +02:00
Update WorldEdit CUI to RBlockDisplay

Bin noch nicht ganz durch, dürfte fürs erste aber schon mal genügend sein.

Lixfel commented on pull request SteamWar/SteamWar#40 2025-04-13 19:55:31 +02:00
Use Transfer Packet for Event Velocity

Genau das ist das Problem, was ich beschrieben habe.

Lixfel commented on pull request SteamWar/SteamWar#40 2025-04-13 18:19:38 +02:00
Use Transfer Packet for Event Velocity

Liste von active Fights gibts schon im Velocity, geht allerdings über die aktiven Server (EventStarter?)

Lixfel commented on pull request SteamWar/SteamWar#40 2025-04-13 18:19:38 +02:00
Use Transfer Packet for Event Velocity

Damit werden sie nicht umgeleitet, sondern direkt auf den subserver geschickt.

Lixfel commented on pull request SteamWar/SteamWar#40 2025-04-13 18:19:38 +02:00
Use Transfer Packet for Event Velocity

Nice Idee!

Lixfel suggested changes for SteamWar/SteamWar#40 2025-04-13 18:19:38 +02:00
Use Transfer Packet for Event Velocity

Massives Problem: Es können maximal z.B. 6 Spieler in die Arena, der Rest kann nicht mehr auf den Kampfserver, sondern möchte ja prinzipiell explizit als Spotter auf den Spectatorserver. Oder nicht jeder ist Fighter... Daher finde ich das mit dem automatischen Senden leider nicht gut.

Lixfel suggested changes for SteamWar/SteamWar#40 2025-04-13 17:56:56 +02:00
Use Transfer Packet for Event Velocity

Bei aller Liebe zum Reformatting: Es macht mir den PR hier etwas dezent unleserlich!

Lixfel commented on pull request SteamWar/SteamWar#37 2025-04-12 14:52:10 +02:00
Improve and fix the discord rate limit warnings

Die ganze Funktion (die durchaus länger blockieren kann) als synchronized ist ungeil.

Lixfel commented on pull request SteamWar/SteamWar#37 2025-04-12 14:52:10 +02:00
Improve and fix the discord rate limit warnings

Hast du überprüft, ob es andere IP-basierte Webhook-restriktionen gibt? Wenn wir da plötzlich mit einer Menge Webhooks ankommen...

Lixfel commented on pull request SteamWar/SteamWar#37 2025-04-12 14:52:10 +02:00
Improve and fix the discord rate limit warnings

Bitte umsortieren, sodass das Argument im AllArgsConstructor wie auch den anderen Konstruktoren hinten dran ist.