Closed jwahlstrand closed 2 months ago
Attention: Patch coverage is 0%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 85.74%. Comparing base (
27fb192
) to head (6f632bd
).:exclamation: Current head 6f632bd differs from pull request most recent head ca4de2b
Please upload reports for the commit ca4de2b to get more accurate results.
Files | Patch % | Lines |
---|---|---|
src/contrast_gui.jl | 0.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I have once again checked the implementation of the contrast GUI and noticed several design issues. However, I think it is a good idea to release this first as a quick fix.
Thanks. The code does somehow seem a bit complicated for what it does. I'll merge this now and do another release.
Implements suggestion from @kimikage
Fixes #310
I verified that this still avoids the crash in #308