annofabcli 1.114.6__py3-none-any.whl → 1.114.7__py3-none-any.whl

annofabcli/annotation/change_annotation_attributes.py

@@ -46,7 +46,7 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
         service: annofabapi.Resource,
         *,
         project_id: str,
-        include_completed: bool,
+        include_complete_task: bool,
         all_yes: bool,
     ) -> None:
         self.service = service
@@ -54,7 +54,7 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
         CommandLineWithConfirm.__init__(self, all_yes)
 
         self.project_id = project_id
-        self.include_completed = include_completed
+        self.include_complete_task = include_complete_task
 
         self.dump_annotation_obj = DumpAnnotationMain(service, project_id)
 
@@ -141,9 +141,9 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
             logger.warning(f"task_id='{task_id}': タスクが作業中状態のため、スキップします。")
             return False, 0
 
-        if not self.include_completed:  # noqa: SIM102
+        if not self.include_complete_task:  # noqa: SIM102
             if task.status == TaskStatus.COMPLETE:
-                logger.warning(f"task_id='{task_id}': タスクが完了状態のため、スキップします。完了状態のタスクのアノテーション属性値を変更するには、 ``--include_completed`` を指定してください。")
+                logger.warning(f"task_id='{task_id}': タスクが完了状態のため、スキップします。完了状態のタスクのアノテーション属性値を変更するには、 ``--include_complete_task`` を指定してください。")
                 return False, 0
 
         annotation_list = self.get_annotation_list_for_task(task_id, annotation_query)
@@ -314,15 +314,15 @@ class ChangeAttributesOfAnnotation(CommandLine):
             backup_dir = Path(args.backup)
 
         super().validate_project(project_id, [ProjectMemberRole.OWNER, ProjectMemberRole.ACCEPTER])
-        if args.include_completed:  # noqa: SIM102
+        if args.include_complete_task:  # noqa: SIM102
             if not self.facade.contains_any_project_member_role(project_id, [ProjectMemberRole.OWNER]):
                 print(  # noqa: T201
-                    f"{self.COMMON_MESSAGE} argument --include_completed : '--include_completed' 引数を利用するにはプロジェクトのオーナーロールを持つユーザーで実行する必要があります。",
+                    f"{self.COMMON_MESSAGE} argument --include_complete_task : '--include_complete_task' 引数を利用するにはプロジェクトのオーナーロールを持つユーザーで実行する必要があります。",
                     file=sys.stderr,
                 )
                 sys.exit(COMMAND_LINE_ERROR_STATUS_CODE)
 
-        main_obj = ChangeAnnotationAttributesMain(self.service, project_id=project_id, include_completed=args.include_completed, all_yes=args.yes)
+        main_obj = ChangeAnnotationAttributesMain(self.service, project_id=project_id, include_complete_task=args.include_complete_task, all_yes=args.yes)
         main_obj.change_annotation_attributes_for_task_list(
             task_id_list,
             annotation_query=annotation_query,
@@ -362,7 +362,7 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
     )
 
     parser.add_argument(
-        "--include_completed",
+        "--include_complete_task",
         action="store_true",
         help="完了状態のタスクのアノテーション属性も変更します。ただし、オーナーロールを持つユーザーでしか実行できません。",
     )

annofabcli/annotation/create_classification_annotation.py

@@ -33,7 +33,8 @@ class CreateClassificationAnnotationMain(CommandLineWithConfirm):
         project_id: str,
         all_yes: bool,
         is_change_operator_to_me: bool,
-        include_completed: bool,
+        include_complete_task: bool,
+        include_on_hold_task: bool,
     ) -> None:
         self.service = service
         self.facade = AnnofabApiFacade(service)
@@ -41,7 +42,8 @@ class CreateClassificationAnnotationMain(CommandLineWithConfirm):
 
         self.project_id = project_id
         self.is_change_operator_to_me = is_change_operator_to_me
-        self.include_completed = include_completed
+        self.include_complete_task = include_complete_task
+        self.include_on_hold_task = include_on_hold_task
 
         # アノテーション仕様を取得
         annotation_specs_v3, _ = self.service.api.get_annotation_specs(self.project_id, query_params={"v": "3"})
@@ -69,10 +71,18 @@ class CreateClassificationAnnotationMain(CommandLineWithConfirm):
             logger.info(f"タスク'{task_id}'は作業中状態のため、全体アノテーションの作成をスキップします。")
             return None, False, None
 
-        if not self.include_completed:  # noqa: SIM102
+        if not self.include_complete_task:  # noqa: SIM102
             if task["status"] == TaskStatus.COMPLETE.value:
                 logger.info(
-                    f"タスク'{task_id}'は完了状態のため、全体アノテーションの作成をスキップします。完了状態のタスクに全体アノテーションを作成するには、 ``--include_completed`` を指定してください。"
+                    f"タスク'{task_id}'は完了状態のため、全体アノテーションの作成をスキップします。"
+                    f"完了状態のタスクに全体アノテーションを作成するには、 ``--include_complete_task`` を指定してください。"
+                )
+                return None, False, None
+
+        if not self.include_on_hold_task:  # noqa: SIM102
+            if task["status"] == TaskStatus.ON_HOLD.value:
+                logger.info(
+                    f"タスク'{task_id}'は保留状態のため、全体アノテーションの作成をスキップします。保留状態のタスクに全体アノテーションを作成するには、 ``--include_on_hold_task`` を指定してください。"
                 )
                 return None, False, None
 
@@ -324,11 +334,11 @@ class CreateClassificationAnnotation(CommandLine):
 
         super().validate_project(project_id, [ProjectMemberRole.OWNER, ProjectMemberRole.ACCEPTER])
 
-        if args.include_completed:  # noqa: SIM102
+        if args.include_complete_task:  # noqa: SIM102
             if not self.facade.contains_any_project_member_role(project_id, [ProjectMemberRole.OWNER]):
                 print(  # noqa: T201
-                    "annofabcli annotation create_classification: error: argument --include_completed : "
-                    "'--include_completed' 引数を利用するにはプロジェクトのオーナーロールを持つユーザーで実行する必要があります。",
+                    "annofabcli annotation create_classification: error: argument --include_complete_task : "
+                    "'--include_complete_task' 引数を利用するにはプロジェクトのオーナーロールを持つユーザーで実行する必要があります。",
                     file=sys.stderr,
                 )
                 sys.exit(COMMAND_LINE_ERROR_STATUS_CODE)
@@ -341,7 +351,8 @@ class CreateClassificationAnnotation(CommandLine):
             project_id=project_id,
             all_yes=self.all_yes,
             is_change_operator_to_me=args.change_operator_to_me,
-            include_completed=args.include_completed,
+            include_complete_task=args.include_complete_task,
+            include_on_hold_task=args.include_on_hold_task,
         )
 
         main_obj.main(task_ids, labels, parallelism=args.parallelism)
@@ -377,11 +388,17 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
     )
 
     parser.add_argument(
-        "--include_completed",
+        "--include_complete_task",
         action="store_true",
         help="完了状態のタスクにも全体アノテーションを作成します。ただし、オーナーロールを持つユーザーでしか実行できません。",
     )
 
+    parser.add_argument(
+        "--include_on_hold_task",
+        action="store_true",
+        help="保留状態のタスクにも全体アノテーションを作成します。",
+    )
+
     parser.add_argument(
         "--parallelism",
         type=int,

annofabcli/annotation/delete_annotation.py

@@ -46,7 +46,7 @@ class DeleteAnnotationMain(CommandLineWithConfirm):
     """アノテーション削除処理用のクラス
 
     Args:
-        is_force: 完了状態のタスクを削除するかどうか
+        include_complete_task: 完了状態のタスクを削除するかどうか
     """
 
     def __init__(
@@ -54,12 +54,12 @@ class DeleteAnnotationMain(CommandLineWithConfirm):
         service: annofabapi.Resource,
         project_id: str,
         *,
-        is_force: bool,
+        include_complete_task: bool,
         all_yes: bool,
     ) -> None:
         self.service = service
         self.facade = AnnofabApiFacade(service)
-        self.is_force = is_force
+        self.include_complete_task = include_complete_task
         CommandLineWithConfirm.__init__(self, all_yes)
         self.project_id = project_id
         self.dump_annotation_obj = DumpAnnotationMain(service, project_id)
@@ -133,9 +133,9 @@ class DeleteAnnotationMain(CommandLineWithConfirm):
             logger.info(f"task_id='{task_id}' :: タスクが作業中状態のため、スキップします。")
             return
 
-        if not self.is_force:  # noqa: SIM102
+        if not self.include_complete_task:  # noqa: SIM102
             if task.status == TaskStatus.COMPLETE:
-                logger.info(f"task_id='{task_id}' :: タスクが完了状態のため、スキップします。完了状態のタスクのアノテーションを削除するには、`--include_completed`オプションを指定してください。")
+                logger.info(f"task_id='{task_id}' :: タスクが完了状態のため、スキップします。完了状態のタスクのアノテーションを削除するには、`--include_complete_task`オプションを指定してください。")
                 return
 
         annotation_list = self.get_annotation_list_for_task(task_id, annotation_query=annotation_query)
@@ -182,7 +182,7 @@ class DeleteAnnotationMain(CommandLineWithConfirm):
             if task.status == TaskStatus.WORKING:
                 continue
 
-            if not self.is_force and task.status == TaskStatus.COMPLETE:
+            if not self.include_complete_task and task.status == TaskStatus.COMPLETE:
                 continue
 
             # アノテーション一覧を取得して、削除対象があるかチェック
@@ -316,11 +316,11 @@ class DeleteAnnotationMain(CommandLineWithConfirm):
                 failed_to_delete_annotation_count += annotation_count
                 continue
 
-            if not self.is_force:  # noqa: SIM102
+            if not self.include_complete_task:  # noqa: SIM102
                 if task["status"] == TaskStatus.COMPLETE.value:
                     logger.info(
                         f"task_id='{task_id}' :: タスクが完了状態のため、アノテーション {annotation_count} 件の削除をスキップします。"
-                        f"完了状態のタスクのアノテーションを削除するには、`--include_completed`オプションを指定してください。"
+                        f"完了状態のタスクのアノテーションを削除するには、`--include_complete_task`オプションを指定してください。"
                     )
                     failed_to_delete_annotation_count += annotation_count
                     continue
@@ -376,14 +376,14 @@ class DeleteAnnotation(CommandLine):
         else:
             backup_dir = Path(args.backup)
 
-        if args.include_completed:
-            # --include_completedオプションが指定されている場合は、完了状態のタスクも削除する
-            # 完了状態のタスクを削除するには、オーナーロールである必要があるため、`args.include_completed`で条件を分岐する
+        if args.include_complete_task:
+            # --include_complete_taskオプションが指定されている場合は、完了状態のタスクも削除する
+            # 完了状態のタスクを削除するには、オーナーロールである必要があるため、`args.include_complete_task`で条件を分岐する
             super().validate_project(project_id, [ProjectMemberRole.OWNER])
         else:
             super().validate_project(project_id, [ProjectMemberRole.OWNER, ProjectMemberRole.ACCEPTER])
 
-        main_obj = DeleteAnnotationMain(self.service, project_id, all_yes=args.yes, is_force=args.include_completed)
+        main_obj = DeleteAnnotationMain(self.service, project_id, all_yes=args.yes, include_complete_task=args.include_complete_task)
 
         if args.json is not None:
             dict_annotation_list = get_json_from_args(args.json)
@@ -479,7 +479,7 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
     )
 
     parser.add_argument(
-        "--include_completed",
+        "--include_complete_task",
         action="store_true",
         help="指定した場合は、完了状態のタスクのアノテーションも削除します。ただし、完了状態のタスクを削除するには、オーナーロールを持つユーザーが実行する必要があります。",
     )
@@ -499,7 +499,7 @@ def add_parser(subparsers: argparse._SubParsersAction | None = None) -> argparse
         "タスク配下のアノテーションを削除します。ただし、作業中状態のタスクのアノテーションは削除できません。"
         "間違えてアノテーションを削除したときに復元できるようにするため、 ``--backup`` でバックアップ用のディレクトリを指定することを推奨します。"
     )
-    epilog = "オーナーまたはチェッカーロールを持つユーザで実行してください。ただし``--include_completed``オプションを指定した場合は、オーナーロールを持つユーザで実行してください。"
+    epilog = "オーナーまたはチェッカーロールを持つユーザで実行してください。ただし``--include_complete_task``オプションを指定した場合は、オーナーロールを持つユーザで実行してください。"
 
     parser = annofabcli.common.cli.add_parser(subparsers, subcommand_name, subcommand_help, description, epilog=epilog)
     parse_args(parser)

annofabcli/annotation/import_annotation.py

@@ -410,9 +410,11 @@ class ImportAnnotationMain(CommandLineWithConfirm):
         *,
         project_id: str,
         all_yes: bool,
-        is_force: bool,
+        change_operator_to_me: bool,
         is_merge: bool,
         is_overwrite: bool,
+        include_complete_task: bool,
+        include_on_hold_task: bool,
         converter: AnnotationConverter,
     ) -> None:
         self.service = service
@@ -420,9 +422,11 @@ class ImportAnnotationMain(CommandLineWithConfirm):
         CommandLineWithConfirm.__init__(self, all_yes)
 
         self.project_id = project_id
-        self.is_force = is_force
+        self.change_operator_to_me = change_operator_to_me
         self.is_merge = is_merge
         self.is_overwrite = is_overwrite
+        self.include_complete_task = include_complete_task
+        self.include_on_hold_task = include_on_hold_task
         self.converter = converter
 
     def put_annotation_for_input_data(self, parser: SimpleAnnotationParser) -> int:
@@ -487,7 +491,7 @@ class ImportAnnotationMain(CommandLineWithConfirm):
 
         return success_input_data_count, success_annotation_count
 
-    def execute_task(self, task_parser: SimpleAnnotationParserByTask, task_index: int | None = None) -> bool:
+    def execute_task(self, task_parser: SimpleAnnotationParserByTask, task_index: int | None = None) -> bool:  # noqa: PLR0911
         """
         1個のタスクに対してアノテーションを登録する。
 
@@ -499,26 +503,39 @@ class ImportAnnotationMain(CommandLineWithConfirm):
 
         """
         task_id = task_parser.task_id
-        if not self.confirm_processing(f"task_id='{task_id}' のアノテーションをインポートしますか？"):
-            return False
-
-        logger_prefix = f"{task_index + 1!s} 件目: " if task_index is not None else ""
-        logger.info(f"{logger_prefix}task_id='{task_id}' に対して処理します。")
+        logger_prefix = f"{task_index + 1!s} 件目 :: task_id='{task_id}' :: " if task_index is not None else ""
 
         task = self.service.wrapper.get_task_or_none(self.project_id, task_id)
         if task is None:
-            logger.warning(f"task_id='{task_id}'であるタスクは存在しません。")
+            logger.warning(f"{logger_prefix}タスクは存在しません。")
+            return False
+
+        logger.debug(f"{logger_prefix}phase='{task['phase']}', status='{task['status']}'")
+        if task["status"] == TaskStatus.WORKING.value:
+            logger.info(f"{logger_prefix}タスクは作業中のため、処理をスキップします。 :: status={task['status']}")
+            return False
+
+        if not self.include_complete_task and task["status"] == TaskStatus.COMPLETE.value:
+            logger.info(f"{logger_prefix}タスクは完了状態のため、処理をスキップします。完了状態のタスクを処理する場合は、 '--include_complete_task'を指定してください。 :: status={task['status']}")
             return False
 
-        if task["status"] in [TaskStatus.WORKING.value, TaskStatus.COMPLETE.value]:
-            logger.info(f"タスク'{task_id}'は作業中または受入完了状態のため、インポートをスキップします。 status={task['status']}")
+        if not self.include_on_hold_task and task["status"] == TaskStatus.ON_HOLD.value:
+            logger.info(
+                f"{logger_prefix}タスクは保留中状態のため、処理をスキップします。保留中状態のタスクにアノテーションをインポートする場合は、 "
+                f"'--include_on_hold_task'を指定してください。 :: status={task['status']}"
+            )
             return False
 
+        if not self.confirm_processing(f"task_id='{task_id}'のタスク（phase={task['phase']}, status={task['status']}）にアノテーションをインポートしますか？"):
+            return False
+
+        logger.info(f"{logger_prefix}タスクにアノテーションをインポートします。")
+
         old_account_id: str | None = None
         changed_operator = False
-        if self.is_force:
+        if self.change_operator_to_me:
             if not can_put_annotation(task, self.service.api.account_id):
-                logger.debug(f"タスク'{task_id}' の担当者を自分自身に変更します。")
+                logger.debug(f"{logger_prefix}担当者を自分自身に変更します。")
                 old_account_id = task["account_id"]
                 task = self.service.wrapper.change_task_operator(
                     self.project_id,
@@ -532,7 +549,7 @@ class ImportAnnotationMain(CommandLineWithConfirm):
             if not can_put_annotation(task, self.service.api.account_id):
                 logger.debug(
                     f"タスク'{task_id}'は、過去に誰かに割り当てられたタスクで、現在の担当者が自分自身でないため、アノテーションのインポートをスキップします。"
-                    f"担当者を自分自身に変更してアノテーションを登録する場合は `--force` を指定してください。"
+                    f"担当者を自分自身に変更してアノテーションを登録する場合は `--change_operator_to_me` を指定してください。"
                 )
                 return False
 
@@ -678,7 +695,9 @@ class ImportAnnotation(CommandLine):
             all_yes=self.all_yes,
             is_merge=args.merge,
             is_overwrite=args.overwrite,
-            is_force=args.force,
+            change_operator_to_me=args.change_operator_to_me,
+            include_complete_task=args.include_complete_task,
+            include_on_hold_task=args.include_on_hold_task,
             converter=converter,
         )
 
@@ -700,7 +719,7 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
         "--annotation",
         type=Path,
         required=True,
-        help="Simpleアノテーションと同じフォルダ構成のzipファイル or ディレクトリのパスを指定してください。タスクの状態が作業中/完了の場合はインポートしません。",
+        help="Simpleアノテーションと同じフォルダ構成のzipファイル or ディレクトリのパスを指定してください。",
     )
 
     argument_parser.add_task_id(required=False)
@@ -722,9 +741,23 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
     )
 
     parser.add_argument(
-        "--force",
+        "--change_operator_to_me",
         action="store_true",
-        help="過去に割り当てられていて現在の担当者が自分自身でない場合、タスクの担当者を自分自身に変更してからアノテーションをインポートします。",
+        help="タスクの担当者を自分自身に変更しないとアノテーションをインポートできない場合（担当者が自分自身でない AND 担当者が割れ当てられたことがあるタスク）は、タスクの担当者を自分自身に変更します。アノテーションをインポートが完了したら、担当者を元に戻します。"  # noqa: E501
+        "未指定の場合は、そのようなタスクのアノテーションインポートはスキップされます。",
+    )
+
+    parser.add_argument(
+        "--include_complete_task",
+        action="store_true",
+        help="完了状態のタスクに対してもアノテーションをインポートします。未指定の場合は、完了状態のタスクはスキップされます。",
+    )
+
+    parser.add_argument(
+        "--include_on_hold_task",
+        action="store_true",
+        help="保留中状態のタスクに対してもアノテーションをインポートします。ただし、アノテーションインポート後は保留中状態でなくなる可能性があります。"
+        "未指定の場合は、保留中状態のタスクはスキップされます。",
     )
 
     parser.add_argument(
@@ -749,7 +782,12 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
 def add_parser(subparsers: argparse._SubParsersAction | None = None) -> argparse.ArgumentParser:
     subcommand_name = "import"
     subcommand_help = "アノテーションをインポートします。"
-    description = "アノテーションをインポートします。アノテーションのフォーマットは、Simpleアノテーションと同じフォルダ構成のzipファイルまたはディレクトリです。ただし、作業中/完了状態のタスクはインポートできません。"  # noqa: E501
+    description = (
+        "アノテーションをインポートします。アノテーションのフォーマットは、Simpleアノテーションと同じフォルダ構成のzipファイルまたはディレクトリです。"
+        "ただし、作業中状態のタスクはインポートできません。"
+        "``--include_complete_task`` を指定すれば、完了状態のタスクにもインポートできます。"
+        "``--include_on_hold_task`` を指定すれば、保留中状態のタスクにもインポートできます。"
+    )
     epilog = "オーナロールを持つユーザで実行してください。"
 
     parser = annofabcli.common.cli.add_parser(subparsers, subcommand_name, subcommand_help, description, epilog=epilog)

annofabcli/comment/put_comment.py

@@ -259,7 +259,7 @@ class PutCommentMain(CommandLineWithConfirm):
         task_id: str,
         comments_for_task: AddedCommentsForTask,
         task_index: int | None = None,
-    ) -> int:
+    ) -> tuple[int, int]:
         """
         タスクにコメントを付与します。
 
@@ -269,28 +269,29 @@ class PutCommentMain(CommandLineWithConfirm):
             task_index: タスクの連番
 
         Returns:
-            コメントを付与した入力データの個数
+            (コメントを付与した入力データの個数, 付与したコメントの個数)のタプル
         """
         logging_prefix = f"{task_index + 1} 件目" if task_index is not None else ""
 
         task = self.service.wrapper.get_task_or_none(self.project_id, task_id)
         if task is None:
             logger.warning(f"{logging_prefix} :: task_id='{task_id}' のタスクは存在しないので、スキップします。")
-            return 0
+            return (0, 0)
 
         logger.debug(f"{logging_prefix} : task_id='{task['task_id']}', status='{task['status']}', phase='{task['phase']}'")
 
         if not self._can_add_comment(
             task=task,
         ):
-            return 0
+            return (0, 0)
 
         if not self.confirm_processing(f"task_id='{task_id}' のタスクに{self.comment_type_name}を付与しますか？"):
-            return 0
+            return (0, 0)
 
         # コメントを付与するには作業中状態にする必要がある
         changed_task = self.change_to_working_status(self.project_id, task)
-        added_comments_count = 0
+        added_input_data_count = 0
+        added_comment_count = 0
         for input_data_id, comments in comments_for_task.items():
             if input_data_id not in task["input_data_id_list"]:
                 logger.warning(f"{logging_prefix} :: task_id='{task_id}'のタスクに input_data_id='{input_data_id}'の入力データは存在しません。")
@@ -300,7 +301,8 @@ class PutCommentMain(CommandLineWithConfirm):
                 if len(comments) > 0:
                     request_body = self._create_request_body(task=changed_task, input_data_id=input_data_id, comments=comments)
                     self.service.api.batch_update_comments(self.project_id, task_id, input_data_id, request_body=request_body)
-                    added_comments_count += 1
+                    added_input_data_count += 1
+                    added_comment_count += len(comments)
                     logger.debug(f"{logging_prefix} :: task_id='{task_id}', input_data_id='{input_data_id}' :: {len(comments)}件のコメントを付与しました。")
             except Exception:  # pylint: disable=broad-except
                 logger.warning(
@@ -314,12 +316,12 @@ class PutCommentMain(CommandLineWithConfirm):
             self.service.wrapper.change_task_operator(self.project_id, task_id, task["account_id"])
             logger.debug(f"{logging_prefix} :: task_id='{task_id}' :: 担当者を元のユーザ( account_id='{task['account_id']}'）に戻しました。")
 
-        return added_comments_count
+        return (added_input_data_count, added_comment_count)
 
     def add_comments_for_task_wrapper(
         self,
         tpl: tuple[int, tuple[str, AddedCommentsForTask]],
-    ) -> int:
+    ) -> tuple[int, int]:
         task_index, (task_id, comments_for_task) = tpl
         return self.add_comments_for_task(task_id=task_id, comments_for_task=comments_for_task, task_index=task_index)
 
@@ -328,30 +330,45 @@ class PutCommentMain(CommandLineWithConfirm):
         comments_for_task_list: AddedComments,
         parallelism: int | None = None,
     ) -> None:
-        comments_count = sum(len(e) for e in comments_for_task_list.values())
-        logger.info(f"{self.comment_type_name}を付与するタスク数: {len(comments_for_task_list)}, {self.comment_type_name}を付与する入力データ数: {comments_count}")
+        tasks_count = len(comments_for_task_list)
+        input_data_count = sum(len(e) for e in comments_for_task_list.values())
+        total_comment_count = sum(len(comments) for comments_for_task in comments_for_task_list.values() for comments in comments_for_task.values())
+        logger.info(
+            f"{self.comment_type_name}を付与するタスク数: {tasks_count}, {self.comment_type_name}を付与する入力データ数: {input_data_count}, {self.comment_type_name}の総数: {total_comment_count}"
+        )
 
         if parallelism is not None:
             with multiprocessing.Pool(parallelism) as pool:
-                result_bool_list = pool.map(self.add_comments_for_task_wrapper, enumerate(comments_for_task_list.items()))
-                added_comments_count = sum(e for e in result_bool_list)
+                result_list = pool.map(self.add_comments_for_task_wrapper, enumerate(comments_for_task_list.items()))
+                added_input_data_count = sum(e[0] for e in result_list)
+                added_comment_count = sum(e[1] for e in result_list)
+                succeeded_tasks_count = sum(1 for e in result_list if e[0] > 0)
 
         else:
             # 逐次処理
-            added_comments_count = 0
+            added_input_data_count = 0
+            added_comment_count = 0
+            succeeded_tasks_count = 0
             for task_index, (task_id, comments_for_task) in enumerate(comments_for_task_list.items()):
                 try:
-                    result = self.add_comments_for_task(
+                    result_input_data, result_comment = self.add_comments_for_task(
                         task_id=task_id,
                         comments_for_task=comments_for_task,
                         task_index=task_index,
                     )
-                    added_comments_count += result
+                    added_input_data_count += result_input_data
+                    added_comment_count += result_comment
+                    if result_input_data > 0:
+                        succeeded_tasks_count += 1
                 except Exception:  # pylint: disable=broad-except
                     logger.warning(f"task_id='{task_id}' :: コメントの付与に失敗しました。", exc_info=True)
                     continue
 
-        logger.info(f"{added_comments_count} / {comments_count} 件の入力データに{self.comment_type_name}を付与しました。")
+        logger.info(
+            f"{succeeded_tasks_count} / {tasks_count} 件のタスク, "
+            f"{added_input_data_count} / {input_data_count} 件の入力データ, "
+            f"{added_comment_count} / {total_comment_count} 件の{self.comment_type_name}を付与しました。"
+        )
 
 
 def convert_cli_comments(dict_comments: dict[str, Any], *, comment_type: CommentType) -> AddedComments:

annofabcli/filesystem/draw_annotation.py

@@ -182,7 +182,7 @@ class DrawingAnnotationForOneImage:
                 image.save(output_file)
 
         elif image_size is not None:
-            image = Image.new("RGBA", image_size, color="black")
+            image = Image.new("RGBA", image_size, color="black")  # type: ignore[assignment]
             draw = ImageDraw.Draw(image)
             self._draw_annotations(draw, parser)
             output_file.parent.mkdir(parents=True, exist_ok=True)

annofabcli/instruction/copy_instruction.py

@@ -31,7 +31,7 @@ class CopyInstruction(CommandLine):
     @staticmethod
     def get_instruction_image_id_from_url(url: str) -> str:
         # URL Queryを除いたURLを取得する
-        url_without_query = url.split("?")[0]
+        url_without_query = url.split("?", maxsplit=1)[0]
         return url_without_query.split("/")[-1]
 
     @staticmethod

annofabcli/statistics/histogram.py

@@ -34,7 +34,7 @@ def get_bin_edges(min_value: float, max_value: float, bin_width: float) -> numpy
     # stop引数に、`bin_width*2`を指定している理由：
     # 引数が小数のときは`len(bin_edges)``期待通りにならないときがあるので、ビンの数を少し増やしている
     # https://qiita.com/yuji38kwmt/items/ff00f3cb9083567d083f
-    bin_edges = numpy.arange(start=min_value, stop=max_value + bin_width * 2, step=bin_width)
+    bin_edges = numpy.arange(start=min_value, stop=max_value + bin_width * 2, step=bin_width)  # type: ignore[call-overload]
     return bin_edges
 
 

annofabcli/task/change_operator.py

@@ -88,7 +88,7 @@ class ChangeOperatorMain:
 
         if task.status == TaskStatus.ON_HOLD and not self.include_on_hold:
             logger.warning(
-                f"{logging_prefix} :: task_id='{task_id}' :: タスクが保留中状態なので、担当者を変更できません。保留中状態のタスクの担当者も変更する場合は、'--include_on_hold'を指定してください。"
+                f"{logging_prefix} :: task_id='{task_id}' :: タスクが保留中状態なので、担当者を変更できません。保留中状態のタスクの担当者も変更する場合は、'--include_on_hold_task'を指定してください。"
             )
             return False
 
@@ -222,7 +222,7 @@ class ChangeOperator(CommandLine):
         project_id = args.project_id
         super().validate_project(project_id, [ProjectMemberRole.OWNER, ProjectMemberRole.ACCEPTER])
 
-        main_obj = ChangeOperatorMain(self.service, all_yes=self.all_yes, include_on_hold=args.include_on_hold)
+        main_obj = ChangeOperatorMain(self.service, all_yes=self.all_yes, include_on_hold=args.include_on_hold_task)
         main_obj.change_operator(
             project_id,
             task_id_list=task_id_list,
@@ -253,7 +253,7 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
     argument_parser.add_task_query()
 
     parser.add_argument(
-        "--include_on_hold",
+        "--include_on_hold_task",
         action="store_true",
         help="指定した場合、保留中のタスクの担当者も変更します。指定しない場合、保留中のタスクはスキップされます。",
     )

annofabcli/task/complete_tasks.py

@@ -179,9 +179,11 @@ class CompleteTasksMain(CommandLineWithConfirm):
             # 教師付フェーズになった日時より、後に付与された返信コメントを取得する
             answered_comment = first_true(
                 comment_list,
-                pred=lambda e: e["comment_node"]["_type"] == "Reply"
-                and e["comment_node"]["root_comment_id"] == parent_comment_id
-                and dateutil.parser.parse(e["created_datetime"]) >= dateutil.parser.parse(task_started_datetime),
+                pred=lambda e: (
+                    e["comment_node"]["_type"] == "Reply"
+                    and e["comment_node"]["root_comment_id"] == parent_comment_id
+                    and dateutil.parser.parse(e["created_datetime"]) >= dateutil.parser.parse(task_started_datetime)
+                ),
             )
             return answered_comment is not None
 

{annofabcli-1.114.6.dist-info → annofabcli-1.114.7.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: annofabcli
-Version: 1.114.6
+Version: 1.114.7
 Summary: Utility Command Line Interface for AnnoFab
 Author: Kurusugawa Computer Inc.
 License: MIT

{annofabcli-1.114.6.dist-info → annofabcli-1.114.7.dist-info}/RECORD

@@ -3,15 +3,15 @@ annofabcli/__main__.py,sha256=w3g-2v0rcR8yZwDglX2ZK3d2fZBY_RQ0VNZHGPzoAr8,5522
 annofabcli/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 annofabcli/annotation/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 annofabcli/annotation/annotation_query.py,sha256=iGpbg5L6sRona3ADsBfT4lyXmyEiD5i3ujYvPhiiTbM,15681
-annofabcli/annotation/change_annotation_attributes.py,sha256=zKuWsDfgwrOiUDOMydCQlNPg_P88o2f6DlLwY9hWjDU,17863
+annofabcli/annotation/change_annotation_attributes.py,sha256=G2g5R5Vb8udhxJlN-qO8-3E801WVGcpreOG-Qd12SbA,17907
 annofabcli/annotation/change_annotation_attributes_per_annotation.py,sha256=nbhosWabJl1t4exWantLJ-Z5lKL6HoeBdd6WOC7VGms,15491
 annofabcli/annotation/change_annotation_properties.py,sha256=d2bs_s0jTq3zeDYAE5LlbYdIeIaMe7UJ9xO6x0WuaJo,18274
 annofabcli/annotation/copy_annotation.py,sha256=8ayliCb3XCsswEW_E_blmSR1aLt00Swfq9_hejp0v5k,18413
-annofabcli/annotation/create_classification_annotation.py,sha256=OVMFyL-CrZH444V1EkaTt068cEG834LIfdMDAspn3PA,18577
-annofabcli/annotation/delete_annotation.py,sha256=NCA8e9FQsqaCVcQmteRQcLoNrzAcoRdpFoiWjZhswSY,25124
+annofabcli/annotation/create_classification_annotation.py,sha256=nryBN9aTiVpjCTHBrjjPnmwii3j17U_02CxOquHdV1I,19464
+annofabcli/annotation/delete_annotation.py,sha256=1ZPX49YKnK3nLKk9d_Xc59xWCYCPYGj56rtVHxwbSds,25260
 annofabcli/annotation/download_annotation_zip.py,sha256=o2p2TfK8-jz15TWrkWxhX9eAnoATHjxkTy44q-BbI14,3265
 annofabcli/annotation/dump_annotation.py,sha256=AWEAR4Q38Oo9yA6plo51F7_ZSxwfuaxQeFDZ66gsiIA,10844
-annofabcli/annotation/import_annotation.py,sha256=cbTLD2BZjZ2xCDvRriCHkVz2Twdnbm0r2igtLHw9tMI,35849
+annofabcli/annotation/import_annotation.py,sha256=rRLTILcy1e4wN2Acue9scIaLTg6pQ44z2H_CkyliP9A,38393
 annofabcli/annotation/list_annotation.py,sha256=NQ-G5Ip6Mk2tWLI0c5mR982MhgJNrvCnqskDjrelvJc,10675
 annofabcli/annotation/list_annotation_count.py,sha256=Zez7jkWOhhp3FwfKRM4VhyL26qdNuKYfQwb0Qm4tElk,6442
 annofabcli/annotation/merge_segmentation.py,sha256=BeLzOjDy6iOgp2z9GbFiUE_KRjv9UKFyczWl4_6aYDs,18096
@@ -48,7 +48,7 @@ annofabcli/comment/delete_comment.py,sha256=DT6s07IyCYXlSqEZop1CXloaORJVgcBvwR4B
 annofabcli/comment/download_comment_json.py,sha256=wB8U31G-7pkvN05N2erZch91bqx6oumGr9ziIXIyLuY,2392
 annofabcli/comment/list_all_comment.py,sha256=fFBBY6-9Tx51X7lVT6qblTfkQ7jWmZiqtNqkt-RB14c,8233
 annofabcli/comment/list_comment.py,sha256=SMTGCALxnddDxy-xnqOjBc3ijfb9QIZ-G4ZMw2BwyxA,6137
-annofabcli/comment/put_comment.py,sha256=lpisYmcSYRqygdpZFB_XmXFWjk82EaCNXYhGfQE3MYA,18646
+annofabcli/comment/put_comment.py,sha256=AdIaKcrfOmyyJMXgCN4TZk04TmMtBX-rV1mzs6RMC5Q,19700
 annofabcli/comment/put_comment_simply.py,sha256=2nWnv0ZzeemJ99h0c2mGqA1ydsfuim6CGJkY_Y-uPyg,8298
 annofabcli/comment/put_inspection_comment.py,sha256=XJt1fS-lWIH54PNGKzU8FLYUITtpEk6HCn7Wp47esgk,3944
 annofabcli/comment/put_inspection_comment_simply.py,sha256=sgDKoBXlAPcTaf4GpU4qtfB3VRADNn0cdr0Ritg54Bo,6749
@@ -79,7 +79,7 @@ annofabcli/experimental/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG
 annofabcli/experimental/list_out_of_range_annotation_for_movie.py,sha256=Ok3b1CNuTzzalC9u0wHSFwTO7Qvi4PZDx7TZu-TB84Y,8485
 annofabcli/experimental/subcommand_experimental.py,sha256=aKaimneErwv5epJaBpmAviuRz8O4I8eNSBvhr0JwSPI,862
 annofabcli/filesystem/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-annofabcli/filesystem/draw_annotation.py,sha256=L3fmOulT0PDm8rzrSoLoW1gcz8MKXgPshzWhavJdPiU,19197
+annofabcli/filesystem/draw_annotation.py,sha256=HG6YCEjJO9Brk0g7sqkv40DcktjxrnzeiZnkmcIIGjY,19225
 annofabcli/filesystem/filter_annotation.py,sha256=sV6d_YwHGFIGet0RldQ9GfNyUTfMGU_vYE-8cn3EapA,11095
 annofabcli/filesystem/mask_user_info.py,sha256=j29AmAIz06efsUBkjxbbCNKCaEb_PPFqW_n9T3yjIVM,13563
 annofabcli/filesystem/merge_annotation.py,sha256=JXjcwhcc_SgNW04rLNhqZCrmKoJeaeDnXtTPIJAM5gU,10502
@@ -99,7 +99,7 @@ annofabcli/input_data/update_input_data.py,sha256=dSvPcjgvdtWSFPA62Vi9a29-_v7x8X
 annofabcli/input_data/update_metadata_of_input_data.py,sha256=FbDVPoX0RwA3vX3a2pi_cpN5L6usrAXL6ZFEevvbTU8,11405
 annofabcli/input_data/utils.py,sha256=F3mYbWmRwXfEtiIo9dvSysfuZUdyQzJmmq94eM-C5IY,723
 annofabcli/instruction/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-annofabcli/instruction/copy_instruction.py,sha256=vkpdReK9CfIx5ll0l6MgnVRLop9UKcWX4GjXZQscUQs,6675
+annofabcli/instruction/copy_instruction.py,sha256=iA-PFZE-s79Hc-wWljvlI_lwR5PjH13NKiDluEJzHfY,6687
 annofabcli/instruction/download_instruction.py,sha256=m04SgChdwGSUZ_kNpZkLvPZFeYEs654wuzgZB_6Fo8g,8288
 annofabcli/instruction/list_instruction_history.py,sha256=PVcCgb1Ax8pfv_nKG-B38Xc5GjcZS7jT4-ItGCk57vg,2073
 annofabcli/instruction/subcommand_instruction.py,sha256=RgMcjpIY6gq4gkqvV_55G0Wa7lKJNZKQAiX-WMSP98o,1120
@@ -149,7 +149,7 @@ annofabcli/stat_visualization/summarize_whole_performance_csv.py,sha256=tUt4O1sV
 annofabcli/stat_visualization/write_graph.py,sha256=DWRgJuflHaNfdXewdD_p7BQYSjqNZMNErpO6QWCYkFM,9067
 annofabcli/stat_visualization/write_performance_rating_csv.py,sha256=ElRk-9ZPkVK0rqj1UCuk9yLWsOy7TvIAGzRSk7ZsiVQ,30675
 annofabcli/statistics/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-annofabcli/statistics/histogram.py,sha256=hvGszTqC1TYmf0mXoR53MslHdH5wOQBVD_rX6Ygu-dE,3265
+annofabcli/statistics/histogram.py,sha256=lsLyLSmPdFERAbtuZS_UTGfOGliy9h_p4s9YQYWrnps,3296
 annofabcli/statistics/linegraph.py,sha256=D2qMBykZAfGN8lf5EaOsbpEqH4mzl5E5ga9CI_XH9go,12391
 annofabcli/statistics/list_annotation_area.py,sha256=GiEwrE-DuOUavAVDSb0l5qcgGkn3gGn5SAtpa8Mwhak,13234
 annofabcli/statistics/list_annotation_attribute.py,sha256=AH4OeqUhwCh19Kv93bFLZRV6o6oKwFklRdEw2FrOaVk,12669
@@ -197,10 +197,10 @@ annofabcli/supplementary/put_supplementary_data.py,sha256=VGieXgpGgF7qBZSULq7XP8
 annofabcli/supplementary/subcommand_supplementary.py,sha256=UnnHsxYDmHMGURkWAb-0pBeTYkSZ0YPKOuZW4XSE-RI,1030
 annofabcli/task/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 annofabcli/task/cancel_acceptance.py,sha256=AgCvl7UsbaLO8A_tMGEkFjU10WT9siOQXME4apGAbTk,13778
-annofabcli/task/change_operator.py,sha256=IETB7Pckmx75o6xtDrVqDBvxZqJBZP92xQmn-QzcJ5w,11717
+annofabcli/task/change_operator.py,sha256=A8ucvWgzyZOfrF2-yh6cPuDkjJBqOM3BRp6NATGhqhw,11732
 annofabcli/task/change_status_to_break.py,sha256=hBAPcM6TiXEODYAo8blJNYXuILtKgWlfexcKImdA9rg,8554
 annofabcli/task/change_status_to_on_hold.py,sha256=5mP6CJHlJxfQO4i-imuR-04lK_qAueFepbSaUgvGTsk,12905
-annofabcli/task/complete_tasks.py,sha256=CyQjMWlciHI3IQPm1uez1yFCzZ8IpgktC7grjsYrioI,25093
+annofabcli/task/complete_tasks.py,sha256=b0TDz8_A0SFBnZVM0GEdUDOGfzMlcvMa2Fpj-eYArLI,25141
 annofabcli/task/copy_tasks.py,sha256=cOWtQAXqB1YBKatF2wGMMx9tqQ8RWL-AexrLBecqzqI,8680
 annofabcli/task/delete_metadata_key_of_task.py,sha256=EywRT5LktrXzYM5H5I3EzbzbwdNOfaTFT0IOCPEmifQ,7962
 annofabcli/task/delete_tasks.py,sha256=221gB5gBNlZd4EOF0tkdIsbbsagx6fmPqavpnJwm2GI,13081
@@ -227,8 +227,8 @@ annofabcli/task_history_event/download_task_history_event_json.py,sha256=lWyOoS2
 annofabcli/task_history_event/list_all_task_history_event.py,sha256=51p_c_sWflJ7Jsrkay-OgF6c8wCOHpnqQeGIHVKzi6E,8001
 annofabcli/task_history_event/list_worktime.py,sha256=0X1irND_ElxC_XpHiPac6cjwIz9DhgAccfyDA3M7kh0,15481
 annofabcli/task_history_event/subcommand_task_history_event.py,sha256=dFllzpm8plnnwADwTV74h-R2LOA7rZW-xd2YnSkwTHo,1229
-annofabcli-1.114.6.dist-info/METADATA,sha256=d6zPaNaifi8TGvORYdMhH0pp8Ku6Y5Q2pj2OVB6zJYg,4946
-annofabcli-1.114.6.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
-annofabcli-1.114.6.dist-info/entry_points.txt,sha256=C2uSUc-kkLJpoK_mDL5FEMAdorLEMPfwSf8VBMYnIFM,56
-annofabcli-1.114.6.dist-info/licenses/LICENSE,sha256=pcqWYfxFtxBzhvKp3x9MXNM4xciGb2eFewaRhXUNHlo,1081
-annofabcli-1.114.6.dist-info/RECORD,,
+annofabcli-1.114.7.dist-info/METADATA,sha256=r6dmQ7yRVTxCFlRR4L6gQBV5NrNGQw5P3m88YLqQWRA,4946
+annofabcli-1.114.7.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
+annofabcli-1.114.7.dist-info/entry_points.txt,sha256=C2uSUc-kkLJpoK_mDL5FEMAdorLEMPfwSf8VBMYnIFM,56
+annofabcli-1.114.7.dist-info/licenses/LICENSE,sha256=pcqWYfxFtxBzhvKp3x9MXNM4xciGb2eFewaRhXUNHlo,1081
+annofabcli-1.114.7.dist-info/RECORD,,