Zhongxian Pan

Results 15 comments of Zhongxian Pan

Hi @sundb . I tried the `__builtin_ctzll ` and updated my test file. It has almost same time cost with De Bruijn method. And they are both faster than the...

> @panzhongxian please also update the top comment which doesn't mention `__builtin_ctzll`. @sundb Updated.

> outdated? > > ``` > * This may sound like inefficient, but actually in the average case > * there are high probabilities to find a 1 after a...

> @panzhongxian unrolled loops also means reducing the call of jump. did you do some benchmark to verify it? @sundb There seems no `PFADD` and other hll commands in src/benchmark.c....

> @panzhongxian we just put benchmarks that can't be used directly with benchmarks into benchmark.c. in this case we can use `./src/redis-benchmark -n 10000000 PFADD key 1000` @sundb OK. To...

Hi, @sundb. This PR indeed bring no performance improvement, but it can simplify the code and improve readability without compromising performance.

@sundb I understand what you mean. So I write a comaprison test by extracting the `hllDenseRegHisto()` [here](https://gist.github.com/panzhongxian/c7dd59bdafb42435c9701df6c90813fc). I built the simplified and not simplified hllDenseRegHisto() function by whether defining `SIMPLIFIED`....

> @panzhongxian it's merged, thanks. @sundb Thank you for your help.

@CapitaineNemo 我看你应是说的微信客服账号对吧?我看API好像有些类似,但是没有实际使用。 你可以试试,然后反馈一下有什么问题哈。 看上去机器人接收消息的协议和其他接收消息的协议是不同的,我们可以做一个兼容。让一个框架,来匹配两个种协议,用户可以在创建基础的Server的时候自行选择,而无需用户了解,只需要进行消息处理即可。 机器人协议:https://developer.work.weixin.qq.com/document/path/95280 客服接收消息协议:https://developer.work.weixin.qq.com/document/path/94670 公众号接收消息协议:https://developers.weixin.qq.com/doc/offiaccount/Getting_Started/Getting_Started_Guide.html

@CapitaineNemo https://developer.work.weixin.qq.com/document/path/90238 我刚才试了一下,可以创建“应用”,也可以实现接收消息和回复消息的功能。但是无法被邀请到群中 但是他的协议跟机器人的不同,我打算另外开一个仓库,来适配企业微信应用的回复消息。 另外的一点不同是,应用这里需要填上企业的ID