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.

Sign up for GitHub

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

Jump to bottom

Updates in lollipops #768

Merged
merged 12 commits into from
Apr 28, 2023
Merged

Updates in lollipops #768

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
351997c
Add conditional including of zero to the Y-range for the lollipop geo…
ASmirnov-HORIS Apr 24, 2023
47294a8
Increase multiplicative expand for the continuous scale when geometry…
ASmirnov-HORIS Apr 25, 2023
961d56b
Update the cookbook-notebook for the lollipop geometry.
ASmirnov-HORIS Apr 25, 2023
b8f4b9d
Refactor continuous scale expanding for the lollipop geometry.
ASmirnov-HORIS Apr 25, 2023
68d251f
Refactor scale expand parameters for the lollipop geometry.
ASmirnov-HORIS Apr 25, 2023
b0bf374
Fix rangeIncludesZero() for lollipops.
ASmirnov-HORIS Apr 25, 2023
c449f99
Rerun cookbook-notebook for the lollipop geometry.
ASmirnov-HORIS Apr 25, 2023
9824de6
Change behaviour of the dir parameter for the lollipop geometry.
ASmirnov-HORIS Apr 27, 2023
163d7bf
Undo expanding scales for lollipops.
ASmirnov-HORIS Apr 28, 2023
ce2bcd2
Update API for geom_lollipop() (change behaviour of the dir parameter).
ASmirnov-HORIS Apr 28, 2023
74d26eb
Fix the future_changes.md file.
ASmirnov-HORIS Apr 28, 2023
b308b7b
Rebuild the cookbook-notebook for the lollipop geometry.
ASmirnov-HORIS Apr 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix rangeIncludesZero() for lollipops.
  • Loading branch information
@ASmirnov-HORIS
ASmirnov-HORIS committed Apr 25, 2023
commit b0bf374bea056ce4f992de7cdc1fef4088a957a7
3 changes: 2 additions & 1 deletion ...rtable/src/commonMain/kotlin/jetbrains/datalore/plot/builder/assemble/GeomLayerBuilder.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,8 @@ class GeomLayerBuilder(
@Suppress("NAME_SHADOWING")
val aes = aes.afterOrientation(isYOrientation)
return if (geom is LollipopGeom && aes == Aes.Y) {
geom.direction == LollipopGeom.Direction.ORTHOGONAL_TO_AXIS && geom.slope == 0.0 && geom.intercept == 0.0
// Pin the lollipops to an axis when baseline coincides with this axis and sticks are perpendicular to it
geom.slope == 0.0 && geom.intercept == 0.0 && geom.direction != LollipopGeom.Direction.ALONG_AXIS
} else {
aestheticsDefaults.rangeIncludesZero(aes)
}
Expand Down