Skip to content

Commit

Permalink
style: auto-formatting [CI]
Browse files Browse the repository at this point in the history
  • Loading branch information
GitHub Actions Bot committed Jan 25, 2025
1 parent 71cf1df commit 3734844
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/stack.ts
Original file line number Diff line number Diff line change
Expand Up @@ -184,10 +184,10 @@ export function stack(m: Mark | MarkDef, encoding: Encoding<string>): StackPrope

// TODO: Figure out why groupBy is only sometimes necessary?
// Unclear why the test case and the bug report differ in whether a groupBy should be supplied to vega or not
const shouldAddPolarGroupBy = dimensionField === 'dir' || (fieldChannel === dimensionChannel);
const shouldAddPolarGroupBy = dimensionField === 'dir' || fieldChannel === dimensionChannel;

// if (!hasSameDimensionAndStackedField) {
if (!hasSameDimensionAndStackedField && (isPolar ? (shouldAddPolarGroupBy) : true)) {
if (!hasSameDimensionAndStackedField && (isPolar ? shouldAddPolarGroupBy : true)) {
// avoid grouping by the stacked field
// TKTK: find out why
groupbyChannels.push(dimensionChannel);
Expand Down

0 comments on commit 3734844

Please sign in to comment.