Skip to content

Commit

Permalink
bug(job.modules): reassigning job modules failed
Browse files Browse the repository at this point in the history
Becuase it applied to a different reference

Note: it is worth auditng Job class and using the correct references to self.
  • Loading branch information
wiwichips committed Sep 4, 2024
1 parent 078c33b commit b7784bb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion dcp/api/job.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ def __init__(self, job_js):

self._wrapper_set_attribute("serializers", default_serializers)
self._wrapper_set_attribute("env", {})
self._wrapper_set_attribute("modules", [])
job_js.modules = [] #TODO: why is this only done this way for job modules?
self._wrapper_set_attribute("fs", JobFS())
self._wrapper_set_attribute("_exec_called", False)
self.aio.exec = self._exec;
Expand Down

0 comments on commit b7784bb

Please sign in to comment.