review

aiida_optimade/cli/cmd_calc.py

@@ -40,6 +40,8 @@ def calc(obj: dict, fields: Tuple[str], force_yes: bool, silent: bool):
     from aiida import load_profile
     from aiida.cmdline.utils import echo
 
+    # The default aiida.cmdline loglevel inherit from aiida loglevel is REPORT
+    # Here we use INFO loglevel for the operations
     echo.CMDLINE_LOGGER.setLevel("INFO")
 
     try:

aiida_optimade/translators/entities.py

@@ -157,14 +157,14 @@ def _store_attributes_node_extra(self) -> None:
             "(%s) Updating Node %s in AiiDA DB!", profile.storage_backend, self._pk
         )
         if profile.storage_backend == "psql_dos":
-            # TODO: this can be `set_extra` directly? is there performance issue with sqlite_zip?
+            # TODO: this can be `set_extra` directly? is there performance issue?
             from aiida.storage.psql_dos.models.node import DbNode
 
             with get_manager().get_backend().transaction() as session:
                 session.query(DbNode).filter(DbNode.id == self._pk).update(
                     values={"extras": extras}
                 )
-        # TODO: should be also support sqlite_zip
+
         else:
             raise AiidaError(
                 f'Unknown AiiDA backend "{profile.database_backend}" for profile'