flixopt 3.0.3__py3-none-any.whl → 3.1.1__py3-none-any.whl

flixopt/__init__.py

@@ -7,7 +7,7 @@ from importlib.metadata import PackageNotFoundError, version
 
 try:
     __version__ = version('flixopt')
-except PackageNotFoundError:
+except (PackageNotFoundError, TypeError):
     # Package is not installed (development mode without editable install)
     __version__ = '0.0.0.dev0'
 

flixopt/components.py

@@ -1304,16 +1304,18 @@ class Sink(Component):
         prevent_simultaneous_flow_rates: bool = False,
         **kwargs,
     ):
-        """
-        Initialize a Sink (consumes flow from the system).
-
-        Supports legacy `sink=` keyword for backward compatibility (deprecated): if `sink` is provided it is used as the single input flow and a DeprecationWarning is issued; specifying both `inputs` and `sink` raises ValueError.
-
-        Parameters:
-            label (str): Unique element label.
-            inputs (list[Flow], optional): Input flows for the sink.
-            meta_data (dict, optional): Arbitrary metadata attached to the element.
-            prevent_simultaneous_flow_rates (bool, optional): If True, prevents simultaneous nonzero flow rates across the element's inputs by wiring that restriction into the base Component setup.
+        """Initialize a Sink (consumes flow from the system).
+
+        Supports legacy `sink=` keyword for backward compatibility (deprecated): if `sink` is provided
+        it is used as the single input flow and a DeprecationWarning is issued; specifying both
+        `inputs` and `sink` raises ValueError.
+
+        Args:
+            label: Unique element label.
+            inputs: Input flows for the sink.
+            meta_data: Arbitrary metadata attached to the element.
+            prevent_simultaneous_flow_rates: If True, prevents simultaneous nonzero flow rates
+                across the element's inputs by wiring that restriction into the base Component setup.
 
         Note:
             The deprecated `sink` kwarg is accepted for compatibility but will be removed in future releases.

flixopt/effects.py

@@ -480,19 +480,17 @@ class EffectCollection:
     def create_effect_values_dict(
         self, effect_values_user: PeriodicEffectsUser | TemporalEffectsUser
     ) -> dict[str, Scalar | TemporalDataUser] | None:
-        """
-        Converts effect values into a dictionary. If a scalar is provided, it is associated with a default effect type.
-
-        Examples
-        --------
-        effect_values_user = 20                               -> {'<standard_effect_label>': 20}
-        effect_values_user = {None: 20}                       -> {'<standard_effect_label>': 20}
-        effect_values_user = None                             -> None
-        effect_values_user = {'effect1': 20, 'effect2': 0.3}  -> {'effect1': 20, 'effect2': 0.3}
-
-        Returns
-        -------
-        dict or None
+        """Converts effect values into a dictionary. If a scalar is provided, it is associated with a default effect type.
+
+        Examples:
+            ```python
+            effect_values_user = 20                               -> {'<standard_effect_label>': 20}
+            effect_values_user = {None: 20}                       -> {'<standard_effect_label>': 20}
+            effect_values_user = None                             -> None
+            effect_values_user = {'effect1': 20, 'effect2': 0.3}  -> {'effect1': 20, 'effect2': 0.3}
+            ```
+
+        Returns:
             A dictionary keyed by effect label, or None if input is None.
             Note: a standard effect must be defined when passing scalars or None labels.
         """

flixopt/elements.py

@@ -207,6 +207,10 @@ class Bus(Element):
                 logger.warning(
                     f'In Bus {self.label_full}, the excess_penalty_per_flow_hour is 0. Use "None" or a value > 0.'
                 )
+        if len(self.inputs) == 0 and len(self.outputs) == 0:
+            raise ValueError(
+                f'Bus "{self.label_full}" has no Flows connected to it. Please remove it from the FlowSystem'
+            )
 
     @property
     def with_excess(self) -> bool:

flixopt/interface.py

@@ -963,6 +963,11 @@ class InvestParameters(Interface):
                 raise TypeError(
                     f'If you provide a tuple to "linked_periods", it needs to be len=2. Got {len(self.linked_periods)=}'
                 )
+            if flow_system.periods is None:
+                raise ValueError(
+                    f'Cannot use linked_periods={self.linked_periods} when FlowSystem has no periods defined. '
+                    f'Please define periods in FlowSystem or use linked_periods=None.'
+                )
             logger.debug(f'Computing linked_periods from {self.linked_periods}')
             start, end = self.linked_periods
             if start not in flow_system.periods.values: