-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
unclear mass flow results, maybe bug #50
Comments
Hi @cardosan The main problem is that the example model predates the mass flow diagram so:
The code to draw the mass flow diagram is designed to ignore items with non-mass units - usually energy processes. The problem is litre is also a non-mass unit, and because the densities of all of the different things you can measure in litres vary wildly, you can't really include litres as a mass unit. Because the example was written before I had the idea for the mass flow diagram, the fact that the example model uses litres (which previously made no difference) now means the diagram doesn't work. In terms of your fixes: The The idea is that it can be set for the first level in the tree, but defaults back to false for the rest of the recursion through the tree. As you found out, if you set it to By adding litres to the allowed units you've kind of fluked upon the correct solution for this particular model, but only because one litre of water is equal to 1kg... The weird bit with the tea leaves is because the masses don't quite balance. The solution is that I write a new version of the example model, where the intermediates are in kg, and the masses balance properly - I'll add it to the to do list! |
Hi James (or Peter?)
I was puzzled by the results of the mass flow diagram of the lcopt example.
While the
black tea
shows nothing. This is what I get with theTea cup
as functional unit for themilky tea
Which is not was was expecting
Tried to set
allow_no_mass
in bothrecurse_mass
hereCloser to what I was expecting but still weird (see electricity)
the only way I get something closer to what I would expect is adding also litres in the allowed units here and here
Have the feeling something need to be fixed but do not have time to further investigate now due to a crying daughter :)
The text was updated successfully, but these errors were encountered: