posit-dev / posit-dev/ggsql

We should be able to set groups globally

Open
#102 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

syntax
Dominant language
Rust
Stars
554
Forks
30
Avg merge
17h 27m
Merged PRs (30d)
8

Description

Because PARTITION BY is analogous the group aesthetic, we should be able to set it at the top-level to be inherited by individual layers.
Currently, we're doomed to repeat it at every layer.

I want to be able to do (something like) this:

VISUALISE x, y FROM df PARTITION BY id
  DRAW polygon
  DRAW path

Instead of having to do this:

VISUALISE x, y FROM df 
  DRAW polygon PARTITION BY id
  DRAW path PARTITION BY id

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No files or tests are named. Start by tracing how PARTITION BY is parsed and applied to individual DRAW layers, then determine where a top-level PARTITION BY could be inherited. Done means the VISUALISE example applies the partition to both polygon and path layers without repeating it.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust, sql
Domain
compilers
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 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.