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 warning if data == 0 #93

Merged
merged 2 commits into from
Nov 14, 2023
Merged

add warning if data == 0 #93

merged 2 commits into from
Nov 14, 2023

Conversation

behinger
Copy link
Member

No description provided.

@codecov-commenter
Copy link

Codecov Report

Merging #93 (407d129) into main (372accc) will decrease coverage by 0.13%.
Report is 19 commits behind head on main.
The diff coverage is 60.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main      #93      +/-   ##
==========================================
- Coverage   69.44%   69.32%   -0.13%     
==========================================
  Files          17       17              
  Lines         635      639       +4     
==========================================
+ Hits          441      443       +2     
- Misses        194      196       +2     
Files Coverage Δ
src/plot_topoplot.jl 76.47% <60.00%> (-8.15%) ⬇️

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

Copy link
Collaborator

@vladdez vladdez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would change the text on smth like:

The min and max of the value represented by the color are the same, it seems that the data values are identical. 
We disable the color bar in this figure.
Note: The identical min and max may cause an interpolation error when plotting the topoplot.

It took we a while to understand the initial error message, maybe more explicit message will be more human friendly.

@behinger
Copy link
Member Author

sounds good!

@vladdez vladdez merged commit 6a0bd28 into main Nov 14, 2023
2 checks passed
@behinger
Copy link
Member Author

200w

@behinger behinger deleted the topowarn branch November 28, 2023 10:23
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