annofabcli 1.96.1__py3-none-any.whl → 1.98.0__py3-none-any.whl

annofabcli/annotation/remove_segmentation_overlap.py

@@ -0,0 +1,343 @@
+from __future__ import annotations
+
+import argparse
+import copy
+import logging
+import multiprocessing
+import sys
+import tempfile
+from collections.abc import Collection
+from pathlib import Path
+from typing import Any, Optional
+
+import annofabapi
+import numpy
+from annofabapi.pydantic_models.task_status import TaskStatus
+from annofabapi.segmentation import read_binary_image, write_binary_image
+from annofabapi.utils import can_put_annotation
+
+import annofabcli
+from annofabcli.common.cli import (
+    COMMAND_LINE_ERROR_STATUS_CODE,
+    PARALLELISM_CHOICES,
+    ArgumentParser,
+    CommandLine,
+    CommandLineWithConfirm,
+    build_annofabapi_resource_and_login,
+)
+from annofabcli.common.facade import AnnofabApiFacade
+
+logger = logging.getLogger(__name__)
+
+
+def remove_overlap_of_binary_image_array(
+    binary_image_array_by_annotation: dict[str, numpy.ndarray], annotation_id_list: list[str]
+) -> dict[str, numpy.ndarray]:
+    """
+    塗りつぶし画像の重なりを除去したbool配列をdictで返します。
+
+    Args:
+        binary_image_array_by_annotation: annotation_idをkeyとし、塗りつぶし画像のbool配列をvalueとするdict
+        annotation_id_list: 塗りつぶし画像のannotation_idのlist。背面から前面の順に格納されている
+
+    Returns:
+        重なりを除去した塗りつぶし画像のbool配列が格納されているdict。keyはannotation_id
+
+    """
+    assert set(binary_image_array_by_annotation.keys()) == set(annotation_id_list)
+
+    whole_2d_array = None  # 複数の塗りつぶしアノテーションを1枚に重ね合わせた状態。各要素はannotation_id
+
+    # 背面から塗りつぶしアノテーションのbool配列を重ねていく
+    for annotation_id in annotation_id_list:
+        input_binary_image_array = binary_image_array_by_annotation[annotation_id]
+        if whole_2d_array is None:
+            whole_2d_array = numpy.full(input_binary_image_array.shape, "", dtype=str)
+
+        whole_2d_array = numpy.where(input_binary_image_array, annotation_id, whole_2d_array)
+
+    output_binary_image_array_by_annotation = {}
+    for annotation_id in annotation_id_list:
+        output_binary_image_array: numpy.ndarray = whole_2d_array == annotation_id  # type: ignore[assignment]
+        output_binary_image_array_by_annotation[annotation_id] = output_binary_image_array
+
+    return output_binary_image_array_by_annotation
+
+
+class RemoveSegmentationOverlapMain(CommandLineWithConfirm):
+    def __init__(self, annofab_service: annofabapi.Resource, *, project_id: str, all_yes: bool, is_force: bool) -> None:
+        self.annofab_service = annofab_service
+        self.project_id = project_id
+        self.is_force = is_force
+        super().__init__(all_yes)
+
+    def remove_segmentation_overlap_and_save(self, details: list[dict[str, Any]], output_dir: Path) -> list[str]:
+        """
+        `getEditorAnnotation` APIで取得した`details`から、塗りつぶし画像の重なりの除去が必要な場合に、
+        重なりを除去した塗りつぶし画像を`output_dir`に出力します。
+        塗りつぶし画像のファイル名は`${annotation_id}.png`です。
+
+        Args:
+            details: `getEditorAnnotation` APIで取得した`details`
+            output_dir: 塗りつぶし画像の出力先のディレクトリ。
+
+        Returns:
+            重なりの除去が必要な塗りつぶし画像のannotation_idのlist
+        """
+        input_binary_image_array_by_annotation = {}
+        segmentation_annotation_id_list = []
+
+        for detail in details:
+            if detail["body"]["_type"] != "Outer":
+                continue
+
+            segmentation_response = self.annofab_service.wrapper.execute_http_get(detail["body"]["url"], stream=True)
+            segmentation_response.raw.decode_content = True
+            input_binary_image_array_by_annotation[detail["annotation_id"]] = read_binary_image(segmentation_response.raw)
+            segmentation_annotation_id_list.append(detail["annotation_id"])
+
+        # reversedを使っている理由:
+        # `details`には、前面から背面の順にアノテーションが格納されているため、
+        output_binary_image_array_by_annotation = remove_overlap_of_binary_image_array(
+            input_binary_image_array_by_annotation, list(reversed(segmentation_annotation_id_list))
+        )
+
+        updated_annotation_id_list = []
+        for annotation_id, output_binary_image_array in output_binary_image_array_by_annotation.items():
+            input_binary_image_array = input_binary_image_array_by_annotation[annotation_id]
+            if not numpy.array_equal(input_binary_image_array, output_binary_image_array):
+                output_file_path = output_dir / f"{annotation_id}.png"
+                write_binary_image(output_binary_image_array, output_file_path)
+                updated_annotation_id_list.append(annotation_id)
+
+        return updated_annotation_id_list
+
+    def update_segmentation_annotation(self, task_id: str, input_data_id: str, log_message_prefix: str = "") -> bool:
+        """
+        塗りつぶしアノテーションの重なりがあれば、`putAnnotation` APIを使用して重なりを除去します。
+
+        Args:
+            project_id: プロジェクトID
+            task_id: タスクID
+            annotation_id_list: 更新する塗りつぶし画像のannotation_idのlist
+        """
+        old_annotation, _ = self.annofab_service.api.get_editor_annotation(self.project_id, task_id, input_data_id, query_params={"v": "2"})
+        old_details = old_annotation["details"]
+        with tempfile.TemporaryDirectory() as temp_dir:
+            temp_dir_path = Path(temp_dir)
+            updated_annotation_id_list = self.remove_segmentation_overlap_and_save(old_details, temp_dir_path)
+            if len(updated_annotation_id_list) == 0:
+                logger.debug(
+                    f"{log_message_prefix}塗りつぶしアノテーションの重なりはなかったので、スキップします。 :: "
+                    f"task_id='{task_id}', input_data_id='{input_data_id}'"
+                )
+                return False
+
+            logger.debug(
+                f"{log_message_prefix}{len(updated_annotation_id_list)} 件の塗りつぶしアノテーションを更新します。 :: "
+                f"task_id='{task_id}', input_data_id='{input_data_id}', annotation_id_list={updated_annotation_id_list}"
+            )
+            new_details = []
+            for detail in old_details:
+                annotation_id = detail["annotation_id"]
+                new_detail = copy.deepcopy(detail)
+                new_detail["_type"] = "Update"
+                if annotation_id in updated_annotation_id_list:
+                    with (temp_dir_path / f"{annotation_id}.png").open("rb") as f:
+                        s3_path = self.annofab_service.wrapper.upload_data_to_s3(self.project_id, data=f, content_type="image/png")
+
+                    new_detail["body"]["path"] = s3_path
+
+                else:
+                    # 更新しない場合は、`body`をNoneにする
+                    new_detail["body"] = None
+
+                new_details.append(new_detail)
+
+        request_body = {
+            "project_id": self.project_id,
+            "task_id": task_id,
+            "input_data_id": input_data_id,
+            "details": new_details,
+            "format_version": "2.0.0",
+            "updated_datetime": old_annotation["updated_datetime"],
+        }
+        self.annofab_service.api.put_annotation(self.project_id, task_id, input_data_id, query_params={"v": "2"}, request_body=request_body)
+        logger.debug(
+            f"{log_message_prefix}{len(updated_annotation_id_list)} 件の塗りつぶしアノテーションを更新しました。 :: "
+            f"task_id='{task_id}', input_data_id='{input_data_id}'"
+        )
+        return True
+
+    def update_segmentation_annotation_for_task(self, task_id: str, *, task_index: Optional[int] = None) -> int:
+        """
+        1個のタスクに対して、塗りつぶしアノテーションの重なりを除去します。
+
+        Returns:
+            アノテーションを更新した入力データ数（フレーム数）
+        """
+        log_message_prefix = f"{task_index + 1} 件目 :: " if task_index is not None else ""
+
+        task = self.annofab_service.wrapper.get_task_or_none(project_id=self.project_id, task_id=task_id)
+        if task is None:
+            logger.warning(f"{log_message_prefix}task_id='{task_id}'であるタスクは存在しません。")
+            return 0
+
+        if task["status"] in {TaskStatus.WORKING.value, TaskStatus.COMPLETE.value}:
+            logger.debug(
+                f"{log_message_prefix}task_id='{task_id}'のタスクの状態は「作業中」または「完了」であるため、"
+                f"アノテーションの更新をスキップします。  :: status='{task['status']}'"
+            )
+            return 0
+
+        if not self.confirm_processing(f"task_id='{task_id}'の塗りつぶしアノテーションの重なりを除去しますか？"):
+            return 0
+
+        # 担当者割り当て変更チェック
+        changed_operator = False
+        original_operator_account_id = task["account_id"]
+        if not can_put_annotation(task, self.annofab_service.api.account_id):
+            if self.is_force:
+                logger.debug(f"{log_message_prefix}task_id='{task_id}' のタスクの担当者を自分自身に変更します。")
+                changed_operator = True
+                task = self.annofab_service.wrapper.change_task_operator(
+                    self.project_id,
+                    task_id,
+                    self.annofab_service.api.account_id,
+                    last_updated_datetime=task["updated_datetime"],
+                )
+            else:
+                logger.debug(
+                    f"{log_message_prefix}task_id='{task_id}' のタスクは、過去に誰かに割り当てられたタスクで、"
+                    f"現在の担当者が自分自身でないため、アノテーションの更新をスキップします。"
+                    f"担当者を自分自身に変更してアノテーションを更新する場合は、コマンドライン引数 '--force' を指定してください。"
+                )
+                return 0
+
+        success_input_data_count = 0
+        for input_data_id in task["input_data_id_list"]:
+            try:
+                result = self.update_segmentation_annotation(task_id, input_data_id, log_message_prefix=log_message_prefix)
+                if result:
+                    success_input_data_count += 1
+            except Exception:
+                logger.warning(
+                    f"{log_message_prefix}task_id='{task_id}', input_data_id='{input_data_id}'のアノテーションの更新に失敗しました。", exc_info=True
+                )
+                continue
+
+        # 担当者を元に戻す
+        if changed_operator:
+            logger.debug(
+                f"{log_message_prefix}task_id='{task_id}' のタスクの担当者を、元の担当者（account_id='{original_operator_account_id}'）に変更します。"
+            )
+            self.annofab_service.wrapper.change_task_operator(
+                self.project_id,
+                task_id,
+                original_operator_account_id,
+                last_updated_datetime=task["updated_datetime"],
+            )
+
+        return success_input_data_count
+
+    def update_segmentation_annotation_for_task_wrapper(self, tpl: tuple[int, str]) -> int:
+        try:
+            task_index, task_id = tpl
+            return self.update_segmentation_annotation_for_task(task_id, task_index=task_index)
+        except Exception:
+            logger.warning(f"task_id='{task_id}' のアノテーションの更新に失敗しました。", exc_info=True)
+            return 0
+
+    def main(
+        self,
+        task_ids: Collection[str],
+        parallelism: Optional[int] = None,
+    ) -> None:
+        logger.info(f"{len(task_ids)} 件のタスクの塗りつぶしアノテーションの重なりを除去します。")
+        success_input_data_count = 0
+        if parallelism is not None:
+            with multiprocessing.Pool(parallelism) as pool:
+                result_count_list = pool.map(self.update_segmentation_annotation_for_task_wrapper, enumerate(task_ids))
+                success_input_data_count = sum(result_count_list)
+
+        else:
+            for task_index, task_id in enumerate(task_ids):
+                try:
+                    result = self.update_segmentation_annotation_for_task(task_id, task_index=task_index)
+                    success_input_data_count += result
+                except Exception:
+                    logger.warning(f"task_id='{task_id}' のアノテーションの更新に失敗しました。", exc_info=True)
+                    continue
+
+        logger.info(f"{len(task_ids)} 件のタスクに含まれる入力データ {success_input_data_count} 件の塗りつぶしアノテーションを更新しました。")
+
+
+class RemoveSegmentationOverlap(CommandLine):
+    COMMON_MESSAGE = "annofabcli annotation remove_segmentation_overlap: error:"
+
+    def validate(self, args: argparse.Namespace) -> bool:
+        if args.parallelism is not None and not args.yes:
+            print(  # noqa: T201
+                f"{self.COMMON_MESSAGE} argument --parallelism: '--parallelism'を指定するときは、'--yes' を指定してください。",
+                file=sys.stderr,
+            )
+            return False
+
+        return True
+
+    def main(self) -> None:
+        args = self.args
+        if not self.validate(args):
+            sys.exit(COMMAND_LINE_ERROR_STATUS_CODE)
+
+        project_id = args.project_id
+        task_id_list = annofabcli.common.cli.get_list_from_args(args.task_id)
+
+        main_obj = RemoveSegmentationOverlapMain(
+            self.service,
+            project_id=project_id,
+            all_yes=self.all_yes,
+            is_force=args.force,
+        )
+
+        main_obj.main(task_id_list, parallelism=args.parallelism)
+
+
+def main(args: argparse.Namespace) -> None:
+    service = build_annofabapi_resource_and_login(args)
+    facade = AnnofabApiFacade(service)
+    RemoveSegmentationOverlap(service, facade, args).main()
+
+
+def parse_args(parser: argparse.ArgumentParser) -> None:
+    argument_parser = ArgumentParser(parser)
+    argument_parser.add_project_id()
+    argument_parser.add_task_id()
+
+    parser.add_argument(
+        "--force",
+        action="store_true",
+        help="過去に担当者を割り当てられていて、かつ現在の担当者が自分自身でない場合、タスクの担当者を一時的に自分自身に変更してからアノテーションをコピーします。",
+    )
+
+    parser.add_argument(
+        "--parallelism",
+        type=int,
+        choices=PARALLELISM_CHOICES,
+        help="並列度。指定しない場合は、逐次的に処理します。指定した場合は、``--yes`` も指定してください。",
+    )
+
+    parser.set_defaults(subcommand_func=main)
+
+
+def add_parser(subparsers: Optional[argparse._SubParsersAction] = None) -> argparse.ArgumentParser:
+    subcommand_name = "remove_segmentation_overlap"
+    subcommand_help = "塗りつぶしアノテーションの重なりを除去します。"
+    description = (
+        "塗りつぶしアノテーションの重なりを除去します。"
+        "Annofabでインスタンスセグメンテーションは重ねることができてしまいます。"
+        "この重なりをなくしたいときに有用です。"
+    )
+    parser = annofabcli.common.cli.add_parser(subparsers, subcommand_name, subcommand_help, description)
+    parse_args(parser)
+    return parser

annofabcli/annotation/subcommand_annotation.py

@@ -10,6 +10,8 @@ import annofabcli.annotation.dump_annotation
 import annofabcli.annotation.import_annotation
 import annofabcli.annotation.list_annotation
 import annofabcli.annotation.list_annotation_count
+import annofabcli.annotation.merge_segmentation
+import annofabcli.annotation.remove_segmentation_overlap
 import annofabcli.annotation.restore_annotation
 import annofabcli.common.cli
 
@@ -27,6 +29,8 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
     annofabcli.annotation.import_annotation.add_parser(subparsers)
     annofabcli.annotation.list_annotation.add_parser(subparsers)
     annofabcli.annotation.list_annotation_count.add_parser(subparsers)
+    annofabcli.annotation.merge_segmentation.add_parser(subparsers)
+    annofabcli.annotation.remove_segmentation_overlap.add_parser(subparsers)
     annofabcli.annotation.restore_annotation.add_parser(subparsers)
 
 

annofabcli/input_data/change_input_data_name.py

@@ -143,11 +143,9 @@ def create_changed_input_data_list_from_csv(csv_file: Path) -> list[ChangedInput
         変更対象の入力データのlist
     """
     df_input_data = pandas.read_csv(
-        str(csv_file),
-        header=None,
-        names=("input_data_id", "input_data_name"),
+        csv_file,
         # 文字列として読み込むようにする
-        dtype={"input_data_id": str, "input_data_name": str},
+        dtype={"input_data_id": "string", "input_data_name": "string"},
     )
 
     input_data_dict_list = df_input_data.to_dict("records")
@@ -211,9 +209,9 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
             "変更対象の入力データが記載されたCSVファイルのパスを指定してください。\n"
             "CSVのフォーマットは以下の通りです。"
             "\n"
-            " * ヘッダ行なし, カンマ区切り\n"
-            " * 1列目: input_data_id (required)\n"
-            " * 2列目: input_data_name (required)\n"
+            " * ヘッダ行あり, カンマ区切り\n"
+            " * input_data_id (required)\n"
+            " * input_data_name (required)\n"
         ),
     )
 

annofabcli/input_data/update_metadata_of_input_data.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import argparse
+import copy
 import json
 import logging
 import multiprocessing
@@ -182,7 +183,7 @@ class UpdateMetadata(CommandLine):
                 sys.exit(COMMAND_LINE_ERROR_STATUS_CODE)
 
             assert input_data_id_list is not None, "'--metadata'を指定したときは'--input_data_id'は必須です。"
-            metadata_by_input_data_id = {input_data_id: metadata for input_data_id in input_data_id_list}
+            metadata_by_input_data_id = {input_data_id: copy.deepcopy(metadata) for input_data_id in input_data_id_list}
 
         elif args.metadata_by_input_data_id is not None:
             metadata_by_input_data_id = annofabcli.common.cli.get_json_from_args(args.metadata_by_input_data_id)

annofabcli/project_member/put_project_members.py

@@ -5,7 +5,6 @@ from pathlib import Path
 from typing import Any, Optional
 
 import more_itertools
-import numpy
 import pandas
 import requests
 from annofabapi.models import ProjectMemberRole, ProjectMemberStatus
@@ -26,8 +25,8 @@ class Member(DataClassJsonMixin):
 
     user_id: str
     member_role: ProjectMemberRole
-    sampling_inspection_rate: Optional[int]
-    sampling_acceptance_rate: Optional[int]
+    sampling_inspection_rate: Optional[int] = None
+    sampling_acceptance_rate: Optional[int] = None
 
 
 class PutProjectMembers(CommandLine):
@@ -44,7 +43,7 @@ class PutProjectMembers(CommandLine):
     def member_exists(members: list[dict[str, Any]], user_id: str) -> bool:
         return PutProjectMembers.find_member(members, user_id) is not None
 
-    def invite_project_member(self, project_id: str, member: Member, old_project_members: list[dict[str, Any]]):  # noqa: ANN201
+    def invite_project_member(self, project_id: str, member: Member, old_project_members: list[dict[str, Any]]) -> dict[str, Any]:
         old_member = self.find_member(old_project_members, member.user_id)
         last_updated_datetime = old_member["updated_datetime"] if old_member is not None else None
 
@@ -58,7 +57,7 @@ class PutProjectMembers(CommandLine):
         updated_project_member = self.service.api.put_project_member(project_id, member.user_id, request_body=request_body)[0]
         return updated_project_member
 
-    def delete_project_member(self, project_id: str, deleted_member: dict[str, Any]):  # noqa: ANN201
+    def delete_project_member(self, project_id: str, deleted_member: dict[str, Any]) -> dict[str, Any]:
         request_body = {
             "member_status": ProjectMemberStatus.INACTIVE.value,
             "member_role": deleted_member["member_role"],
@@ -67,7 +66,7 @@ class PutProjectMembers(CommandLine):
         updated_project_member = self.service.api.put_project_member(project_id, deleted_member["user_id"], request_body=request_body)[0]
         return updated_project_member
 
-    def put_project_members(self, project_id: str, members: list[Member], delete: bool = False):  # noqa: ANN201, FBT001, FBT002
+    def put_project_members(self, project_id: str, members: list[Member], *, delete: bool = False) -> None:
         """
         プロジェクトメンバを一括で登録する。
 
@@ -88,7 +87,7 @@ class PutProjectMembers(CommandLine):
 
         count_invite_members = 0
         # プロジェクトメンバを登録
-        logger.info(f"{project_title} に、{len(members)} 件のプロジェクトメンバを登録します。")
+        logger.info(f"プロジェクト '{project_title}' に、{len(members)} 件のプロジェクトメンバを登録します。")
         for member in members:
             if member.user_id == self.service.api.login_user_id:
                 logger.debug(f"ユーザ '{member.user_id}'は自分自身なので、登録しません。")
@@ -99,7 +98,7 @@ class PutProjectMembers(CommandLine):
                 continue
 
             message_for_confirm = (
-                f"ユーザ '{member.user_id}'を、{project_title} プロジェクトのメンバに登録しますか？member_role={member.member_role.value}"
+                f"ユーザ '{member.user_id}'を、プロジェクト'{project_title}'のメンバーに登録しますか？ member_role='{member.member_role.value}'"
             )
             if not self.confirm_processing(message_for_confirm):
                 continue
@@ -107,14 +106,15 @@ class PutProjectMembers(CommandLine):
             # メンバを登録
             try:
                 self.invite_project_member(project_id, member, old_project_members)
-                logger.debug(f"user_id = {member.user_id}, member_role = {member.member_role.value} のユーザをプロジェクトメンバに登録しました。")
+                logger.debug(f"user_id = '{member.user_id}', member_role = '{member.member_role.value}' のユーザをプロジェクトメンバに登録しました。")
                 count_invite_members += 1
 
-            except requests.exceptions.HTTPError as e:
-                logger.warning(e)
-                logger.warning(f"プロジェクトメンバの登録に失敗しました。user_id = {member.user_id}, member_role = {member.member_role.value}")
+            except requests.exceptions.HTTPError:
+                logger.warning(
+                    f"プロジェクトメンバの登録に失敗しました。user_id = '{member.user_id}', member_role = '{member.member_role.value}'", exc_info=True
+                )
 
-        logger.info(f"{project_title} に、{count_invite_members} / {len(members)} 件のプロジェクトメンバを登録しました。")
+        logger.info(f"プロジェクト'{project_title}' に、{count_invite_members} / {len(members)} 件のプロジェクトメンバを登録しました。")
 
         # プロジェクトメンバを削除
         if delete:
@@ -125,7 +125,7 @@ class PutProjectMembers(CommandLine):
             ]
 
             count_delete_members = 0
-            logger.info(f"{project_title} から、{len(deleted_members)} 件のプロジェクトメンバを削除します。")
+            logger.info(f"プロジェクト '{project_title}' から、{len(deleted_members)} 件のプロジェクトメンバを削除します。")
             for deleted_member in deleted_members:
                 message_for_confirm = f"ユーザ '{deleted_member['user_id']}'を、{project_title} のプロジェクトメンバから削除しますか？"
                 if not self.confirm_processing(message_for_confirm):
@@ -135,31 +135,18 @@ class PutProjectMembers(CommandLine):
                     self.delete_project_member(project_id, deleted_member)
                     logger.debug(f"ユーザ '{deleted_member['user_id']}' をプロジェクトメンバから削除しました。")
                     count_delete_members += 1
-                except requests.exceptions.HTTPError as e:
-                    logger.warning(e)
-                    logger.warning(f"プロジェクトメンバの削除に失敗しました。user_id = '{deleted_member['user_id']}' ")
+                except requests.exceptions.HTTPError:
+                    logger.warning(f"プロジェクトメンバの削除に失敗しました。user_id = '{deleted_member['user_id']}' ", exc_info=True)
 
-            logger.info(f"{project_title} から {count_delete_members} / {len(deleted_members)} 件のプロジェクトメンバを削除しました。")
+            logger.info(f"プロジェクト '{project_title}' から {count_delete_members} / {len(deleted_members)} 件のプロジェクトメンバを削除しました。")
 
     @staticmethod
     def get_members_from_csv(csv_path: Path) -> list[Member]:
-        def create_member(e):  # noqa: ANN001, ANN202
-            return Member(
-                user_id=e.user_id,
-                member_role=ProjectMemberRole(e.member_role),
-                sampling_inspection_rate=e.sampling_inspection_rate,
-                sampling_acceptance_rate=e.sampling_acceptance_rate,
-            )
-
         df = pandas.read_csv(
-            str(csv_path),
-            sep=",",
-            header=None,
-            names=("user_id", "member_role", "sampling_inspection_rate", "sampling_acceptance_rate"),
-            # IDは必ず文字列として読み込むようにする
-            dtype={"user_id": str},
-        ).replace({numpy.nan: None})
-        members = [create_member(e) for e in df.itertuples()]
+            csv_path,
+            dtype={"user_id": "string", "member_role": "string", "sampling_inspection_rate": "Int64", "sampling_acceptance_rate": "Int64"},
+        )
+        members = [Member.from_dict(e) for e in df.to_dict("records")]
         return members
 
     def main(self) -> None:
@@ -184,12 +171,14 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
         type=str,
         required=True,
         help=(
-            "プロジェクトメンバが記載されたCVファイルのパスを指定してください。"
-            "CSVのフォーマットは、「1列目:user_id(required), 2列目:member_role(required), "
-            "3列目:sampling_inspection_rate, 4列目:sampling_acceptance_rate, ヘッダ行なし, カンマ区切り」です。"
-            "member_roleは ``owner``, ``worker``, ``accepter``, ``training_data_user`` のいずれかです。"
-            "sampling_inspection_rate, sampling_acceptance_rate を省略した場合は未設定になります。"
-            "ただし自分自身は登録しません。"
+            "プロジェクトメンバが記載されたCSVファイルのパスを指定してください。"
+            "CSVのフォーマットは、ヘッダあり、カンマ区切りです。\n"
+            " * user_id (required)\n"
+            " * member_role (required)\n"
+            " * sampling_inspection_rate\n"
+            " * sampling_acceptance_rate\n"
+            "member_roleには ``owner``, ``worker``, ``accepter``, ``training_data_user`` のいずれかを指定します。\n"
+            "自分自身は登録できません。"
         ),
     )
 
@@ -202,10 +191,9 @@ def parse_args(parser: argparse.ArgumentParser) -> None:
 
 def add_parser(subparsers: Optional[argparse._SubParsersAction] = None) -> argparse.ArgumentParser:
     subcommand_name = "put"
-    subcommand_help = "プロジェクトメンバを登録する。"
-    description = "プロジェクトメンバを登録する。"
-    epilog = "オーナロールを持つユーザで実行してください。"
+    subcommand_help = "プロジェクトメンバを登録します。"
+    epilog = "オーナーロールを持つユーザで実行してください。"
 
-    parser = annofabcli.common.cli.add_parser(subparsers, subcommand_name, subcommand_help, description, epilog=epilog)
+    parser = annofabcli.common.cli.add_parser(subparsers, subcommand_name, subcommand_help, epilog=epilog)
     parse_args(parser)
     return parser

annofabcli/statistics/list_annotation_count.py

@@ -552,7 +552,7 @@ class AttributeCountCsv:
 
         # アノテーション数の列のNaNを0に変換する
         value_columns = self._value_columns(counter_list, prior_attribute_columns)
-        df = df.fillna({column: 0 for column in value_columns})
+        df = df.fillna(dict.fromkeys(value_columns, 0))
 
         print_csv(df, output=str(output_file), to_csv_kwargs=self.csv_format)
 
@@ -655,7 +655,7 @@ class LabelCountCsv:
 
         # アノテーション数列のNaNを0に変換する
         value_columns = self._value_columns(counter_list, prior_label_columns)
-        df = df.fillna({column: 0 for column in value_columns})
+        df = df.fillna(dict.fromkeys(value_columns, 0))
 
         print_csv(df, output=str(output_file), to_csv_kwargs=self.csv_format)
 

annofabcli/statistics/list_annotation_duration.py

@@ -411,7 +411,7 @@ class AnnotationDurationCsvByAttribute:
 
         # アノテーション数の列のNaNを0に変換する
         value_columns = self._value_columns(annotation_duration_list, prior_attribute_columns)
-        df = df.fillna({column: 0 for column in value_columns})
+        df = df.fillna(dict.fromkeys(value_columns, 0))
         return df
 
 
@@ -473,7 +473,7 @@ class AnnotationDurationCsvByLabel:
 
         # アノテーション数列のNaNを0に変換する
         value_columns = self._value_columns(annotation_duration_list, prior_label_columns)
-        df = df.fillna({column: 0 for column in value_columns})
+        df = df.fillna(dict.fromkeys(value_columns, 0))
 
         return df
 

annofabcli/statistics/visualization/dataframe/productivity_per_date.py

@@ -181,17 +181,17 @@ class AbstractPhaseProductivityPerDate(abc.ABC):
 
         # その他の欠損値（作業時間や生産量）を0で埋める
         df2 = df2.fillna(
-            {
-                col: 0
-                for col in [
+            dict.fromkeys(
+                [
                     "annotation_worktime_hour",
                     "inspection_worktime_hour",
                     "acceptance_worktime_hour",
                     "task_count",
                     "inspection_comment_count",
                     *self.production_volume_columns,
-                ]
-            }
+                ],
+                0,
+            )
         )
 
         return df2