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

Tooltip does not reflect ..quantile.. aesthetic change #658

Closed
ASmirnov-HORIS opened this issue Dec 26, 2022 · 1 comment
Closed

Tooltip does not reflect ..quantile.. aesthetic change #658

ASmirnov-HORIS opened this issue Dec 26, 2022 · 1 comment
Assignees
@ASmirnov-HORIS
Milestone
2023Q1

Comments

@ASmirnov-HORIS
Copy link
Collaborator

Example:

data = {
    "x": [0, 1, 1, 2],
    "y": [0, 0, 0, 0],
}
ggplot(data) + \
    geom_area_ridges(aes("x", "y", fill="..quantile..")) + \
    scale_fill_gradient(low="red", high="green")

Result:

tooltips_for_area_ridges
@ASmirnov-HORIS ASmirnov-HORIS self-assigned this Dec 26, 2022
@alshan alshan added this to the 2023Q1 milestone Dec 28, 2022
ASmirnov-HORIS added a commit that referenced this issue Jan 20, 2023
@ASmirnov-HORIS
Fix issue #658.
93aca5f
alshan pushed a commit that referenced this issue Feb 9, 2023
@ASmirnov-HORIS
Quantile Parameters (#689)
787c4b5 
* Use quantile stat var in the geom_violin().

* Fix test for the YDensityStat.

* Fix quantile line for the last quantile for the geom_violin().

* Fix tooltips color when quantiles using for the geom_violin().

* Fix docstrings for the geom_violin() function.

* Tiny fix in the docstrings. Update future_changes.md file.

* Add quantiles to the geom_area() and geom_density().

* Hide quantile aesthetic for the all geometries that are uses it.

* Update docstrings for the geom_density() function.

* Tiny refactor fixes.

* Update future_changes.md.

* Rename param 'quantiles' back to 'draw_quantiles' for the geom_violin().

* Refactor using of the QuantilesHelper.

* Use grouping to remove extra lines for the area geometry.

* Fix issue #658.

* Tiny fix in the future_changes.md file.

* Tiny fixes after merging with the master branch.

* In the geom_violin() parameter 'draw_quantiles' renamed to 'quantiles'.

* Add example to AreaBatik.

* Use expandByGroupEnds() in DensityStat. Add StackingType and use LN type for the AreaGeom.

* Small fixes in code style in geometries.

* Remove extra parameters from the AreaGeom.

* Refactor DensityStatUtil and QuantilesHelper (mostly about visibility).

* Upgrade spec descriptions in demo.

* Small fixes in code.

* Refactor DensityStatUtil.expandByGroupEnds and add some tests for it.

* Add tests for the QuantilesHelper and refactor it.

* Remove extra borders simplification from AreaGeom and ViolinGeom.

* Fix throwing messages in the BogusContext.

* Refactor QuantilesHelper.

* Refactor DensityStatUtil.calculateStatQuantile().

* Tiny fix in StatProto.

* Refactor DensityStatUtil.

* Refactor StackPos.
@ASmirnov-HORIS
Copy link
Collaborator Author

Closed with PR #689.

ASmirnov-HORIS added a commit that referenced this issue Feb 15, 2023
@ASmirnov-HORIS
Mention issue #658 in the future_changes.md file.
9975234
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ASmirnov-HORIS ASmirnov-HORIS

Labels
None yet
Projects
None yet
Milestone
2023Q1
Development

No branches or pull requests
2 participants
@alshan @ASmirnov-HORIS