-
Notifications
You must be signed in to change notification settings - Fork 9
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
More energy demos #79
Comments
@andi-huber Similar to #91 there are also issues with CO2CarDemo. It should show 1033.9838600 kg for the "Ferrari-Enzo6" but only does 1.0339838600 kg. I will have a look but if you got a chance to address it in a similar way, it would be appreciated. |
ok no problem ... should be able to find some time for it next week |
The calculation really feels awkward, what was the exact requirement to change this calculation to always use a system unit and can you point us to the ticket where this requirement was defined? I recall it was only for a very limited case like |
I'll look into this. Should not be too hard to restore the previous multiplication behavior in Indriya. (Introducing support for doing calculation with relative scoped quantities was tricky, I should clean this up a bit anyway.) |
It seems that cases like unitsofmeasurement/indriya#288 were a reason for this, but maybe the behavior should be restricted to e.g. |
Self assigned unitsofmeasurement/indriya#294 |
CO2CarDemo fixed |
Following #10 we should improve them by adding more demos, e.g. based on https://co2.myclimate.org/de/flight_calculators/new
The text was updated successfully, but these errors were encountered: