Skip to content

Commit

Permalink
Merge pull request #191 from agarny/issue190
Browse files Browse the repository at this point in the history
oSPARC: need to account for the release of the new Web API
  • Loading branch information
alan-wu authored Nov 8, 2023
2 parents 7be81e7 + 64b1e9b commit 52b9dfe
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions app/osparc/osparc.py
Original file line number Diff line number Diff line change
Expand Up @@ -152,12 +152,10 @@ def check_simulation(data):
solver_version = data["solver"]["version"]
status = solvers_api.inspect_job(solver_name, solver_version, job_id)

if status.progress == 100:
# The simulation has completed, but was it successful?

if status.state != "SUCCESS":
raise SimulationException("the simulation failed")
if status.state == "FAILED":
raise SimulationException("the simulation failed")

if status.state == "SUCCESS":
# Retrieve the simulation job outputs.

try:
Expand Down

0 comments on commit 52b9dfe

Please sign in to comment.