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

Improvements in performance #757

Closed
netil opened this issue Jan 31, 2019 · 3 comments
Closed

Improvements in performance #757

netil opened this issue Jan 31, 2019 · 3 comments
Assignees
Milestone

Comments

@netil
Copy link
Member

netil commented Jan 31, 2019

Description

Find the way to increase performance in:

@netil netil added this to the 2.0-next milestone Jan 31, 2019
@netil netil self-assigned this Jan 31, 2019
@Cyphur
Copy link

Cyphur commented Mar 13, 2019

@netil - I am wondering if you plan on investigating these performance issues in the near future, or is this something farther off in priority?

@netil
Copy link
Member Author

netil commented Mar 14, 2019

@Cyphur, performance improvements are always a topic of concern and it can't be done by once.
If you've been following this repos issues, that efforts are shown one by one(ex. #759).

I'm planning v2.0 by the end of this year, so hopefully these improvements were made for that release.

Thanks anyway for the interests :)

netil added a commit that referenced this issue Jun 8, 2020
BREAKING CHANGE: v2 updates

Ref #758, #757, #756, #36
netil pushed a commit that referenced this issue Jun 8, 2020
# [2.0.0-next.1](1.12.0-next.5...2.0.0-next.1) (2020-06-08)

### Bug Fixes

* **all:** Fix test cases ([2e1ad79](2e1ad79))
* **point:** Correct focus.only to work in mobile env ([67eea16](67eea16))
* **point:** Correct point.focus.only ([1686594](1686594))

### Code Refactoring

* **all:** v2 updates ([e23998f](e23998f)), closes [#758](#758) [#757](#757) [#756](#756) [#36](#36)
* **module:** implement ESM index ([85caf71](85caf71))

### Features

* **point:** Intent to ship point.focus.only ([bb70347](bb70347))

### BREAKING CHANGES

* **all:** v2 updates
* **module:** new index for ESM build

- split Axis releated size from size.ts --> size.axis.ts
- split common main option from Options.ts --> ./common/main.ts
- Instead export Axis class, add .getAxisInstance() to make beneficial
  from tree-shaking
@netil
Copy link
Member Author

netil commented Jul 9, 2020

Is a never ending task where should aim continuously to improve the performance, but it has reached some progress from the work of v2. Enough to close the issue for now.

@netil netil closed this as completed Jul 9, 2020
netil pushed a commit that referenced this issue Jul 16, 2020
# [2.0.0](1.12.11...2.0.0) (2020-07-16)

### Bug Fixes

* **all:** Fix test cases ([2e1ad79](2e1ad79))
* **arc:** fix applying data.labels.colors ([#1448](#1448)) ([c128fad](c128fad)), closes [#1440](#1440)
* **axis:** fix incorrect clip node handling ([a8c6f96](a8c6f96)), closes [#1449](#1449)
* **axis:** make axis clip-path to fit real axis size ([7419f44](7419f44)), closes [#1449](#1449)
* **bar:** fix bar width scale on zoom ([59073bd](59073bd)), closes [#1476](#1476)
* **data:** fix for data.labels=false ([b7a0972](b7a0972)), closes [#1444](#1444)
* **data.selection:** fix selection.isselectable value check ([9d41a04](9d41a04))
* **gauge:** fixed wrong bottom padding calculation ([0542586](0542586)), closes [#1441](#1441) [#1471](#1471)
* **legend:** Don't bind event when interaction is false ([4546c00](4546c00))
* **point:** Correct focus.only to work in mobile env ([67eea16](67eea16))
* **point:** Correct point.focus.only ([1686594](1686594))
* **point:** update point generation ([da63e39](da63e39))
* **subchar:** correct subchart rendering ([44ed216](44ed216)), closes [#1458](#1458)

### Code Refactoring

* **all:** v2 updates ([e23998f](e23998f)), closes [#758](#758) [#757](#757) [#756](#756) [#36](#36)
* **module:** implement ESM index ([85caf71](85caf71))

### Features

* **axis:** Intent to ship log scale  ([6fdf3e4](6fdf3e4)), closes [#1351](#1351)
* **bar:** Intent to ship bar.label.threshold ([72a7b7f](72a7b7f)), closes [#1427](#1427)
* **gauge:** Intent to ship gauge.label.threshold ([#1443](#1443)) ([9a0807e](9a0807e)), closes [#1439](#1439)
* **interaction:** split selection, subchart & zoom ([ba1e4f2](ba1e4f2))
* **point:** Intent to ship point.focus.only ([bb70347](bb70347))

### BREAKING CHANGES

* **all:** v2 updates
* **module:** new index for ESM build

- split Axis releated size from size.ts --> size.axis.ts
- split common main option from Options.ts --> ./common/main.ts
- Instead export Axis class, add .getAxisInstance() to make beneficial
  from tree-shaking
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants