Benutzer beschweren sich über Punktverlust

Victor_Bramante · 21. Dezember 2024 um 16:10

Hallo, ich versuche zu verstehen, wie das funktioniert.

Ich habe versucht, 2 Ranglisten einzurichten, um die Gewinner für November 2024 und Dezember 2024 zu ermitteln. Ich habe für beide feste Daten festgelegt (d. h. 01.11. bis 30.11.), und keine funktioniert richtig, weil:

Sie sind gezwungen, einen Standardzeitraum zu verwenden, auch wenn dieser als optional angegeben ist

Standardzeitraum-Option auf "Alle Zeiten" gesetzt307×105 2.4 KB
Wenn ich “Monatlich” wähle, werden die Punkte der letzten 30 Tage berechnet, auch mit festen Daten.
Wenn ich “Immer” wähle, werden alle Punkte summiert und die festen Daten umgangen.

Was muss ich tun, um die Punkte für November 2024 zu berechnen? Die Nutzer sehen, dass sie jeden Tag Punkte verlieren, und schicken mir deshalb beschwerende Nachrichten.

Außerdem wäre es cool, eine Option zu haben, um die Auswahl in der öffentlichen Ansicht der Rangliste auszublenden.

selase · 30. Dezember 2024 um 19:04

Die Beibehaltung des Standardzeitraums All Time sollte in Ordnung sein. All Time soll hier innerhalb der Grenzen der Von-/Bis-Daten der Rangliste liegen, falls diese festgelegt sind.

Es scheint jedoch ein Fehler vorzuliegen, bei dem die Filterung der Rangliste nach bestimmten Zeiträumen zu falschen Ergebnissen führt. Die Implementierung von score_period_condition berücksichtigt die Von-/Bis-Daten der Rangliste nicht richtig und steht im Konflikt mit der Hauptgrenzprüfung der Rangliste.

github.com/discourse/discourse-gamification

lib/discourse_gamification/leaderboard_cached_view.rb

03ea20da0


      
          def score_period_condtion(period)
            date =
              case period
              when "yearly"
                "CURRENT_DATE - INTERVAL '1 year'"
              when "monthly"
                "CURRENT_DATE - INTERVAL '1 month'"
              when "quarterly"
                "CURRENT_DATE - INTERVAL '3 month'"
              when "weekly"
                "CURRENT_DATE - INTERVAL '1 week'"
              when "daily"
                "CURRENT_DATE - INTERVAL '1 day'"
              else
                nil
              end
          
            date ? "AND gs.date >= #{date}" : ""
          end

github.com/discourse/discourse-gamification

lib/discourse_gamification/leaderboard_cached_view.rb

03ea20da0


      
          WHERE
            (
              -- Leaderboard with both "to/from" dates
              -- Only scores created between specified dates
              (
                lb.from_date IS NOT NULL
                AND lb.to_date IS NOT NULL
                AND gs.date BETWEEN lb.from_date AND lb.to_date
              )
              OR
              -- Leaderboard without "from/to" dates. All scores
              (lb.from_date IS NULL AND lb.to_date IS NULL)
              OR
              -- Leaderboard with just "from" date
              -- Only scores created starting from the specified date
              (
                lb.from_date IS NOT NULL
                AND lb.to_date IS NULL
                AND gs.date >= lb.from_date
              )

This file has been truncated. show original

Ich untersuche dies, eine Korrektur sollte bald verfügbar sein.

Victor_Bramante · 30. Dezember 2024 um 19:51

Danke, gib mir Bescheid, wenn du es tust, damit ich es noch einmal überprüfen kann.

Frohes neues Jahr!

Victor_Bramante · 13. Januar 2025 um 23:19

Hallo!

Wurde der Fix angewendet? Anscheinend zeigen meine Ranglisten vom Dezember keine konsistenten Ergebnisse an.

selase · 14. Januar 2025 um 11:30

Entschuldigung für die Verzögerung. Die Korrektur ist noch nicht angewendet, aber ich habe zwei ausstehende PRs, die zur Überprüfung bereit sind. Ich werde ein Update geben, sobald sie zusammengeführt wurden.

FIX: Redo period filtering to respect leaderboard date bounds by s3lase · Pull Request #178 · discourse/discourse-gamification · GitHub behebt die Abfrage der materialisierten Ansicht.
FEATURE: Add option to lock leaderboard view to default period by s3lase · Pull Request #179 · discourse/discourse-gamification · GitHub fügt eine Option zum Deaktivieren des Zeitperioden-Selectors hinzu.

selase · 20. Januar 2025 um 12:41

Vielen Dank für Ihre Geduld. Die Änderungen wurden zusammengeführt und stehen nun zur Überprüfung bereit. Bitte lassen Sie uns wissen, wenn Sie weitere Probleme feststellen oder Fragen haben.

system · 18. März 2025 um 20:29

This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.

Thema		Antworten	Aufrufe
Gamification leaderboard pagination doesn't respect the selected date period Bug gamification	2	552	4. Januar 2023
Setting a leaderboard date range should lock the default period Feature gamification	4	480	22. Januar 2025
Gamification period selector constraints UX gamification	1	348	9. März 2023
Leaderboard doesn't show score for recent periods Support gamification	5	350	12. Januar 2023
How to create a new leaderboard for a specific month? Support gamification	3	530	12. Januar 2023

Benutzer beschweren sich über Punktverlust

Verwandte Themen