Skip to content
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

add static lcia score for from_timeline approach #75

Open
jakobsarthur opened this issue Jul 30, 2024 · 1 comment
Open

add static lcia score for from_timeline approach #75

jakobsarthur opened this issue Jul 30, 2024 · 1 comment
Labels
bug Something isn't working enhancement New feature or request

Comments

@jakobsarthur
Copy link
Collaborator

Currently, the the matrix is not expanded, the static lcia score (using prospective db's but no time explicit emissions) can not be calculated. This is normally done by just doing the matrix multiplication with the expanded matrices. But these are not built, so it gives a wrong score.

-> solution: sum the dynamic inventory over time and multiply with characterisation factor.

Cuurently it produces a result which is wrong...! So it should at least produce a warning to calculate the matrix based approach

@jakobsarthur jakobsarthur added bug Something isn't working enhancement New feature or request labels Jul 30, 2024
@jakobsarthur
Copy link
Collaborator Author

jakobsarthur commented Jul 30, 2024

static_lcia() now raises an error if the timexlca.lci() was run with expand_technosphere=False

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant