Skip to content

filter non-defined elementary flows#71

Open
mfastudillo wants to merge 1 commit intoPascalLesage:masterfrom
mfastudillo:missing_cf
Open

filter non-defined elementary flows#71
mfastudillo wants to merge 1 commit intoPascalLesage:masterfrom
mfastudillo:missing_cf

Conversation

@mfastudillo
Copy link

Hi!

This is a pull request related to the issue identified here where the calculation crashes if we use samples of characterisation factors of elementary flows that are not present in the database.

I think this would fix the problem, I hope it does not break anything. It would be cool to have in the readme some instructions about how to run the tests :)

@PascalLesage
Copy link
Owner

Hi @mfastudillo, and thanks!
Unfortunately, the proposed strategy for filtering out the characterization factors for elementary flows not in the biosphere matrix does not work: https://ci.appveyor.com/project/PascalLesage/presamples/builds/35677211/job/s43vg7w6pb3nnesr
You can always check the test results directly on github when you commit new changes.
Also, you should include new tests to make sure your filter is working.
Looking forward to reviewing the next iteration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants