Skip to content

Conversation

UsamaEquinorAFK
Copy link
Collaborator

No description provided.

@UsamaEquinorAFK UsamaEquinorAFK force-pushed the downloadtable branch 2 times, most recently from 5fa0d1b to a759bb8 Compare October 16, 2025 09:24
@lars-petter-hauge
Copy link
Collaborator

solves #511

@lars-petter-hauge
Copy link
Collaborator

when the data is downloaded we also include all components that don't have a value. We should remove them and only include the ones with a number.

technically we would like to only show the ones where at least one row has a value greater than 0, with the special case that we want to show output columns even with all 0, with the input column does have value greater than 0 (so if one starts with 10 ppm H2S, the 0 ppm H2S in the output has a meaning)

@UsamaEquinorAFK UsamaEquinorAFK force-pushed the downloadtable branch 2 times, most recently from d8b1831 to b0775cd Compare October 17, 2025 13:36
@UsamaEquinorAFK
Copy link
Collaborator Author

when the data is downloaded we also include all components that don't have a value. We should remove them and only include the ones with a number.

technically we would like to only show the ones where at least one row has a value greater than 0, with the special case that we want to show output columns even with all 0, with the input column does have value greater than 0 (so if one starts with 10 ppm H2S, the 0 ppm H2S in the output has a meaning)

Fixed :))

@UsamaEquinorAFK UsamaEquinorAFK merged commit 7ab493f into main Oct 17, 2025
9 checks passed
@UsamaEquinorAFK UsamaEquinorAFK deleted the downloadtable branch October 17, 2025 13:41
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.

3 participants