ClearML/clearml
1
0
Fork 0
mirror of https://github.com/clearml/clearml synced 2025-03-03 10:42:00 +00:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Fix set_base_docker in remote execution

Browse Source
This commit is contained in:
allegroai 2021-03-06 00:47:06 +02:00
parent 1374543b93
commit 4106aa9c49
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
clearml/backend_interface/task/task.py
View File

@ -1167,9 +1167,6 @@ class Task(IdObjectBase, AccessMixin, SetupUploadMixin):
inside the provided docker image.
When running remotely the call is ignored
"""
if not self.running_locally():
return
with self._edit_lock:
self.reload()
execution = self.data.execution
@ -1622,6 +1619,9 @@ class Task(IdObjectBase, AccessMixin, SetupUploadMixin):
:param str package_name: The package name to add to the "Installed Packages" section of the task.
:param package_version: The package version requirements. If ``None``, then use the installed version.
"""
if not running_remotely() and hasattr(cls, 'current_task') and cls.current_task():
get_logger('task').warning(
'Requirement ignored, Task.add_requirements() must be called before Task.init()')
cls._force_requirements[package_name] = package_version
def _get_models(self, model_type='output'):