[utils]受入/検査がスキップされたタスク履歴番号を取得するメソッドを追加 #108
Conversation
| index_list.append(index) | ||
| else: | ||
| # 直後の履歴がない | ||
| index_list.append(index) |
There was a problem hiding this comment.
最後の履歴が受入フェーズで担当者なしというのは、受入未着手も含まれるのでは？
There was a problem hiding this comment.
「受入フェーズで担当者なし」のタスクは、受入フェーズのタスク履歴がなかったため、問題ないかと思います。
「受入フェーズの担当者が割り当てられた」タスクは、受入フェーズのタスク履歴がありました。
There was a problem hiding this comment.
とはいえ、「started_datetimeとended_datetime`の両方がnullである」という条件を加えた方が確実だと思ったので、修正します。
annofabapi/utils.py
Outdated
| """ | ||
| index_list = [] | ||
| for index, history in enumerate(task_history_list): | ||
| if not (TaskPhase(history["phase"]) == TaskPhase.ACCEPTANCE and history["account_id"] is None): |
There was a problem hiding this comment.
作業時間が0であることも確認した方がよいのでは。ないと困る一例として、受入後のアノテーション一括編集とかがあります。
There was a problem hiding this comment.
メモ：受入完了状態のタスクに対してアノテーション一括編集すると、
- 受入フェーズが追加される
- 担当者はなし
- 作業時間は0でない
になる
annofabapi/utils.py
Outdated
| if index + 1 < len(task_history_list): | ||
| # 直後の履歴あり | ||
| next_history = task_history_list[index + 1] | ||
| # 直後の履歴が受入の場合、受入がスキップされた履歴（提出取り消しではない） |
There was a problem hiding this comment.
個人的な感想ですが、この説明の正しさを理解するのに少し時間がかかりました。
「直後が受入であること」ではなく「直後が前段のフェーズでないこと」とした方が、確かに提出取消ならそうなるはずだ、と理解しやすいのではないでしょうか。
There was a problem hiding this comment.
「直後が受入であること」ではなく「直後が前段のフェーズでないこと」とした方が、確かに提出取消ならそうなるはずだ、と理解しやすいのではないでしょうか。
「前段≠直前」で合っていますか？
There was a problem hiding this comment.
たとえば3番目の履歴の「前段フェーズ」は検査でしょうか？教師付でしょうか？
close #107