antvis / layout

Layout algorithms for graphs.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

工程改进建议

hustcc opened this issue · comments

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

  • npm 包含上 src ant-design/ant-design-charts#786
  • 去除 tslint,已经不维护了,改成 eslint,参考模板
  • 一些依赖好像并没有直接使用,比如 g-webgpu ,是否去除 #43
  • 如果定位是和 scale 一样,底层依赖,且可以社区使用,建议 readme 和文档参考 scale 补充详细
  • 这个代码的设计上是否可以去掉 register 和 unregister 的概念,全部保持是纯函数的代码片段,并设置 sideEffects: false
  • register 和 unregister 是原来G6原有的概念,所以应该可以拆除
  • 关于 g-webgpu 的话,可以考虑维护 layout 和 layout-gpu两个包,按需引用
  • 因为是个break change,不如直接来维护成 1.0.0

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

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

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