8000 [proto] 順位表の proto の命名を見直す · Issue #151 · szpp-dev-team/szpp-judge · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
[proto] 順位表の proto の命名を見直す #151
Open
@h-takeyeah

Description

@h-takeyeah

69行目のコメント通りです

message StandingsRecord {
message TaskDetail {
int32 task_id = 1;
int32 score = 2;
int32 penalty_count = 3;
optional int32 ac_submit_id = 4;
optional google.protobuf.Duration until_ac = 5; // コンテスト開始からの経過時間
}
int32 rank = 1;
string username = 2;
int32 total_score = 3;
int32 total_penalty_count = 4;
optional google.protobuf.Duration latest_ac_at = 5; // コンテスト開始からの経過時間(諸事情でよくない命名をしています)
repeated TaskDetail task_detail_list = 6;
}

またこれに関連する,フロントエンド側の TODO コメントも対応することを考えています.

latestAcAt?: number | bigint; // TODO: proto とともに名前を直す

latestAcAt: maxUntilAc, // TODO: proto とともに名前を直す 7084bd5

関連する PR など

  • StandingsElement.TaskDetail の ac_elapsed は AC の時刻を表わしているのか、コンテスト開始からACまでの経過時間を表しているのかどっち?
    • もし時刻を表わしているなら名前は ac_at もしくは ac_submit_at のように suffix at があったほうが分かりやすい (型で分かるけど)
    • もし経過時間を表わしているのなら型は Timestamp ではなく Duration な気がする
    • どちらにせよコメントは欲しい

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions

    0