search

watertap-org / parameter-sweep

https://parameter-sweep.readthedocs.io
Other
0 stars 5 forks source link

Resolve Pysmo sampling causing a requirement on `idaes-pse` #5

Open lbianchi-lbl opened 7 months ago

lbianchi-lbl commented 7 months ago

https://github.com/watertap-org/parameter-sweep/blob/b3023b5d9d0a6b911669270081b023411d4f273f/src/parameter_sweep/parameter_sweep.py#L21

Possible solutions (in rough order of increasing effort needed):

  1. Make the availability of the Pysmo sampling schemes contingent on whether Pysmo is importable (i.e. making idaes-pse an optional, rather than required, dependency)
  2. Copy the code from Pysmo to this repository
  3. Look into whether Pysmo can be made distributable as a standalone package independently from idaes-pse