Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fixes brushing error on plot. #226

Merged
merged 2 commits into from
Apr 25, 2024
Merged

fixes brushing error on plot. #226

merged 2 commits into from
Apr 25, 2024

Conversation

kartikeyakirar
Copy link
Contributor

@kartikeyakirar kartikeyakirar added bug Something isn't working core labels Apr 23, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 23, 2024 •
edited
Loading

badge

Code Coverage Summary

Filename                           Stmts    Miss  Cover    Missing
-------------------------------  -------  ------  -------  ---------
R/g_boxplot.R                        116     116  0.00%    147-310
R/g_correlationplot.R                135     135  0.00%    251-419
R/g_density_distribution_plot.R       86      86  0.00%    125-236
R/g_lineplot.R                       275     275  0.00%    262-606
R/g_scatterplot.R                    130     130  0.00%    142-310
R/g_spaghettiplot.R                  101     101  0.00%    243-376
R/geom_axes_line.R                   167     167  0.00%    48-360
R/t_summarytable.R                   102     102  0.00%    87-224
R/utils.R                             70      70  0.00%    17-137
TOTAL                               1182    1182  0.00%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: aca0f00

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@kartikeyakirar kartikeyakirar mentioned this pull request Apr 24, 2024
Closed
vedhav
vedhav approved these changes Apr 25, 2024
View reviewed changes
Copy link
Contributor

@vedhav vedhav left a comment

Choose a reason for hiding this comment

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

Verified that all these plots work in teal.goshawk 👌🏽

@kartikeyakirar kartikeyakirar merged commit 62e01a6 into main Apr 25, 2024
20 checks passed
@kartikeyakirar kartikeyakirar deleted the 539_fix_brushing_issue@main branch April 25, 2024 07:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@vedhav vedhav vedhav approved these changes

Assignees

@vedhav vedhav

Labels
bug Something isn't working core
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kartikeyakirar @vedhav