layout icon indicating copy to clipboard operation
layout copied to clipboard

工程改进建议

Open hustcc opened this issue 2 years ago • 3 comments

可以直接基于这个模板:https://github.com/antvis/template

  • [ ] npm 包含上 src https://github.com/ant-design/ant-design-charts/issues/786
  • [ ] 去除 tslint,已经不维护了,改成 eslint,参考模板
  • [ ] 一些依赖好像并没有直接使用,比如 g-webgpu ,是否去除 https://github.com/antvis/layout/issues/43
  • [ ] 如果定位是和 scale 一样,底层依赖,且可以社区使用,建议 readme 和文档参考 scale 补充详细
  • [ ] 这个代码的设计上是否可以去掉 register 和 unregister 的概念,全部保持是纯函数的代码片段,并设置 sideEffects: false

hustcc avatar May 07 '22 01:05 hustcc

  • register 和 unregister 是原来G6原有的概念,所以应该可以拆除
  • 关于 g-webgpu 的话,可以考虑维护 layout 和 layout-gpu两个包,按需引用
  • 因为是个break change,不如直接来维护成 1.0.0

mxz96102 avatar May 07 '22 03:05 mxz96102

可以的,layout 就组成各种图布局算法的函数库。另外,类似的 graphlib 在工程上也注意些类似的问题~~

hustcc avatar May 07 '22 03:05 hustcc

这个代码的设计上是否可以去掉 register 和 unregister 的概念,全部保持是纯函数的代码片段,并设置 sideEffects: false

这个包体积还是挺大的,强烈支持最后一点,方便使用者可以 tree shaking 掉没有使用的特性。

jtsang4 avatar May 10 '22 12:05 jtsang4