JetBrains / JetBrains/lets-plot

Allow geom_bar() as a marginal layer

Open
#972 1 comment 0 reactions 0 assignees View on GitHub
failure
Dominant language
Kotlin
Stars
1.8k
Forks
60
PR merge metrics
No merged PRs in 30d

Description

Example:

```python
data = {
'x': ["a", "a", "b", "b"],
'y': ["f", "g", "g", "g"],
}
ggplot(data, aes('x', 'y')) + \
geom_count() + \
ggmarginal('r', layer=geom_bar())
```

Output:

> Internal error: IllegalStateException : Not a collections of Double(s)

Can be fixed by adding `orientation='y'` into the `geom_bar()`.

It seems that the ability to add discrete geoms as marginal layers was overlooked, but with the introduction of `geom_count()` it makes sense.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by running the provided Python example with geom_count() and ggmarginal('r', layer=geom_bar()). Trace the marginal-layer handling for geom_bar and its orientation inference. Done means the example works without explicitly setting orientation='y' and still produces the expected marginal bar layer.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data-visualization
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.