search

BioSTEAMDevelopmentGroup / Bioindustrial-Park

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

Little error for BioSTEAM #72

Closed zasddsgg closed 1 year ago

zasddsgg commented 1 year ago

I seem to find a little error for BioSTEAM when using BioSTEAM. Could I bother you check them? Thanks a lot. Wish you a good day.

a)In the fermentation area of corn stover ethanol model, I found by calling R303 that the outlet stream temperature of SaccharificationAndCoFermentation unit R303 seemed to be 48 degrees (enzymatic hydrolysis temperature) rather than 32 degrees (fermentation temperature). I checked the source code (https://github.com/BioSTEAMDevelopmentGroup/Bioindustrial-Park/blob/master/biorefineries/cellulosic/units.py), although vent. T = effluent. T = self. T_fermentation was set in Class SaccharificationAndCoFermentation (Unit), effluent.copy_like(feed) seems to determine that the effluent temperature is the temperature of the feed stream (48 degrees), resulting in the flow temperature of R303 to M401 is 48 degrees, but it should be 32 degrees. In addition, The effluent.T in self.add_heat_utility(reactor_duty + batch_duty, effluent.T) also seems to be 48 degrees, so the heat cooling from 48 degrees to 32 degrees does not seem to be accounted for in R303's heat duty (R303.Hnet)?

b)For page 29 in Humbird 2011 report, Table 10 gives DAP a load of 0.67 g/L, but it seems that BioSTEAM used 0.33 g/L (https://github.com/BioSTEAMDevelopmentGroup/Bioindustrial-Park/blob/master/biorefineries/cellulosic/systems/fermentation/integrated_bioprocess.py)?

yoelcortes commented 1 year ago

@zasddsgg, thanks for reporting these!

a) Yes, this was actually fixed in another branch, but I'm a little delayed to merge it. I went ahead and fixed it in the master branch since this seems to be important.

b) Nice catch, the seed train DAP is 0.67 g/L while the Cofermentation DAP is 0.33 g/L. This is now fixed in the master branch.

Thanks!