Revert "chore: version (beta)" - #2822
Conversation
This reverts commit c8940d9.
|
Summary of ChangesHello @lzxue, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request performs a complete revert of a previous versioning chore that had introduced a Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request reverts a previous commit that bumped the version to a beta release. The changes correctly revert the version numbers in all package.json files, remove the corresponding beta entries from the CHANGELOG.md files, and reset the changeset configuration. The changes are consistent with the goal of reverting the beta version bump.
* chore: 添加skills 定义 * feat: add workflow to sync skills to chart-visualization-skills - Auto create PR when .skills directory changes - File validation (type, size, content check) - Only run on official antvis/L7 repository - Support manual trigger with force option * chore: 目录更新 * chore: 修复一下文档引用错误 * chore: 优化引用 * docs: update skills documentation and add new reference files * chore: 更新readme * chore: remove baidu map demos and update gitignore * chore: 移除dev demo * chore: 移除 skill async workflow * chore: 更新skill * chore: 修复cdn错误 * fix: 修复蜂窝和网格热力图闭合问题 1. 修复蜂窝热力图 (hexagon): - 调整六边形方向为 pointy-top,与 d3-hexbin 一致 - 翻转 y 坐标方向以匹配屏幕坐标系 2. 修复网格热力图 (grid): - 改用屏幕像素坐标划分网格,与 hexagon 一致 - 修正 xOffset/yOffset 计算:正方形顶点范围是 [-√2/2, √2/2] 3. 修复 count 聚合方法: - 在 statMap 中添加 count 方法 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * perf: 优化瓦片图层性能 主要优化: - TileLayerService: 使用 Map 存储瓦片实例,查找复杂度从 O(n) 降为 O(1) - TileLayerService: 实现分帧销毁机制,避免大量瓦片销毁阻塞渲染 - TileLayerService: 添加渲染列表缓存和脏标记,减少每帧重建 - BaseLayer: 合并多个 filter 操作,减少中间数组创建 - BaseLayer: 优化视图状态比较,使用数值比较替代字符串比较 - TilesetManager: 缓存 tiles getter,避免每次访问都排序 - TilesetManager: 使用 Set 存储当前瓦片 key,加速查找 - TilesetManager: 合并 updateTileVisible 遍历,减少迭代次数 - SourceTile: 缓存 bounds 和 bboxPolygon,避免重复计算 - TilePickService: 优化遍历,跳过未加载的瓦片 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * perf: 优化瓦片数据解析性能 主要优化: - math.ts: 表达式预编译为函数,避免循环中 JSON 序列化/反序列化 - 对于 256x256 瓦片,从 65536 次序列化操作优化为 1 次编译 - 预分配 Float32Array 结果数组,避免动态扩容 - operationSchema.ts: 使用快速选择算法计算百分位 - 复杂度从 O(n log n) 降到 O(n) - 预分配 Uint8Array 数组存储 RGB 数据 - getRasterData.ts: 多波段数据并行请求 - 使用 Promise.all 并行请求,减少网络等待时间 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix: 修复 device 模式下 enableRelativeCoordinates 不生效的问题 主要修复: 1. 修复 UBO std140 布局问题 - vec3 只占 3 floats,之前的布局导致 u_RelativeOrigin 位置错误 2. 修复 extrude/water/ocean 图层在相对坐标模式下 UV 计算错误 3. 修复 Pipeline 缓存 key 生成不完整导致的状态不一致问题 性能优化: 1. 添加颜色解析缓存 (rgb2arr),避免重复解析相同颜色 2. 优化数据映射插件的过滤流程,减少遍历次数 3. 添加质心计算缓存 其他修复: 1. 移除地图适配器中的硬编码 API Keys 2. 修复 Scene.emit 调用错误 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor: 优化依赖和修复拼写错误 - fix: 修复 citybuliding 目录名拼写错误 (citybuliding -> citybuilding) - refactor: 用自定义实现替代 lodash 依赖,优化包大小 - refactor: 将 SimpleMapCoord 移到 maps 包本地 utils - fix: 添加 getDefault 方法返回类型 - fix: 修复 merge 函数的类型参数 - fix: 添加 initCfg 方法 option 参数的条件检查 - chore: 更新 build 脚本,添加 clean-dist 步骤 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor: 用自定义实现替代 d3-dsv 依赖 - 新增 packages/source/src/utils/csv.ts 实现 csvParse 函数 - 支持 CSV 基础解析、引号处理、转义引号、各种换行符 - 移除 d3-dsv 和 @types/d3-dsv 依赖 - 添加完整的单元测试覆盖边界情况 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor: 用自定义实现替代 d3-array 依赖 - 在 lodash-adapter.ts 中添加 extent 函数实现 - 移除 d3-array 和 @types/d3-array 依赖 - 更新 FeatureScalePlugin 使用 lodashUtil.extent - 添加 lodash-adapter 单元测试 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor: 用自定义实现替代 d3-color 依赖 - 新增 packages/utils/src/color-parser.ts 实现颜色解析 - 支持 hex、rgb/rgba、hsl/hsla 和 CSS 颜色名称 - 更新 color.ts 使用自定义 parseColor 函数 - 移除 utils 和 layers 包中的 d3-color 依赖 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor: 用自定义实现替代 d3-interpolate 依赖 - 在 color-parser.ts 中添加 interpolateRgbBasis 函数 - 实现分段线性颜色插值功能 - 更新 FeatureScalePlugin 使用自定义 interpolateRgbBasis - 移除 layers 包中的 d3-interpolate 依赖 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor: 用自定义实现替代 d3-hexbin 依赖 - 新增 packages/source/src/utils/hexbin.ts 实现六边形分箱算法 - 使用 odd-q offset 坐标系统与 d3-hexbin 兼容 - 更新 hexagon.ts 使用自定义 hexbin 函数 - 移除 d3-hexbin 和 @types/d3-hexbin 依赖 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix: 修复 hexbin 六边形方向问题 - 将 odd-q 偏移(奇数列偏移)改为 odd-r 偏移(奇数行偏移) - 完全复制 d3-hexbin 的分箱算法逻辑 - 修复中心坐标计算方式: - x = (pi + (pj & 1) / 2) * dx - y = pj * dy - 修复访问器类型签名以支持 3 个参数 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor: 用自定义实现替代 d3-scale 依赖 - 新增 packages/layers/src/utils/scale.ts 实现 10 种比例尺 - scaleLinear, scalePow, scaleLog: 连续比例尺 - scaleSequential, scaleDiverging: 连续颜色比例尺 - scaleQuantile, scaleQuantize, scaleThreshold: 离散比例尺 - scaleOrdinal: 枚举比例尺 - scaleTime: 时间比例尺 - 更新 FeatureScalePlugin 使用自定义比例尺 - 移除 d3-scale 和 @types/d3-scale 依赖 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix: 修复 scale 比例尺处理无效 domain 的问题 - 添加 sanitizeDomain 函数处理 domain 中的 undefined、null、NaN 值 - 修复当 extent 返回 [undefined, undefined] 时导致 NaN 计算的问题 - 处理 domain 只有一个值或为空的情况 - 处理 domain[0] === domain[1] 时除以零的问题 - 所有连续比例尺现在都对无效输入有容错处理 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix: restore heatmap model and color mapping * refactor: 移除未使用的第三方依赖,减少包体积 - 移除 lodash 依赖(已有自定义 lodash-adapter 实现) - 移除 layers 包中未使用的 extrude-polyline、polyline-miter-util、gl-vec2 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * refactor: 用原生 ResizeObserver 替换 element-resize-detector - 使用现代浏览器原生支持的 ResizeObserver API - 移除 element-resize-detector 和 @types/element-resize-detector 依赖 - 减少第三方依赖,提升稳定性 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: 修复 ResizeObserver 回调类型兼容性问题 - 为 MediaQueryList 事件创建独立的 handleDPRChange 处理函数 - 解决 ResizeObserverEntry[] 与 MediaQueryListEvent 类型不兼容的问题 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: 在 ILayer 接口中添加 shapeOption 属性定义 - 解决 heatmap.ts 中 this.layer.shapeOption 类型错误 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * refactor: 移除未使用的依赖 web-worker-helper 和 ajv - 移除 utils 包中未使用的 web-worker-helper - 移除 core 包中未使用的 ajv(相关代码已被注释) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * refactor: 用原生 Pointer Events API 替换 hammerjs - 使用 pointerdown/pointermove/pointerup 替换 Hammer.Pan 实现拖拽 - 使用原生 click/dblclick 事件替换 Hammer.Tap - 使用计时器模拟长按(press)事件 - 移除 hammerjs 和 @types/hammerjs 依赖 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * refactor: 优化事件系统架构 主要改进: - 统一使用 Pointer Events API 处理鼠标和触摸事件,移除冗余监听 - 提取坐标转换逻辑为公共方法,减少重复代码 - 优化单击/双击检测逻辑,区分 click 和 dblclick 事件 - 修复 mousedown 监听器内存泄漏(capture 参数不匹配) - 添加容器边界缓存,优化性能 - 新增 Press 事件类型,语义更清晰 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat: 添加重构验证测试Demo 新增 refactor 测试分类,包含以下验证场景: - interactionTest: 交互测试(拖拽、单击、双击、长按、hover、contextmenu) - resizeObserverTest: ResizeObserver测试(容器大小变化、canvas自适应) - displayTest: 显示测试(点、线、多边形、热力图各图层渲染) - animationTest: 动画测试(波纹动画、流动线动画、数据更新动画) - eventsTest: 事件系统测试(所有InteractionEvent类型统计) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: 修复 resize demo 中不存在 getCanvas 方法的问题 - 使用 querySelector 从容器中获取 canvas 元素 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: 修复 InteractionService 与 l7-draw 的兼容性问题 1. 移除 onClick/onDoubleClick 中的 stopPropagation,允许事件冒泡到底层地图 2. 通过 Hover 事件发送 mousedown/mouseup/click/dblclick,保持向后兼容 3. 添加 vite optimizeDeps.exclude 配置,避免 esbuild 预构建本地包 4. 修复 l7-draw demo 文件中的事件监听和 API 使用问题 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: 添加 l7-draw 依赖并格式化代码 1. 添加 @antv/l7-draw 开发依赖用于测试 2. 升级 vite-plugin-string 到 1.2.3 3. 添加 .playwright-mcp 到 .gitignore 4. 代码格式化 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: 修复 Marker 缩放时不跟随地图移动的问题 独立添加的 Marker(通过 scene.addMarker)之前没有注册相机变化事件, 导致地图缩放/平移时 marker 位置不更新。现已在 MarkerService 中添加 camerachange/viewchange 事件监听来同步更新所有 marker 位置。 同时修复了 addMarkerLayers 方法中错误清空 unAddMarkers 的 bug。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * refactor(component): 优化 packages/component 代码质量 ## 高优先级修复 - fix: 修复 Popper destroy 时未从 conflictPopperList 移除实例的内存泄漏 - fix: 删除 Marker 中未实现的遗留方法 addDragHandler 和 onUp - perf: 为 Popup 高频事件添加 RAF 节流,提升性能 ## 中优先级修复 - perf: 优化 sideEffects 配置,精确声明副作用文件以提升 tree-shaking - refactor: 新增 EventManager 工具类统一管理事件绑定/解绑 - chore: 删除源码中的构建产物 index.css,改用 less 源文件 ## 低优先级修复 - test: 恢复 marker.spec.tsx 中被注释的测试用例 - test: 新增 eventManager.spec.ts 测试文件 - test: 新增 popper.spec.ts 测试文件验证内存泄漏修复 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat(component): layerpopup trigger 参数改为可选,默认 hover 修改 ILayerPopupOption 接口,trigger 参数从必填改为可选。 默认值 'hover' 在 getDefault 方法中已定义。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(component): 修复 layerpopup 默认 trigger 不生效的问题 当用户未指定 trigger 时,getDefault 方法中 isHoverTrigger 判断错误。 现在先获取最终 trigger 值再判断是否为 hover 模式。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(component): 修复 getActualTriggerEvent 返回类型错误 添加明确的返回类型声明,并在 trigger 为 undefined 时使用默认值 'hover'。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: 更新 Claude 配置权限设置 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: 优化lint 错误 * test: 修复单元测试环境问题 - Jest 配置添加 .less 文件支持 - ResizeObserver mock 添加 disconnect 方法 - popup 测试添加 fake timers 处理 RAF - marker 测试添加 overflowHide: false 避免位置问题 - imageLayer/raster-tile 测试添加 afterEach 清理 scene - citybuilding 测试跳过(gl 原生模块内存问题) - source 测试更新 grid 聚合期望值 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: 更新 Claude 配置权限及测试快照 - 添加 yarn test、jest 等测试相关权限配置 - 更新集成测试快照图片 - 新增多个测试场景的快照文件 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * build(l7): umd 构建将 css 内联到 js 中 配置 style-loader 替代 mini-css-extract-plugin,使 umd 产物的 css 直接打包进 l7.js,用户无需单独引入 css 文件。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: 允许跨平台集成测试的像素差异 - 将 maxError 从 0 改为 100,允许 macOS 和 Linux CI 环境之间的渲染差异 - 添加 @rollup/rollup-darwin-arm64 依赖以支持本地测试运行 * fix: 增加像素差异阈值到150以解决heatmap_grid的跨平台差异 * fix: 设置bail为0确保测试失败后继续执行 * fix: 增加页面加载超时时间到60秒,避免CI超时 * ci: 临时禁用集成测试,待跨平台渲染差异问题解决后再启用 * chore: 添加 changeset 记录重构三方依赖变更,更新 gitignore * chore: 添加 beta 预发布 changeset 记录 * chore: 更新 changeset 为 patch 版本 * chore: version (beta) (#2821) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Revert "chore: version (beta) (#2821)" (#2822) This reverts commit c8940d9. * chore: version (beta) (#2823) * fix: 移除 symbol-es6 polyfill 避免 Iterator 协议冲突 symbol-es6@0.1.2 是较旧的 polyfill,会覆盖原生 Symbol.iterator 实现, 导致 earcut 返回的 Uint32Array 无法被 spread 操作符展开。 现代浏览器已原生支持 Symbol 和迭代器协议,不需要此 polyfill。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: 更新 VSCode 配置 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: 更新 demo 示例使用 GaodeMap Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: 更新构建配置和高德地图 Key Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: 修复事件系统与 l7-draw 兼容性问题 - SceneService 增加监听 Hover/Click/DblClick 事件并发射到 scene - IInteractionTarget 添加 pixel 属性兼容 l7-draw 事件格式 - 修复 mousedown 事件发送时机,确保 layer 能正确接收拖拽开始事件 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: 版本变更 * chore: bump version to 2.23.3-beta.1 * chore: bump version to 2.23.3-beta.2 * revert: 回滚事件系统到 hammerjs 实现,重新添加 hammerjs 依赖 * chore: bump version to 2.23.3-beta.3 * fix(citybuilding): 修复 CityBuildingLayer 扫描效果动画和着色器编译问题 - 初始化 BaseLayer.animateStartTime 为 0,避免动画时间返回 NaN - 添加 std140 uniform block padding 字段,修正内存对齐(108 -> 112 字节) - 修复 removeDuplicateUniforms 函数处理 L7 默认值语法(:) - 确保在 WebGL1/regl 渲染器下着色器正确编译 * fix: 修复高德地图主题 URL 和 Three.js demo 中的 GLTFLoader 引入问题 - 更新高德地图主题 URL 使用标准格式 - 修复 Three.js demo 中 GLTFLoader 的引入方式,改为使用 window.GLTFLoader Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: 修复 scene.md 中的合并冲突标记 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: release version 2.24.0 - Update all package versions from 2.23.3-beta.3 to 2.24.0 - Clear changeset files for stable release Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: remove hardcoded Mapbox tokens from demo files and gitignore built weather assets --------- Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Reverts #2821