PR

【今日中】DAU集計時にチュートリアル未突破ユーザ除外 #4470

wada opened this at 2026-01-16T10:06:26; closed

Description

opened by wada at 2026-01-16T10:06:26; updated 2026-01-16T14:50:49

概要

dau_login にはチュートリアル未突破でもデータが入ってしまうため、cpanel_daily_active_user の集計結果の信頼性が低い
集計時に未突破を除外することで一旦対応

対応表

その他のテーブルに関しては、以下の順で対応予定
dau_login のチュートリアル中呼び出しを見直し
dau_login の再集計
③見直し前のデータを再集計

<span style="color:red;">対応はゲソてんのみ</span>

Comment

#43916 by wada at 2026-01-16T10:06:35; updated 2026-01-16T10:06:35

@kouho
レビューお願いします。

kouho add_label at 2026-01-16T10:56:42: 皇甫 LGTM

Comment

#43919 by kouho at 2026-01-16T10:57:49; updated 2026-01-16T11:11:18

@wada
~~~ 確認OKです~~~

kouho delete_label at 2026-01-16T11:06:43: 皇甫 LGTM

Comment

#43921 by kouho at 2026-01-16T11:15:25; updated 2026-01-16T11:15:25

@wada
チュートリアル突破の判定をuser_info.tutorial 1000 ではなく、user_setting.tutorial_fin_time <= 集計日にできませんか?
現状再集計時点でチュートリアル突破しているかの判定になり、修正意図とずれているように思いました。

kouho add_label at 2026-01-16T11:15:28: コメントあり

Comment

#43926 by wada at 2026-01-16T12:15:01; updated 2026-01-16T12:15:01

@kouho
確かにそうですね。再集計スクリプトもあわせて修正しました。

> チュートリアル突破の判定をuser_info.tutorial 1000 ではなく

kouho add_label at 2026-01-16T14:08:04: 皇甫 LGTM

Comment

#43931 by kouho at 2026-01-16T14:08:22; updated 2026-01-16T14:08:22

@wada
対応ありがとうございます。確認OKです。

wada merge at 2026-01-16T14:50:43: 【今日中】DAU集計時にチュートリアル未突破ユーザ除外
wada close at 2026-01-16T14:50:43: Close
wada delete_branch at 2026-01-16T14:50:49: fix_cpanel_dau

Commit range

base383e61db2d3e8ba609689ba709bb049e9f08ad84
head2e4ac6dd5a8508ab9ba978271f3c6bcf185e88d2
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_cpanel_dau

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedlib/exec/cpanel_daily_active_user.php
modifiedlib/exec/tool_recount_daily_active_user.php
Patch body is stored as a separate file to keep this page lightweight.