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

Fix bar plot vector attribute with missing values #4736

Merged
merged 2 commits into from
May 4, 2023

Conversation

yha
Copy link
Member

@yha yha commented Apr 28, 2023

Description

Fixes vector attributes on bar plots with missing values.
Before:

plot(bar(-2:2, [2,-2,1,-1,1], color=1:5),
       bar(-2:2, [2,-2,NaN,-1,1], color=1:5))

plot_27
After:
plot_47

Attribution

@codecov
Copy link

codecov bot commented May 4, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.40 🎉

Comparison is base (349eea4) 89.96% compared to head (baec309) 90.37%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4736      +/-   ##
==========================================
+ Coverage   89.96%   90.37%   +0.40%     
==========================================
  Files          40       40              
  Lines        8383     8735     +352     
==========================================
+ Hits         7542     7894     +352     
  Misses        841      841              
Impacted Files Coverage Δ
src/examples.jl 99.20% <ø> (+3.25%) ⬆️
src/recipes.jl 84.85% <100.00%> (+0.37%) ⬆️

... and 31 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@BeastyBlacksmith BeastyBlacksmith merged commit b329bb7 into JuliaPlots:master May 4, 2023
BeastyBlacksmith added a commit that referenced this pull request May 25, 2023
* Fix bar plot vector attribute with missing values

* format

---------

Co-authored-by: Simon Christ <[email protected]>
BeastyBlacksmith added a commit that referenced this pull request May 26, 2023
…t) (#4738)

* CompatHelper: bump compat for StatsBase to 0.34, (keep existing compat)

* Plots 1.38.11

* Fix bar plot vector attribute with missing values (#4736)

* Fix bar plot vector attribute with missing values

* format

---------

Co-authored-by: Simon Christ <[email protected]>

* dev [skip ci]

* Short-circuit `update_clims` if not necessary (#4743)

* Fix straightline filling for PGFPlotsX backend (#4746)

* fill whole line

* adjust to PGFPlotsX v1.6

* Update src/backends/pgfplotsx.jl

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* remove linebreak

* remove another linebreak

* avoid colon in title

* Plots 1.38.12

* dev [skip ci]

* Fix UnitfulExt for PGFPlotsX take 2 (#4747)

* tests and migrate unitformat

* fix tests

* format

* fix tests

* adjust empty label case

* add imports

* add compat

* ditch 1.6

* Revert "ditch 1.6"

This reverts commit 1f0ce95.

* change imports

* fix ci config

---------

Co-authored-by: CompatHelper Julia <[email protected]>
Co-authored-by: Simon Christ <[email protected]>
Co-authored-by: Yuval <[email protected]>
Co-authored-by: Simon Christ <[email protected]>
Co-authored-by: Nicholas Bauer <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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