Commit 9ff47838 authored by Eric Duminil's avatar Eric Duminil
Browse files

Warning?

parent 98531bc7
...@@ -11,7 +11,9 @@ from .solar_potential import SolarPotentialResults ...@@ -11,7 +11,9 @@ from .solar_potential import SolarPotentialResults
def create_simstadt_results(description: str, output_files: list) -> SimStadtResults: def create_simstadt_results(description: str, output_files: list) -> SimStadtResults:
"""Factory: detect workflow type from params.xml and return the correct subclass.""" """Factory: detect workflow type from params.xml and return the correct subclass."""
provider = get_workflow_provider(output_files) provider = get_workflow_provider(output_files)
return SimStadtResults.create(provider, description=description, output_files=sorted(output_files)) return SimStadtResults.create(
provider, description=description, output_files=sorted(output_files)
)
__all__ = [ __all__ = [
......
...@@ -240,5 +240,6 @@ class SimStadtResults(ABC): ...@@ -240,5 +240,6 @@ class SimStadtResults(ABC):
@classmethod @classmethod
def create(cls, provider: str, **kwargs) -> "SimStadtResults": def create(cls, provider: str, **kwargs) -> "SimStadtResults":
if provider not in cls._registry: if provider not in cls._registry:
# TODO: Warning, and return GenericSimStadtResult instead?
raise ValueError(f"Unknown workflow provider: {provider}") raise ValueError(f"Unknown workflow provider: {provider}")
return cls._registry[provider](**kwargs) return cls._registry[provider](**kwargs)
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment