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

Loot Probability Command #483

Closed
wants to merge 9 commits into from
Closed

Loot Probability Command #483

wants to merge 9 commits into from

Conversation

twobiers
Copy link
Collaborator

@twobiers twobiers commented Nov 1, 2024

Guck mal Mama, ich mach was mit staging environment

Copy link
Contributor

github-actions bot commented Nov 1, 2024

Hallo, hier kannst du jetzt deine tolle neue Funktion testen: https://discord.gg/pJjfF6n2ec

};

async function getDropWeightAdjustments(
export async function adjustLootWithBuffsFromUser(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Der Purpose der Funktion sind generell Adjustments in der Situation des Klickens. Das können Buffs, aber auch Debufffs durch aktuelle Items sein. Später war gedacht, die aktulle Uhrzeit des Klickens oder den schenkenden User ("Schenker X hat Item Y, deshalb ist Warhscheinlichkeit Z höher/geringer") etc.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mit "Buffs" meinte ich tatsächlich auch beides. Vielleicht git's da eine bessere Bezeichnung

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weight Adjustments

Comment on lines -263 to +271
weights: newWeights,
loot: newLoot,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Früher war das so und es hatte irgendeinen Grund, warum ich das umgebaut habe. Kann dir nicht genau sagen, welchen

.join("\n");

await interaction.reply(
`Deine persönlichen Loot Wahrscheinlichkeiten:\n\n${textRepresentation}`,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mach mal ephemeral.

@holzmaster
Copy link
Collaborator

Ich halte von dem gesamten Feature nicht so viel. Das ist eher Informatiker Debug-Tool, als Spielmechanik. Gibt auch einen Grund, warum andere Discord-Game-Bots sowas nicht haben (und auch keine Liste, die alle verfügbaren Items auflistet). Wäre deshalb dafür, es als Mod-Command zu machen.

Später wirds auch Dinge kompliziert machen, wenn die Weights nicht mehr so deterministisch sind. Da werden dann die returnten Wahrscheinlichkeiten nicht mehr stimmen. Zumal die Wahrscheinlichkeiten als Prozent intuitiv irreführend sind. Wenn ein User liest, dass er doppelte Wahrscheinlichkeit auf AU hat, aber die Prozentpunkte sich nicht verdoppeln, denkt er, es sei ein Fehler.

Copy link
Contributor

github-actions bot commented Nov 1, 2024

Hallo, hier kannst du jetzt deine tolle neue Funktion testen: https://discord.gg/pJjfF6n2ec

@twobiers
Copy link
Collaborator Author

twobiers commented Nov 1, 2024

Der Sinn darin liegt für mich darin Transparenz zu schaffen. Die Realität ist, dass wir keine ausgeklügelte Spielmechanik haben und die Verteilung auch nicht durchdacht ist. Mit der Gewichtung hat man eine Diskussiongrundlage zur Anpassung -- Soviel zum debugging. Nebeneffekt ist, dass du auch Feedback darüber kriegst wie rar dein Drop war ("Hey schau mal ich hab ne Powerade, die ist super selten"). Als Mod Command will ich das nicht bauen, weil es dann nicht mehr transparent ist und den Sinn verfehlt.
"Es gibt einen Grund wieso andere Server die Funktion nicht haben" halte ich für kein gutes Argument. Ohne den Grund zu benennen ist es für mich äquivalent zu "Es gibt einen Grund wieso Netflix Microservices macht, deshalb machen wir das jetzt auch".

Ich weiß nicht in welche Richtung du das Spiel weiterentwickeln willst, deshalb liegt die Entscheidung bei dir.

@holzmaster
Copy link
Collaborator

Der Sinn darin liegt für mich darin Transparenz zu schaffen.

Genau das will man bei Glücksspiel nicht :harold:

Die Realität ist, dass wir keine ausgeklügelte Spielmechanik haben und die Verteilung auch nicht durchdacht ist.

Das stimmt, aber daran ändert auch ne Übersicht nichts?

Nebeneffekt ist, dass du auch Feedback darüber kriegst wie rar dein Drop war ("Hey schau mal ich hab ne Powerade, die ist super selten").

Dann lass uns doch die Wahrscheinlichkeit anzeigen, die der Drop hatte. Beim Drop selbst im Footer (statt der Buff-/Debuff-Info). Die ist dann auch immer korrekt.

Ohne den Grund zu benennen

Der Grund ist, dass man bei Glücksspiel intransparenz haben will, weshalb es bei actual Glücksspiel Gesetze gibt, die genau das verhindern.

@holzmaster
Copy link
Collaborator

Der Grund ist, dass man bei Glücksspiel intransparenz haben will, weshalb es bei actual Glücksspiel Gesetze gibt, die genau das verhindern.

Falls der Hintergrund dazu nicht klar ist: Das öffnet mehr Möglichkeiten für Konditionierung und Freiheiten bei der Implementierung. Ich persönlcih find auch den Effekt ganz cool, wenn jemand einen Gegenstand "entdeckt", wenn er ihn das erste mal sieht oder bekommt. Letzteres verpufft, wenn er einfach ne Liste mit allen Dingen bekommt.

@twobiers twobiers closed this Nov 1, 2024
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