BioSTEAMDevelopmentGroup / Bioindustrial-Park

BioSTEAM's Premier Repository for Biorefinery Models and Results
MIT License
37 stars 18 forks source link

unexpected keyword error when importing lactic #42

Closed cantwell closed 1 year ago

cantwell commented 1 year ago

When I import biorefineries on a fresh install:

`from biorefineries import lactic as la Traceback (most recent call last):

File "C:\Users\xx\AppData\Local\Temp\ipykernel_14892\1286369224.py", line 1, in from biorefineries import lactic as la

File "C:\Users\xx\Anaconda3\envs\biosteam\lib\site-packages\biorefineries\lactic__init__.py", line 41, in from ._tea import *

File "C:\Users\xx\Anaconda3\envs\biosteam\lib\site-packages\biorefineries\lactic_tea.py", line 16, in from biorefineries.cornstover import CellulosicEthanolTEA

File "C:\Users\xx\Anaconda3\envs\biosteam\lib\site-packages\biorefineries\cornstover__init__.py", line 10, in from . import (units,

File "C:\Users\xx\Anaconda3\envs\biosteam\lib\site-packages\biorefineries\cornstover\systems.py", line 772, in @bst.SystemFactory(

TypeError: new() got an unexpected keyword argument 'optional_outs_index'`

If I comment out the offending line, the error goes away, but I'm not sure what else it's tied to. It's not critical.

yalinli2 commented 1 year ago

Hi @cantwell , are you using pip installed versions of biosteam/thermosteam/biorefineries? If so, can you clone these repositories? All of them have gone through major structural changes so the biosteam/thermosteam you are using might not match the version of the biorefineries

When using the master branch of these biorefineries, I can load the lactic biorefinery using

from biorefineries import lactic as la
la.load()

Thank you!

yalinli2 commented 1 year ago

Actually, if you can get to #43 (I'm looking at it now), you might have figured out the importing...?

yalinli2 commented 1 year ago

closing the issue since it has been fixed