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

#3457 fixed by picking entry by its X value instead of array index. T… #3459

Conversation

shineeth
Copy link

…his will avoid setting x offset on a wrong data entry

Issue Link 🔗

#3457

Implementation Details 🚧

#3457 fixed by picking entry by its X value instead of array index in BarChartData class . This will avoid setting x offset on a wrong data entry

… index. This will avoid setting x offset on a wrong data entry
@codecov-io
Copy link

Codecov Report

Merging #3459 into master will increase coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3459      +/-   ##
==========================================
+ Coverage   29.21%   29.21%   +<.01%     
==========================================
  Files         117      117              
  Lines       13303    13300       -3     
==========================================
  Hits         3886     3886              
+ Misses       9417     9414       -3
Impacted Files Coverage Δ
...s/Data/Implementations/Standard/BarChartData.swift 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc73ba7...699f1c5. Read the comment docs.

@shineeth shineeth closed this May 22, 2018
@shineeth shineeth deleted the hotfix/#3457-grouped-bar-chart-plotting-wrong branch May 22, 2018 08:12
@shineeth shineeth restored the hotfix/#3457-grouped-bar-chart-plotting-wrong branch May 22, 2018 08:21
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