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.

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

Release 14.0.0 #568

Merged
merged 18 commits into from
Oct 16, 2024
Merged

Release 14.0.0 #568

merged 18 commits into from
Oct 16, 2024

Conversation

github-actions[bot]
Copy link

What's changed

  • feat(vis-type: scatter): improve rendering of scatter plot (#530)
  • feat(vis-type: bar): switch to ECharts bar chart (#497)
  • Style: Add new palettes for categorical colors (#567)
  • Style: remove gray from categorical color palette (#566)
  • fix: export box plot interfaces (#562)
  • fix(vis-type: sankey): Handle undefined/missing values (#550)
  • feat(vis-type: scatter): add drag state to useInteraction hook (#560)
  • feat: Adjusted useCanvas to work with WebGL contextIds and typed it correctly (#549)
  • fix: add mantine carousel package to dependencies (#540)
  • feat: add mouse icon (#539)

dvmoritzschoefl and others added 18 commits August 22, 2024 11:34
…orrectly (#549)

* adjusted useCanvas to work with webgl contextIds and fixed typing

* added export to vishooks
remove gray from categorical color palette
* add 10 and 20 color palettes for categorical colors

* Update src/utils/colors.ts

Co-authored-by: Moritz Heckmann <[email protected]>

* Update colors.ts

---------

Co-authored-by: Michael Pühringer <[email protected]>
Co-authored-by: Moritz Heckmann <[email protected]>
* feat: bar chart with echarts (wip)

* feat: support all bar chart config options

* feat: compute data table for each facet

* perf: compose bar data table once and filter them for each facet

* feat: make focus facet selector working

* style: calculate chart height

* feat: scrollable facet views

* feat: select items with click on a bar

* feat: add tooltip on hover

* feat: assign color scale

* feat: binned color scale for numerical group columns

* fix: selection of items on bar click

* feat: highlight incoming selection

* fix: bar height for non-grouped bars

* style: truncate axis labels after 50 chars

* fix: add scrollbar

* fix: if agg column or cat column is undefined

* fix: account for null values in the config

- make group value default
- harmonize bar width/height

* !feat: don't exclude already selected columns in sidebar

* fix: scatter vis for selected points

(cherry picked from commit 8e4a0ea)

* fix: check width and height of bar chart container

Fixes datavisyn/communication_xaira_ordino#26

The ReactECharts component will check the resize of the parent element and update accordingly

* fix: bar chart facets horizontal scrollbar (#480)

* fix: make resizing work for facets

* feat: add a loading overlay when the viewport resizes

* fix: scroll flickering

* fix: remove pos relative

* chore: remove unnecessary overflow hidden

* chore: address PR review

* fix: storybook error

* feat: implement bar plot sorting (#482)

* feat: add sorting controls
wip: add sorting for y axis
wip: add property in config to preload sorting
wip: use in reprovisyn

* feat: improvements in sorting

* feat: handle categorical column change

* feat: implement initial sorting

* feat: move sort into the header

* fix: sort for normalized mode

* chore: sort facets by name

* fix: add null check for sortState

* fix: add null checks for sort button

* fix: fallback value

* fix: remove circular dependency

* fix: address PR feedback

* chore: use color for unknown values

* fix: storybook error

* fix: title of bar plots overlaps with legend when in 5050 view with volcano (#486)

* feat: add sorting controls
wip: add sorting for y axis
wip: add property in config to preload sorting
wip: use in reprovisyn

* feat: improvements in sorting

* feat: handle categorical column change

* feat: implement initial sorting

* feat: move sort into the header

* fix: sort for normalized mode

* chore: sort facets by name

* fix: add null check for sortState

* fix: add null checks for sort button

* fix: fallback value

* fix: remove circular dependency

* fix: address PR feedback

* chore: use color for unknown values

* fix: storybook error

* fix: overlapping legends in bar chart

* chore: update SingleEChartsBarChart.tsx

* fix: aggregations in bar chart

* fix(wip): select bars in sorted chart
wip: the current config in main app has a bug which needs to be resolved

* feat: add shift + select to add to the selection

* fix: temporarily disable bar chart selection

* fix: wip show categories for all the facets

* chore: update config

* fix: add padding top and bottom to bar chart height

* chore: rename variables

* fix: update bars on sorting
wip: currently only ascending sort works for horizontal bars
wip: grouping with the same column does not work
wip: ⚠️ major performance issue

* fix: display same grouped bars in sorted order correctly
wip: colors do not match
wip: selection in sorting does not work as expected

* fix: sorting using copilot powered development

* chore: cleanup after feature

* feat(wip): add condition to sort unknown at the end of the list

wip: the behavior is inconsistent for vertical and horizontal orientation

* chore: minor improvements

* fix: add react-window virtualizer

* docs: add documentation for matrixSort method

* fix: handle NaN values in matrix aggregation

* feat: improve shift + click to select usability

* fix: remove control key event listener

* chore: remove internal sorting state

* chore: remove redundant code

* Fixed virtualization remount by having a stable renderer and using the itemData property of react-window

* fix: adjust max label length

* fix: bar label overlap + add optimization for large data

* fix: rotate x-axis labels by 45 degrees

* fix: emphasize and blur options

* fix: make barWidth responsive when there are a lot of bars

* fix: horizontal scrollbar issue

* refactor: optimize sorting for performance

* Fixed rerenders caused by new settings object, incorrect height of facets, added hook with setRef function to use instead of react echarts component

* added useChart to exported hooks in visyn_core

* linted hook

* removed option and converted it to a usememo

* Combined state of vis to one state object

* cleaned up

* chore: cleanup + minor fixes

* fix: build error

* fix: build errors

* chore: rename variables

* fix: infinite loop

* feat: add axis label

* fix: min and max aggregations

* fix: lint error

* chore: cleanup

* chore: move function to proper utils file

* chore: cleanup

* fix: battle with strict mode for affected files

* chore: move calculate chart height function to bar chart

* feat: use mantine scrollarea for variable size list

* feat: use mantine scrollarea for variable size list

* chore: continuing battle with strict mode

* fix: empty bar plot on facets

* fix: color and selection

* fix: use higher opacity for unselected

* fix: set opacity back to 0.5

* fix: selection broke

* fix: selection again

* fix: add temp xAxis domain

* fix: selection for same group and facet combination

* fix: modify domain for aggregation axis

* fix: add higher opacity to to unselected  bar label items in grouped mode

* fix: reverse condition

* fix: adapt opacity and colors

* fix: reaggregate bar chart data + add global min and max
wip: selection does not work

* fix: selection

* feat: sort categories

* fix: tooltip clipping

* fix: follow up fix for selection

* feat: add useFullHeight flag to use full height if available

* fix: labels when direction is changed

* fix: aggregation type sensitive global min and max

* feat: implement sorting along both axes

* feat: add DOM sensitive label truncate

* fix: recalculate height

* fix: single value in facet

* fix: minor improvements

* style: make chart responsive to the container

* feat: add responsive labels for y axis for horizontal bar chart

* chore: cleanup unnecessary ref

* fix: click same bar to deselect

* fix: labels width be min of max label width or container / 3

* refactor: add selection information in the header

* fix: categoriesList null check

* fix: invalidate item size cache on config change

* fix: invalidate item size cache on datatable change as well

* feat: sync grid left across facets

* feat(wip): show axis label tooltip
wip: position the tooltip correctly
wip: add a flag to enable / disable the tooltip

* chore: sync colors

* fix: tooltip position + refactoring

* fix: storybook

* chore: select bars when label clicked

* fix: tooltip position for horizontal as well as vertical bars

* fix: strict mode issues in useChart.ts

* fix: add min-width to the bars and make the chart scrollable

* style: make title in sync with other charts

* style: add descriptive title for facets

* style: change axis title for normalized

* style: fix cursor pointer for non interactive elements

* style: round to 4 digits in legend  for numerical group

* fix: add custom binning

* fix: always show tooltip for vertical bars

* fix: axis label distance

* chore: restructure bar plot files + remove legacy bar plot components

* feat: add axis label click for sort

* fix: color scale for faceted plots

* chore: expose isBarConfig function

* chore: restructure files again

* fix: do not compute legend when no grouping is available

* fix: min max calculation

* chore: deprecated bar chart sort buttons

* chore: make useFullHeight as default

* fix: build error

* chore: add comments

* fix: label of aggregation axis

* fix: reduce unnecessary recalculation of aggregatedData

* fix: numerical color scale

* chore: remove optional check

* chore: improve tooltip

* style: improve tooltip

* style: sync tooltip appearance

* style: make axis well spaced

* fix: make loader appear only once

* chore: cleanup

* fix: scrollbar overlap on legend

* fix: tooltip rounding for numerical scales

* chore: update label helper

* Adjusted imports for echarts

* feat: implement responsive width flag

* chore: update title as per the feedback

* chore: minor improvements

* fix: hide tooltip labels in stacked mode and show only when hovered

* fix: display facets in the tooltip first

* fix: htmlize tooltip

* fix: bug when same facet and category selected for numerical group

* fix: color scale for numerical grouping and faceting

* fix: color of legend and tooltip

* fix: dimensions

* chore: remove bar chart sort button from the header

* chore: add placeholder tests

* fix: improve median logic

* fix: bar chart color scale for facet and group combinations

* chore: display labels for grouped bars faceted by same column

* fix: bar chart story

* fix: tooltip label for group having only one bar

* fix: selection when only one bar is present in grouped mode

* chore: remove group.ts

* chore: add group.ts

* chore: improve diversity of test data

* fix: group legend and tooltip for single numerical value grouping

* fix: iris story

* fix: generalize the fix

* fix: generalize the fix 2

* chore: optimizations on previous fix

* fix: limit min and max to 0

* refactor: updates to aggregation types + reduce complexity

* fix: global domain calculation of min max and median

* fix: global domain calculation of average

* fix: use the correct median function import

* fix: label visibility

* fix: median logic

* chore: reduce number of looping for median calculation

* fix: tooltip content for same group and facet column

* test: add tests for bar dimensions

* test: add tests for bin lookup

* chore: cleanup in singleechartbarchart

* test: add tests for generated-aggregated-data-lookup

* docs: add comment about `merge` flag in BaseVisConfig

* chore: cleanup main app

* docs: update comment

* fix: median aggregation

* fix: storybook

* feat(vis-type: bar): download chart is clipped for long charts (#561)

Co-authored-by: Holger Stitz <[email protected]>

* chore: extend bar tests

* chore: extend bar tests

* chore: add comment

* chore: extend bar tests

* chore: update storybook

* feat: add tooltip for disabled segmented control

* fix: color scale order for numerical groupings

* fix: main app for the PW tests to pass

---------

Co-authored-by: Holger Stitz <[email protected]>
Co-authored-by: oltionchampari <[email protected]>
Co-authored-by: Moritz Heckmann <[email protected]>
Co-authored-by: Champari Oltion <[email protected]>
Co-authored-by: Michael Puehringer <[email protected]>
Co-authored-by: Christian Bors <[email protected]>
Co-authored-by: oltionchampari <[email protected]>
Co-authored-by: Holger Stitz <[email protected]>
Co-authored-by: Michael Pühringer <[email protected]>
@github-actions github-actions bot requested a review from a team as a code owner October 16, 2024 19:46
@thinkh thinkh added the release: major PR merge results in a new major version label Oct 16, 2024
@thinkh thinkh self-assigned this Oct 16, 2024
@thinkh
Copy link
Member

thinkh commented Oct 16, 2024

The PW are failing for the download of hexbin and boxPlot.

2 failed [chromium] › boxPlot/01_general.spec.ts:4:5 › download ───────────────────────────────────────── [chromium] › hexbin/01_general.spec.ts:25:5 › download ───────────────────────────────────────── 92 passed (6.7m)

I tested this manually and they work. We should fix the PW tests though.

@thinkh thinkh merged commit b4fb8e9 into main Oct 16, 2024
3 of 4 checks passed
@thinkh thinkh deleted the release-14.0.0 branch October 16, 2024 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release: major PR merge results in a new major version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants