Skip to content

Commit

Permalink
Merge pull request deepmodeling#68 from dptech-corp/zjgemi
Browse files Browse the repository at this point in the history
fix: handle big parameter whose name changes
  • Loading branch information
zjgemi authored Jun 14, 2022
2 parents 685eba5 + 7513567 commit 671cf94
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 2 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
# CHANGELOG

## 1.0.19

### Fix

- fix: handle big parameter whose name changes

## 1.0.18

### Added
Expand Down
2 changes: 1 addition & 1 deletion VERSION
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1.0.18
1.0.19
4 changes: 3 additions & 1 deletion src/dflow/argo_objects.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,9 @@ def handle_big_parameters(self, io):
io.parameters = {}
with tempfile.TemporaryDirectory() as tmpdir:
download_artifact(art, path=tmpdir)
with open(os.path.join(tmpdir, name[13:]), "r") as f:
fs = os.listdir(tmpdir)
assert len(fs) == 1
with open(os.path.join(tmpdir, fs[0]), "r") as f:
content = jsonpickle.loads(f.read())
param = {"name": name[13:], "save_as_artifact": True}
if "type" in content:
Expand Down

0 comments on commit 671cf94

Please sign in to comment.