annofabcli 1.106.6__py3-none-any.whl → 1.106.8__py3-none-any.whl

annofabcli/annotation/change_annotation_attributes.py

@@ -46,7 +46,7 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
         service: annofabapi.Resource,
         *,
         project_id: str,
-        is_force: bool,
+        include_completed: 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.is_force = is_force
+        self.include_completed = include_completed
 
         self.dump_annotation_obj = DumpAnnotationMain(service, project_id)
 
@@ -114,9 +114,9 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
         *,
         backup_dir: Optional[Path] = None,
         task_index: Optional[int] = None,
-    ) -> bool:
+    ) -> tuple[bool, int]:
         """
-        タスクに対してアノテーション属性を変更する。
+        タスクに対してアノテーション属性値を変更する。
 
         Args:
             project_id:
@@ -127,23 +127,24 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
             backup_dir: アノテーションをバックアップとして保存するディレクトリ。指定しない場合は、バックアップを取得しない。
 
         Returns:
-            アノテーションの属性を変更するAPI ``change_annotation_attributes`` を実行したか否か
+            tuple[0]: 成功した場合はTrue、失敗した場合はFalse
+            tuple[1]: 変更したアノテーションの個数
         """
         logger_prefix = f"{task_index + 1!s} 件目: " if task_index is not None else ""
         dict_task = self.service.wrapper.get_task_or_none(self.project_id, task_id)
         if dict_task is None:
             logger.warning(f"task_id = '{task_id}' は存在しません。")
-            return False
+            return False, 0
 
         task: Task = Task.from_dict(dict_task)
         if task.status == TaskStatus.WORKING:
             logger.warning(f"task_id='{task_id}': タスクが作業中状態のため、スキップします。")
-            return False
+            return False, 0
 
-        if not self.is_force:  # noqa: SIM102
+        if not self.include_completed:  # noqa: SIM102
             if task.status == TaskStatus.COMPLETE:
-                logger.warning(f"task_id='{task_id}': タスクが完了状態のため、スキップします。")
-                return False
+                logger.warning(f"task_id='{task_id}': タスクが完了状態のため、スキップします。完了状態のタスクのアノテーション属性値を変更するには、 ``--include_completed`` を指定してください。")
+                return False, 0
 
         annotation_list = self.get_annotation_list_for_task(task_id, annotation_query)
         logger.info(
@@ -151,17 +152,17 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
         )
         if len(annotation_list) == 0:
             logger.info(f"{logger_prefix}task_id='{task_id}'には変更対象のアノテーションが存在しないので、スキップします。")
-            return False
+            return False, 0
 
-        if not self.confirm_processing(f"task_id='{task_id}' のアノテーション属性を変更しますか？"):
-            return False
+        if not self.confirm_processing(f"task_id='{task_id}' のアノテーション属性値を変更しますか？"):
+            return False, 0
 
         if backup_dir is not None:
             self.dump_annotation_obj.dump_annotation_for_task(task_id, output_dir=backup_dir)
 
         self.change_annotation_attributes(annotation_list, additional_data_list)
         logger.info(f"{logger_prefix}task_id='{task_id}': {len(annotation_list)} 個のアノテーションの属性値を変更しました。")
-        return True
+        return True, len(annotation_list)
 
     def change_attributes_for_task_wrapper(
         self,
@@ -170,7 +171,7 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
         additional_data_list: list[dict[str, Any]],
         *,
         backup_dir: Optional[Path] = None,
-    ) -> bool:
+    ) -> tuple[bool, int]:
         task_index, task_id = tpl
         try:
             return self.change_attributes_for_task(
@@ -181,8 +182,8 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
                 task_index=task_index,
             )
         except Exception:  # pylint: disable=broad-except
-            logger.warning(f"タスク'{task_id}'のアノテーションの属性の変更に失敗しました。", exc_info=True)
-            return False
+            logger.warning(f"タスク'{task_id}'のアノテーションの属性値の変更に失敗しました。", exc_info=True)
+            return False, 0
 
     def change_annotation_attributes_for_task_list(
         self,
@@ -209,8 +210,9 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
 
         if backup_dir is not None:
             backup_dir.mkdir(exist_ok=True, parents=True)
-
         success_count = 0
+        # 変更したアノテーションの個数
+        changed_annotation_count = 0
         if parallelism is not None:
             func = functools.partial(
                 self.change_attributes_for_task_wrapper,
@@ -219,26 +221,28 @@ class ChangeAnnotationAttributesMain(CommandLineWithConfirm):
                 backup_dir=backup_dir,
             )
             with multiprocessing.Pool(parallelism) as pool:
-                result_bool_list = pool.map(func, enumerate(task_id_list))
-                success_count = len([e for e in result_bool_list if e])
+                result_tuple_list = pool.map(func, enumerate(task_id_list))
+                success_count = len([e for e in result_tuple_list if e[0]])
+                changed_annotation_count = sum(e[1] for e in result_tuple_list)
 
         else:
             for task_index, task_id in enumerate(task_id_list):
                 try:
-                    result = self.change_attributes_for_task(
+                    result, sub_changed_annotation_count = self.change_attributes_for_task(
                         task_id,
                         annotation_query=annotation_query,
                         additional_data_list=additional_data_list,
                         backup_dir=backup_dir,
                         task_index=task_index,
                     )
+                    changed_annotation_count += sub_changed_annotation_count
                     if result:
                         success_count += 1
                 except Exception:
-                    logger.warning(f"タスク'{task_id}'のアノテーションの属性の変更に失敗しました。", exc_info=True)
+                    logger.warning(f"タスク'{task_id}'のアノテーションの属性値の変更に失敗しました。", exc_info=True)
                     continue
 
-        logger.info(f"{success_count} / {len(task_id_list)} 件のタスクに対してアノテーションの属性を変更しました。")
+        logger.info(f"{success_count} / {len(task_id_list)} 件のタスクに対して {changed_annotation_count} 件のアノテーションの属性値を変更しました。")
 
 
 class ChangeAttributesOfAnnotation(CommandLine):
@@ -310,15 +314,15 @@ class ChangeAttributesOfAnnotation(CommandLine):
             backup_dir = Path(args.backup)
 
         super().validate_project(project_id, [ProjectMemberRole.OWNER, ProjectMemberRole.ACCEPTER])
-        if args.force:  # noqa: SIM102
+        if args.include_completed:  # noqa: SIM102
             if not self.facade.contains_any_project_member_role(project_id, [ProjectMemberRole.OWNER]):
                 print(  # noqa: T201
-                    f"{self.COMMON_MESSAGE} argument --force : '--force' 引数を利用するにはプロジェクトのオーナーロールを持つユーザーで実行する必要があります。",
+                    f"{self.COMMON_MESSAGE} argument --include_completed : '--include_completed' 引数を利用するにはプロジェクトのオーナーロールを持つユーザーで実行する必要があります。",
                     file=sys.stderr,
                 )
                 sys.exit(COMMAND_LINE_ERROR_STATUS_CODE)
 
-        main_obj = ChangeAnnotationAttributesMain(self.service, project_id=project_id, is_force=args.force, all_yes=args.yes)
+        main_obj = ChangeAnnotationAttributesMain(self.service, project_id=project_id, include_completed=args.include_completed, all_yes=args.yes)
         main_obj.change_annotation_attributes_for_task_list(
             task_id_list,
             annotation_query=annotation_query,
@@ -358,7 +362,7 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
     )
 
     parser.add_argument(
-        "--force",
+        "--include_completed",
         action="store_true",
         help="完了状態のタスクのアノテーション属性も変更します。ただし、オーナーロールを持つユーザーでしか実行できません。",
     )
@@ -381,10 +385,11 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
 
 def add_parser(subparsers: Optional[argparse._SubParsersAction] = None) -> argparse.ArgumentParser:
     subcommand_name = "change_attributes"
-    subcommand_help = "アノテーションの属性を変更します。"
+    subcommand_help = "アノテーションの属性値を変更します。"
     description = (
-        "アノテーションの属性を一括で変更します。ただし、作業中状態のタスクのアノテーションの属性は変更できません。"
-        "間違えてアノテーション属性を変更したときに復元できるようにするため、 ``--backup`` でバックアップ用のディレクトリを指定することを推奨します。"
+        "アノテーションの属性値を一括で変更します。ただし、作業中状態のタスクに含まれるアノテーションは変更できません。"
+        "完了状態のタスクに含まれるアノテーションは、デフォルトでは変更できません。"
+        "間違えてアノテーション属性値を変更したときに復元できるようにするため、 ``--backup`` でバックアップ用のディレクトリを指定することを推奨します。"
     )
     epilog = "オーナロールまたはチェッカーロールを持つユーザで実行してください。"
 

annofabcli/annotation/change_annotation_attributes_per_annotation.py

@@ -6,7 +6,7 @@ import logging
 import sys
 from collections import defaultdict
 from pathlib import Path
-from typing import Any, Optional, Union
+from typing import Optional, Union
 
 import annofabapi
 import pandas
@@ -76,7 +76,7 @@ class ChangeAnnotationAttributesPerAnnotationMain(CommandLineWithConfirm):
         self.dump_annotation_obj = DumpAnnotationMain(service, project_id)
         super().__init__(all_yes)
 
-    def change_annotation_attributes_by_frame(self, task_id: str, input_data_id: str, anno_list: list[TargetAnnotation]) -> bool:
+    def change_annotation_attributes_by_frame(self, task_id: str, input_data_id: str, anno_list: list[TargetAnnotation]) -> tuple[int, int]:
         """
         フレームごとにアノテーション属性値を変更する。
 
@@ -85,6 +85,10 @@ class ChangeAnnotationAttributesPerAnnotationMain(CommandLineWithConfirm):
             input_data_id: 入力データID
             additional_data_list: 変更後の属性値(`AdditionalDataListV2`スキーマ)
 
+        Returns:
+            [1]: 属性値の変更に成功したアノテーション数
+            [2]: 属性値を変更できなかった（変更対象のアノテーションが存在しなかった）アノテーション数
+
         """
         editor_annotation, _ = self.service.api.get_editor_annotation(self.project_id, task_id=task_id, input_data_id=input_data_id, query_params={"v": "2"})
 
@@ -94,26 +98,47 @@ class ChangeAnnotationAttributesPerAnnotationMain(CommandLineWithConfirm):
 
         details_map = {detail["annotation_id"]: detail for detail in editor_annotation["details"]}
 
-        def _to_request_body_elm(anno: TargetAnnotation) -> dict[str, Any]:
+        request_body = []
+        non_target_annotation_count = 0
+        for anno in anno_list:
+            if anno.annotation_id not in details_map:
+                logger.warning(
+                    f"task_id='{task_id}', input_data_id='{input_data_id}' :: "
+                    f"annotation_id='{anno.annotation_id}'であるアノテーションが存在しないため、"
+                    "このアノテーションの属性値の変更をスキップします。"
+                )
+                non_target_annotation_count += 1
+                continue
+
+            label_id = details_map[anno.annotation_id]["label_id"]
             additional_data_list = convert_attributes_from_cli_to_additional_data_list_v2(anno.attributes, annotation_specs=self.annotation_specs)
-            return {
-                "data": {
-                    "project_id": editor_annotation["project_id"],
-                    "task_id": editor_annotation["task_id"],
-                    "input_data_id": editor_annotation["input_data_id"],
-                    "updated_datetime": editor_annotation["updated_datetime"],
-                    "annotation_id": anno.annotation_id,
-                    "label_id": details_map[anno.annotation_id]["label_id"],
-                    "additional_data_list": additional_data_list,
-                },
-                "_type": "PutV2",
-            }
-
-        request_body = [_to_request_body_elm(annotation) for annotation in anno_list]
-
-        self.service.api.batch_update_annotations(self.project_id, request_body=request_body)
-        logger.debug(f"task_id='{task_id}', input_data_id='{input_data_id}' :: {len(request_body)}件の属性値を変更しました。")
-        return True
+            request_body.append(
+                {
+                    "data": {
+                        "project_id": editor_annotation["project_id"],
+                        "task_id": editor_annotation["task_id"],
+                        "input_data_id": editor_annotation["input_data_id"],
+                        "updated_datetime": editor_annotation["updated_datetime"],
+                        "annotation_id": anno.annotation_id,
+                        "label_id": label_id,
+                        "additional_data_list": additional_data_list,
+                    },
+                    "_type": "PutV2",
+                }
+            )
+
+        if request_body:
+            self.service.api.batch_update_annotations(self.project_id, request_body=request_body)
+        else:
+            logger.debug(f"task_id='{task_id}', input_data_id='{input_data_id}' :: 変更対象のアノテーションがありませんでした。")
+
+        succeed_to_change_annotation_count = len(request_body)
+        logger.debug(
+            f"task_id='{task_id}', input_data_id='{input_data_id}' :: "
+            f"{succeed_to_change_annotation_count}/{len(anno_list)}件の属性値を変更しました。"
+            f"{non_target_annotation_count}件のアノテーションは存在しなかったため、属性値の変更をスキップしました。"
+        )
+        return succeed_to_change_annotation_count, non_target_annotation_count
 
     def change_annotation_attributes_for_task(self, task_id: str, annotation_list_per_input_data_id: dict[str, list[TargetAnnotation]]) -> tuple[bool, int, int]:
         """
@@ -159,10 +184,9 @@ class ChangeAnnotationAttributesPerAnnotationMain(CommandLineWithConfirm):
 
         for input_data_id, sub_anno_list in annotation_list_per_input_data_id.items():
             try:
-                if self.change_annotation_attributes_by_frame(task_id, input_data_id, sub_anno_list):
-                    succeed_to_change_annotation_count += len(sub_anno_list)
-                else:
-                    failed_to_change_annotation_count += len(sub_anno_list)
+                tmp_succeed_to_change_annotation_count, tmp_failed_to_change_annotation_count = self.change_annotation_attributes_by_frame(task_id, input_data_id, sub_anno_list)
+                succeed_to_change_annotation_count += tmp_succeed_to_change_annotation_count
+                failed_to_change_annotation_count += tmp_failed_to_change_annotation_count
             except Exception:
                 logger.warning(f"task_id='{task_id}', input_data_id='{input_data_id}' :: アノテーションの属性値変更に失敗しました。", exc_info=True)
                 failed_to_change_annotation_count += len(sub_anno_list)

annofabcli/annotation/import_annotation.py

@@ -9,7 +9,6 @@ import uuid
 import zipfile
 from collections.abc import Iterator
 from dataclasses import dataclass
-from functools import partial
 from pathlib import Path
 from typing import Any, Optional, Union
 
@@ -408,6 +407,7 @@ class ImportAnnotationMain(CommandLineWithConfirm):
         is_force: bool,
         is_merge: bool,
         is_overwrite: bool,
+        converter: AnnotationConverter,
     ) -> None:
         self.service = service
         self.facade = AnnofabApiFacade(service)
@@ -417,8 +417,9 @@ class ImportAnnotationMain(CommandLineWithConfirm):
         self.is_force = is_force
         self.is_merge = is_merge
         self.is_overwrite = is_overwrite
+        self.converter = converter
 
-    def put_annotation_for_input_data(self, parser: SimpleAnnotationParser, converter: AnnotationConverter) -> bool:
+    def put_annotation_for_input_data(self, parser: SimpleAnnotationParser) -> bool:
         task_id = parser.task_id
         input_data_id = parser.input_data_id
 
@@ -444,18 +445,18 @@ class ImportAnnotationMain(CommandLineWithConfirm):
 
         logger.info(f"task_id='{task_id}', input_data_id='{input_data_id}' :: {len(simple_annotation.details)} 件のアノテーションを登録します。")
         if self.is_merge:
-            request_body = converter.convert_annotation_details(parser, simple_annotation.details, old_details=old_annotation["details"], updated_datetime=old_annotation["updated_datetime"])
+            request_body = self.converter.convert_annotation_details(parser, simple_annotation.details, old_details=old_annotation["details"], updated_datetime=old_annotation["updated_datetime"])
         else:
-            request_body = converter.convert_annotation_details(parser, simple_annotation.details, old_details=[], updated_datetime=old_annotation["updated_datetime"])
+            request_body = self.converter.convert_annotation_details(parser, simple_annotation.details, old_details=[], updated_datetime=old_annotation["updated_datetime"])
 
         self.service.api.put_annotation(self.project_id, task_id, input_data_id, request_body=request_body, query_params={"v": "2"})
         return True
 
-    def put_annotation_for_task(self, task_parser: SimpleAnnotationParserByTask, converter: AnnotationConverter) -> int:
+    def put_annotation_for_task(self, task_parser: SimpleAnnotationParserByTask) -> int:
         success_count = 0
         for parser in task_parser.lazy_parse():
             try:
-                if self.put_annotation_for_input_data(parser, converter):
+                if self.put_annotation_for_input_data(parser):
                     success_count += 1
             except Exception:  # pylint: disable=broad-except
                 logger.warning(
@@ -465,7 +466,7 @@ class ImportAnnotationMain(CommandLineWithConfirm):
 
         return success_count
 
-    def execute_task(self, task_parser: SimpleAnnotationParserByTask, converter: AnnotationConverter, task_index: Optional[int] = None) -> bool:
+    def execute_task(self, task_parser: SimpleAnnotationParserByTask, task_index: Optional[int] = None) -> bool:
         """
         1個のタスクに対してアノテーションを登録する。
 
@@ -514,7 +515,7 @@ class ImportAnnotationMain(CommandLineWithConfirm):
                 )
                 return False
 
-        result_count = self.put_annotation_for_task(task_parser, converter)
+        result_count = self.put_annotation_for_task(task_parser)
         logger.info(f"{logger_prefix}タスク'{task_parser.task_id}'の入力データ {result_count} 個に対してアノテーションをインポートしました。")
 
         if changed_operator:
@@ -531,11 +532,10 @@ class ImportAnnotationMain(CommandLineWithConfirm):
     def execute_task_wrapper(
         self,
         tpl: tuple[int, SimpleAnnotationParserByTask],
-        converter: AnnotationConverter,
     ) -> bool:
         task_index, task_parser = tpl
         try:
-            return self.execute_task(task_parser, converter=converter, task_index=task_index)
+            return self.execute_task(task_parser, task_index=task_index)
         except Exception:  # pylint: disable=broad-except
             logger.warning(f"task_id='{task_parser.task_id}' のアノテーションのインポートに失敗しました。", exc_info=True)
             return False
@@ -543,18 +543,9 @@ class ImportAnnotationMain(CommandLineWithConfirm):
     def main(
         self,
         iter_task_parser: Iterator[SimpleAnnotationParserByTask],
-        converter: AnnotationConverter,
         target_task_ids: Optional[set[str]] = None,
         parallelism: Optional[int] = None,
     ) -> None:
-        """
-        アノテーションのインポート処理を実行するメイン関数です。
-
-        Notes:
-            `converter`をインスタンス変数でなく引数として渡している理由：
-            `multiprocessing.Pool`でシリアライズ化する際、"TypeError: cannot pickle '_thread.RLock' object"というエラーが発生するため
-        """
-
         def get_iter_task_parser_from_task_ids(_iter_task_parser: Iterator[SimpleAnnotationParserByTask], _target_task_ids: set[str]) -> Iterator[SimpleAnnotationParserByTask]:
             for task_parser in _iter_task_parser:
                 if task_parser.task_id in _target_task_ids:
@@ -571,15 +562,14 @@ class ImportAnnotationMain(CommandLineWithConfirm):
         task_count = 0
         if parallelism is not None:
             with multiprocessing.Pool(parallelism) as pool:
-                func = partial(self.execute_task_wrapper, converter=converter)
-                result_bool_list = pool.map(func, enumerate(iter_task_parser))
+                result_bool_list = pool.map(self.execute_task_wrapper, enumerate(iter_task_parser))
                 success_count = len([e for e in result_bool_list if e])
                 task_count = len(result_bool_list)
 
         else:
             for task_index, task_parser in enumerate(iter_task_parser):
                 try:
-                    result = self.execute_task(task_parser, converter=converter, task_index=task_index)
+                    result = self.execute_task(task_parser, task_index=task_index)
                     if result:
                         success_count += 1
                 except Exception:
@@ -618,6 +608,13 @@ class ImportAnnotation(CommandLine):
             print(f"{COMMON_MESSAGE} argument --annotation: ZIPファイルまたはディレクトリを指定してください。", file=sys.stderr)  # noqa: T201
             return False
 
+        if args.parallelism is not None and annotation_path.is_file() and zipfile.is_zipfile(annotation_path):
+            print(  # noqa: T201
+                f"{COMMON_MESSAGE} argument --parallelism: '--annotation'にZIPファイルを指定した場合は、'--parallelism'を指定できません。",
+                file=sys.stderr,
+            )
+            return False
+
         if args.parallelism is not None and not args.yes:
             print(  # noqa: T201
                 f"{COMMON_MESSAGE} argument --parallelism: '--parallelism'を指定するときは、必ず '--yes' を指定してください。",
@@ -642,7 +639,7 @@ class ImportAnnotation(CommandLine):
         # Simpleアノテーションの読み込み
         if annotation_path.is_dir():
             iter_task_parser = lazy_parse_simple_annotation_dir_by_task(annotation_path)
-        elif zipfile.is_zipfile(str(annotation_path)):
+        elif zipfile.is_zipfile(annotation_path):
             iter_task_parser = lazy_parse_simple_annotation_zip_by_task(annotation_path)
         else:
             logger.warning(f"annotation_path: '{annotation_path}' は、zipファイルまたはディレクトリではありませんでした。")
@@ -659,9 +656,10 @@ class ImportAnnotation(CommandLine):
             is_merge=args.merge,
             is_overwrite=args.overwrite,
             is_force=args.force,
+            converter=converter,
         )
 
-        main_obj.main(iter_task_parser, target_task_ids=target_task_ids, converter=converter, parallelism=args.parallelism)
+        main_obj.main(iter_task_parser, target_task_ids=target_task_ids, parallelism=args.parallelism)
 
 
 def main(args: argparse.Namespace) -> None:
@@ -717,7 +715,9 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
         "--parallelism",
         type=int,
         choices=PARALLELISM_CHOICES,
-        help="並列度。指定しない場合は、逐次的に処理します。指定した場合は、``--yes`` も指定してください。",
+        help="並列度。指定しない場合は、逐次的に処理します。"
+        "ただし ``--annotation`` にZIPファイルを指定した場合は、``--parallelism`` を指定できません。"
+        "また、``--parallelism`` を指定した場合は、``--yes`` も指定してください。",
     )
 
     parser.set_defaults(subcommand_func=main)

annofabcli/task/change_operator.py

@@ -26,10 +26,11 @@ logger = logging.getLogger(__name__)
 
 
 class ChangeOperatorMain:
-    def __init__(self, service: annofabapi.Resource, all_yes: bool) -> None:  # noqa: FBT001
+    def __init__(self, service: annofabapi.Resource, *, all_yes: bool, include_on_hold: bool = False) -> None:
         self.service = service
         self.facade = AnnofabApiFacade(service)
         self.all_yes = all_yes
+        self.include_on_hold = include_on_hold
 
     def confirm_processing(self, confirm_message: str) -> bool:
         """
@@ -55,10 +56,10 @@ class ChangeOperatorMain:
         return yes
 
     def confirm_change_operator(self, task: Task) -> bool:
-        confirm_message = f"task_id = {task.task_id} のタスクの担当者を変更しますか？"
+        confirm_message = f"task_id='{task.task_id}' のタスクの担当者を変更しますか？"
         return self.confirm_processing(confirm_message)
 
-    def change_operator_for_task(
+    def change_operator_for_task(  # noqa: PLR0911
         self,
         project_id: str,
         task_id: str,
@@ -69,7 +70,7 @@ class ChangeOperatorMain:
         logging_prefix = f"{task_index + 1} 件目" if task_index is not None else ""
         dict_task = self.service.wrapper.get_task_or_none(project_id, task_id)
         if dict_task is None:
-            logger.warning(f"{logging_prefix}: task_id='{task_id}'のタスクは存在しないので、スキップします。")
+            logger.warning(f"{logging_prefix} :: task_id='{task_id}'のタスクは存在しないので、スキップします。")
             return False
 
         task: Task = Task.from_dict(dict_task)
@@ -78,14 +79,23 @@ class ChangeOperatorMain:
         if task.account_id is not None:
             now_user_id = self.facade.get_user_id_from_account_id(project_id, task.account_id)
 
-        logger.debug(f"{logging_prefix} : task_id = {task.task_id}, status = {task.status.value}, phase = {task.phase.value}, phase_stage = {task.phase_stage}, user_id = {now_user_id}")
+        logger.debug(f"{logging_prefix} :: task_id='{task.task_id}', status='{task.status.value}', phase='{task.phase.value}', phase_stage='{task.phase_stage}', user_id='{now_user_id}'")
+        if task.account_id == new_account_id:
+            logger.info(f"{logging_prefix} :: task_id='{task_id}' :: タスクの担当者はすでにuser_id='{now_user_id}'のユーザーです。担当者を変更する必要がないのでスキップします。")
+            return False
 
         if task.status in [TaskStatus.COMPLETE, TaskStatus.WORKING]:
-            logger.warning(f"{logging_prefix} : task_id = {task_id} : タスクのstatusがworking or complete なので、担当者を変更できません。")
+            logger.warning(f"{logging_prefix} :: task_id='{task_id}' :: タスクが作業中状態または完了状態なので、担当者を変更できません。 :: status='{task.status.value}'")
+            return False
+
+        if task.status == TaskStatus.ON_HOLD and not self.include_on_hold:
+            logger.warning(
+                f"{logging_prefix} :: task_id='{task_id}' :: タスクが保留中状態なので、担当者を変更できません。保留中状態のタスクの担当者も変更する場合は、'--include_on_hold'を指定してください。"
+            )
             return False
 
         if not match_task_with_query(task, task_query):
-            logger.debug(f"{logging_prefix} : task_id = {task_id} : `--task_query` の条件にマッチしないため、スキップします。task_query={task_query}")
+            logger.debug(f"{logging_prefix} :: task_id='{task_id}' :: `--task_query` の条件にマッチしないため、スキップします。task_query='{task_query}'")
             return False
 
         if not self.confirm_change_operator(task):
@@ -94,11 +104,11 @@ class ChangeOperatorMain:
         try:
             # 担当者を変更する
             self.service.wrapper.change_task_operator(project_id, task_id, operator_account_id=new_account_id)
-            logger.debug(f"{logging_prefix} : task_id = {task_id}, phase={dict_task['phase']} のタスクの担当者を変更しました。")
+            logger.debug(f"{logging_prefix} :: task_id='{task_id}'であるタスクの担当者を変更しました。 :: phase='{dict_task['phase']}'")
             return True  # noqa: TRY300
 
         except requests.exceptions.HTTPError:
-            logger.warning(f"{logging_prefix} : task_id = {task_id} の担当者を変更するのに失敗しました。", exc_info=True)
+            logger.warning(f"{logging_prefix} :: task_id='{task_id}'である担当者を変更するのに失敗しました。", exc_info=True)
             return False
 
     def change_operator_for_task_wrapper(
@@ -118,19 +128,21 @@ class ChangeOperatorMain:
                 new_account_id=new_account_id,
             )
         except Exception:  # pylint: disable=broad-except
-            logger.warning(f"タスク'{task_id}'の担当者の変更に失敗しました。", exc_info=True)
+            logger.warning(f"task_id='{task_id}'であるタスク担当者の変更に失敗しました。", exc_info=True)
             return False
 
     def change_operator(
         self,
         project_id: str,
         task_id_list: list[str],
+        *,
         new_user_id: Optional[str] = None,
         task_query: Optional[TaskQuery] = None,
         parallelism: Optional[int] = None,
     ) -> None:
         """
-        検査コメントを付与して、タスクを差し戻す
+        指定した複数のタスクの担当者を変更します。
+
         Args:
             project_id:
             task_id_list:
@@ -143,13 +155,13 @@ class ChangeOperatorMain:
         if new_user_id is not None:
             new_account_id = self.facade.get_account_id_from_user_id(project_id, new_user_id)
             if new_account_id is None:
-                logger.error(f"ユーザ '{new_user_id}' のaccount_idが見つかりませんでした。終了します。")
+                logger.error(f"user_id='{new_user_id}'であるユーザーは、project_id='{project_id}'のプロジェクトのメンバーではありません。終了します。")
                 return
             else:
-                logger.info(f"{len(task_id_list)} 件のタスクの担当者を、{new_user_id}に変更します。")
+                logger.info(f"{len(task_id_list)} 件のタスクの担当者を、user_id='{new_user_id}'のユーザーに変更します。")
         else:
             new_account_id = None
-            logger.info(f"{len(task_id_list)} 件のタスクの担当者を未割り当てに変更します。")
+            logger.info(f"{len(task_id_list)} 件のタスクの担当者を「未割り当て」に変更します。")
 
         success_count = 0
 
@@ -172,7 +184,7 @@ class ChangeOperatorMain:
                     if result:
                         success_count += 1
                 except Exception:  # pylint: disable=broad-except
-                    logger.warning(f"タスク'{task_id}'の担当者の変更に失敗しました。", exc_info=True)
+                    logger.warning(f"task_id='{task_id}'であるタスクの担当者の変更に失敗しました。", exc_info=True)
                     continue
 
         logger.info(f"{success_count} / {len(task_id_list)} 件 タスクの担当者を変更しました。")
@@ -212,7 +224,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)
+        main_obj = ChangeOperatorMain(self.service, all_yes=self.all_yes, include_on_hold=args.include_on_hold)
         main_obj.change_operator(
             project_id,
             task_id_list=task_id_list,
@@ -242,6 +254,12 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
 
     argument_parser.add_task_query()
 
+    parser.add_argument(
+        "--include_on_hold",
+        action="store_true",
+        help="指定した場合、保留中のタスクの担当者も変更します。指定しない場合、保留中のタスクはスキップされます。",
+    )
+
     parser.add_argument(
         "--parallelism",
         type=int,
@@ -255,7 +273,7 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
 def add_parser(subparsers: Optional[argparse._SubParsersAction] = None) -> argparse.ArgumentParser:
     subcommand_name = "change_operator"
     subcommand_help = "タスクの担当者を変更します。"
-    description = "タスクの担当者を変更します。ただし、作業中また完了状態のタスクは、担当者を変更できません。"
+    description = "タスクの担当者を変更します。作業中状態、完了状態のタスクは、担当者を変更できません。保留中状態のタスクは、デフォルトでは担当者を変更できません。"
     epilog = "チェッカーまたはオーナロールを持つユーザで実行してください。"
 
     parser = annofabcli.common.cli.add_parser(subparsers, subcommand_name, subcommand_help, description, epilog=epilog)

{annofabcli-1.106.6.dist-info → annofabcli-1.106.8.dist-info}/METADATA

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

{annofabcli-1.106.6.dist-info → annofabcli-1.106.8.dist-info}/RECORD

@@ -3,14 +3,14 @@ annofabcli/__main__.py,sha256=YfuJE9E43xSo6iHTxVuQPHCz2eBaJS07QnVU42-0znQ,5293
 annofabcli/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 annofabcli/annotation/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 annofabcli/annotation/annotation_query.py,sha256=VwfPWpLOpVa2SeEJ264LmCKkBGDJvpX8o7GbWIrDE0o,15712
-annofabcli/annotation/change_annotation_attributes.py,sha256=Zjqax-sb7ujJnTUVv4io0GFmZzfqx0rpN2N1W86uVTs,17092
-annofabcli/annotation/change_annotation_attributes_per_annotation.py,sha256=iU199MlMfD-nfyRiHL44Zqrvr12vOeUHXyxCrDgaivI,14130
+annofabcli/annotation/change_annotation_attributes.py,sha256=pCHT12ba5jsT9kqslUXqfy2QriRT8yiAzgHDuL1JkNE,17884
+annofabcli/annotation/change_annotation_attributes_per_annotation.py,sha256=g_SqH1xzAdA8D1Hy1aoOPlpF86nc7F1PENeUX8XftDs,15529
 annofabcli/annotation/change_annotation_properties.py,sha256=Kp_LZ5sSoVmmjGE80ABVO3InxsXBIxiFFvVcIJNsOMk,18309
 annofabcli/annotation/copy_annotation.py,sha256=Pih2k3vvpgfT3Ovb3gZw2L_8fK_ws_wKR7ARYG5hG_8,18407
 annofabcli/annotation/delete_annotation.py,sha256=hQApNrx2Ci1bBWk0dRGA0oJkIgDHwl6Jy0-33gYF6jo,22989
 annofabcli/annotation/download_annotation_zip.py,sha256=P_ZpdqIaSFEmB8jjpdykcRhh2tVlHxSlXFrYreJjShE,3282
 annofabcli/annotation/dump_annotation.py,sha256=Q-p6f5XBs7khDgrfY5Q3CGLBMKEerJWO_CQ8_73UXVM,9972
-annofabcli/annotation/import_annotation.py,sha256=hVX2T7gN3pad2KI7TzB2_fnga7fnfRVTMGUa611m3xE,34612
+annofabcli/annotation/import_annotation.py,sha256=w0iSTmkIY8tz3cTUy2FJ6LCVpVUtKzcD7ej2cznot4A,34533
 annofabcli/annotation/list_annotation.py,sha256=uKcOuGC7lzd6vVbzizkiZtYdXJ7EzY0iifuiqKl2wQM,10707
 annofabcli/annotation/list_annotation_count.py,sha256=T9fbaoxWeDJIVgW_YgHRldbwrVZWiE-57lfJrDQrj80,6474
 annofabcli/annotation/merge_segmentation.py,sha256=kIsCeXtJxzd6nobQPpi0fscaRDlTx3tg1qpy5PDfSJI,18107
@@ -182,7 +182,7 @@ annofabcli/supplementary/put_supplementary_data.py,sha256=Pyq9G6xQFyJ8qrdWLOQvIU
 annofabcli/supplementary/subcommand_supplementary.py,sha256=F8qfuNQzgW5HV1QKB4h0DWN7-kPVQcoFQwPfW_vjZVk,1079
 annofabcli/task/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 annofabcli/task/cancel_acceptance.py,sha256=QG9GhGNfBXntqYTKdYjLUczjD4p_ATkPc_xYCpTO24Y,13851
-annofabcli/task/change_operator.py,sha256=1LLpez79ezW_MqcNN608kjxp3xTS1uRRioqcugui9hM,10446
+annofabcli/task/change_operator.py,sha256=q6pMd1SdsTRgMHS0705dnosTSHprTpYgXtNd0rli2Zg,11793
 annofabcli/task/change_status_to_break.py,sha256=hwdFTFW-zV0VxuinoBB5n6mvHJ7g9ChjrSOXZcNk88w,8621
 annofabcli/task/change_status_to_on_hold.py,sha256=vWRyk6IK3HcgTWDIbbhXzsrtuoa7OlXCf8CvUpFp_Uw,12981
 annofabcli/task/complete_tasks.py,sha256=ssg_Z7ADRQRXvXgK2k5TEmvbRjrJQ33cXeb8kG8Y3jc,24917
@@ -209,8 +209,8 @@ annofabcli/task_history_event/download_task_history_event_json.py,sha256=hQLVbQ0
 annofabcli/task_history_event/list_all_task_history_event.py,sha256=EeKMyPUxGwYCFtWQHHW954ZserGm8lUqrwNnV1iX9X4,6830
 annofabcli/task_history_event/list_worktime.py,sha256=Y7Pu5DP7scPf7HPt6CTiTvB1_5_Nfi1bStUIaCpkhII,15507
 annofabcli/task_history_event/subcommand_task_history_event.py,sha256=mJVJoT4RXk4HWnY7-Nrsl4If-gtaIIEXd2z7eFZwM2I,1260
-annofabcli-1.106.6.dist-info/METADATA,sha256=6fqjbrE7DWDbXzS3Zo4GPm_oHeqhkdfjqqdUlySo8Rw,5286
-annofabcli-1.106.6.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-annofabcli-1.106.6.dist-info/entry_points.txt,sha256=C2uSUc-kkLJpoK_mDL5FEMAdorLEMPfwSf8VBMYnIFM,56
-annofabcli-1.106.6.dist-info/licenses/LICENSE,sha256=pcqWYfxFtxBzhvKp3x9MXNM4xciGb2eFewaRhXUNHlo,1081
-annofabcli-1.106.6.dist-info/RECORD,,
+annofabcli-1.106.8.dist-info/METADATA,sha256=rSWCLpho30p_wsQdKax1vujJoZ3Zlj03RAGDGcth2s8,5286
+annofabcli-1.106.8.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+annofabcli-1.106.8.dist-info/entry_points.txt,sha256=C2uSUc-kkLJpoK_mDL5FEMAdorLEMPfwSf8VBMYnIFM,56
+annofabcli-1.106.8.dist-info/licenses/LICENSE,sha256=pcqWYfxFtxBzhvKp3x9MXNM4xciGb2eFewaRhXUNHlo,1081
+annofabcli-1.106.8.dist-info/RECORD,,