ChenYang He
ChenYang He
@jat001 @dylan-fan @hainingzhang @jarviszeng-zjc 如果同意认可这个特性，我会进行开发并提交pr
MikkiYang give it to me
> 另外，scdbclient目前应该也不支持explain？有考虑一起支持吗，感觉难度应该还好。 暂时没有时间了，因为要去搞明年年初嘉年华涉及的logchain服务联合开发、、、
@tongke6 童柯老师有空帮忙review下哦，我这边搭建环境的机器是共用的，可能已经快要拆除了
> @MiKKiYang 感谢您的贡献，功能实现得很好！ > > 关于代码位置，我有一个建议：为了保持 SCDB 的代码结构一次性，非 DQL 的处理逻辑，最好统一在 executor 里处理，位于 `pkg/scdb/executor` 目录下，里面有其他的 statement 处理，可供参考。 @tongke6 感谢tongke老师的建议，我看了下您这个建议修改可能需要按照executor目录下的其余算子实现重构当前explain特性代码、之后再制作镜像、集成测试、回归测试等；由于我当下有年底嘉年华相关的secretflow/logchain服务开发任务，可能相对会比较紧急，您看是否可以把这次您建议的重构延续到后续任务再做？当前实现我已测试，功能是正常的，可以确保特性能力完备。
> > > @MiKKiYang 感谢您的贡献，功能实现得很好！ > > > 关于代码位置，我有一个建议：为了保持 SCDB 的代码结构一次性，非 DQL 的处理逻辑，最好统一在 executor 里处理，位于 `pkg/scdb/executor` 目录下，里面有其他的 statement 处理，可供参考。 > > > > > > @tongke6 感谢tongke老师的建议，我看了下您这个建议修改可能需要按照executor目录下的其余算子实现重构当前explain特性代码、之后再制作镜像、集成测试、回归测试等；由于我当下有年底嘉年华相关的secretflow/logchain服务开发任务，可能相对会比较紧急，您看是否可以把这次您建议的重构延续到后续任务再做？当前实现我已测试，功能是正常的，可以确保特性能力完备。 > > 可以先合入，后续有空了麻烦再接着把它做完善哈~...
> 请问有实际测试性能提升的幅度吗？如果提升不明显，不建议修改。原有代码更简洁 @wenyuanhust 具体性能提升幅度未测试，因为secretPad本身只是属于隐语全栈技术（pad -> kuscia -> flow -> spu）的产品层，此处的性能影响较之全链路近乎于没有； 该修改主要关注点不在于性能，也跟简洁无关，而在于代码统一，当前项目多处代码分别出现过于随意的数组预分配处理方式，有的按照0，有的按照.size()取大小，建议统一为一种。
38832234 Give it to me
@Candicepan @Yeekin-GYJ @deadlywing hello，我这边目前实现完成了，我看提交说明里提到：“提交说明： 关联该 isuue 并提交代码至 https://github.com/secretflow/spu/tree/main/sml （具体目录位置请与reviewer讨论）” 但是我的实现依赖了如下的库： from typing import Any, Dict, List import jax.numpy as jnp import numpy as np import pandas as pd from secretflow.data...
@deadlywing 我好像理解有点问题，实现的方式没有以算法方式而是以工程方式去做了