Skip to content

Commit

Permalink
Merge pull request #9 from molssi-seamm/dev
Browse files Browse the repository at this point in the history
Bugfix: boolean options when running a job
  • Loading branch information
seamm authored Nov 15, 2023
2 parents f266548 + bcc04f5 commit 07855fd
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions HISTORY.rst
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
=======
History
=======
2023.11.15 -- Bugfix: boolean options now work
* Boolean options were not handled correctly when submitting jobs.

2023.10.24 -- Improvement for job handling.
* Added control parameters to the data stored for the job, to support filling out
Expand Down
2 changes: 1 addition & 1 deletion seamm_dashboard_client/dashboard.py
Original file line number Diff line number Diff line change
Expand Up @@ -455,7 +455,7 @@ def submit(
for name, data in variables.value.items():
if data["optional"] == "Yes":
if data["type"] == "bool":
if values[name] == "Yes":
if values[name]:
optional.append(f"--{name}")
elif data["type"] == "file":
if data["nargs"] == "a single value":
Expand Down

0 comments on commit 07855fd

Please # to comment.