Skip to content

Commit

Permalink
Merge pull request #66 from citysciencelab/fix-result-handling
Browse files Browse the repository at this point in the history
fix: remove guessing result paths
  • Loading branch information
hwbllmnn authored Oct 31, 2024
2 parents 6664573 + 8f9d09e commit e9ca5c9
Show file tree
Hide file tree
Showing 3 changed files with 48 additions and 6 deletions.
51 changes: 47 additions & 4 deletions poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,7 @@ sqlalchemy-serializer = "^1.4.22"
watchdog = "^5.0.3"
ema-workbench = "^2.5.2"
names-generator = "^0.2.0"
ipyparallel = "^8.8.0"

[tool.poetry.group.dev.dependencies]
# formatting, quality, tests
Expand Down
2 changes: 0 additions & 2 deletions src/ump/api/job.py
Original file line number Diff line number Diff line change
Expand Up @@ -337,8 +337,6 @@ async def results_to_geoserver(self):
p = providers.PROVIDERS[self.provider_prefix]['processes'][self.process_id]

results = await self.results()
while "results" in results:
results = results["results"]
if 'result-path' in p:
parts = p['result-path'].split('.')
for part in parts:
Expand Down

0 comments on commit e9ca5c9

Please sign in to comment.