From 90354a775180fe72deebb124d3b5aba0e16ae7a7 Mon Sep 17 00:00:00 2001 From: Alex Burlacu Date: Thu, 23 Mar 2023 13:16:06 +0200 Subject: [PATCH] Fix task.get_user_properties when called with value_only=True --- clearml/backend_interface/task/hyperparams.py | 1 - clearml/task.py | 3 +-- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/clearml/backend_interface/task/hyperparams.py b/clearml/backend_interface/task/hyperparams.py index cbf10a72..9d924a47 100644 --- a/clearml/backend_interface/task/hyperparams.py +++ b/clearml/backend_interface/task/hyperparams.py @@ -42,7 +42,6 @@ class HyperParams(object): selector and not selector(item) ): continue - item = item if not projector else projector(item) if return_obj: item = tasks.ParamsItem() hyperparams[item.get("section")][item.get("name")] = ( diff --git a/clearml/task.py b/clearml/task.py index 05be328a..f0c2ad1f 100644 --- a/clearml/task.py +++ b/clearml/task.py @@ -8,7 +8,6 @@ import threading import time from argparse import ArgumentParser from logging import getLogger -from operator import attrgetter from tempfile import mkstemp, mkdtemp from zipfile import ZipFile, ZIP_DEFLATED @@ -2180,7 +2179,7 @@ class Task(_Task): section = "properties" params = self._hyper_params_manager.get_hyper_params( - sections=[section], projector=attrgetter("value") if value_only else None + sections=[section], projector=(lambda x: x.get("value")) if value_only else None ) return dict(params.get(section, {}))