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

Contribution graphs > Rest should be split into positive and negative sections #886

Open
marc-vdm opened this issue Nov 27, 2022 · 0 comments
Labels
feature Issues/PRs related to a new feature good first issue Issues to get started with AB development with

Comments

@marc-vdm
Copy link
Member

marc-vdm commented Nov 27, 2022

Currently in contribution graphs, AB shows a rest containing a remainder of processes. While this is great, the rest is an aggregate of potentially hundreds or thousands of processes when using common background databases like ecoinvent. Among this 'rest' section, some number of processes will have a negative impact instead of a positive one, though often, the sum of 'rest' will be positive (though lower, as these negative numbers would also be counted, lowering the total sum).

It may be better to split rest up based on the sign of the processes (should be easy in pandas with just a score > 0 selection). This way, AB can show that the remainder has both a positive and a negative impact, which would be more realistic.

Figure below is an example of such a case. I know that in this system there are negative flows for one of the reference flows, but this representation of the contributions only shows a positive 'rest'.
image

For implementation, I think we could keep the same color for both 'rest' flows, just rename them to rest (positive) and rest (negative).

Related issues: #647, #782, #635.

Good place to start is here:

('Rest', ''): C[col, :].sum() - top_contribution[:, 0].sum(),

@marc-vdm marc-vdm added the feature Issues/PRs related to a new feature label Nov 27, 2022
@marc-vdm marc-vdm added the good first issue Issues to get started with AB development with label Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Issues/PRs related to a new feature good first issue Issues to get started with AB development with
Projects
None yet
Development

No branches or pull requests

1 participant