Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

提一些小建议 #2

Open
Elric-pp opened this issue Sep 27, 2017 · 6 comments
Open

提一些小建议 #2

Elric-pp opened this issue Sep 27, 2017 · 6 comments

Comments

@Elric-pp
Copy link

Elric-pp commented Sep 27, 2017

  1. 建议添加 eslint , editorconfig 等代码风格 config , 比较方便其他人提交代码
  2. 可以开一个 slack group 吗?感觉在 qq 群不能贴代码,slack 或者 gitter 比较适合
  3. 看到很多变量放到了 global 变量里,其实是不是应该直接新建一个单例暴露出来会比较好
  4. 感觉如果把接收的逻辑分离出来会比较好,一个是可以避免策略逻辑复杂阻塞 tick 行情的推送,一个是可以支持回测。
@zhangshuiyong
Copy link
Owner

感谢建议,以下简单回复
1.@Elric-pp editorconfig风格的config?你是说修改一下userconfig配置文件的风格吗?其他人提交代码是?
2.因为现在好像量化圈子还比较小,暂时没有开其他讨论组,个人精力有限管理
3.global变量只是用了几个,就是主程序,主引擎,策略引擎,事件分发器
4.接收的逻辑分离?具体是指?由于node.js是一个事件队列,策略耗时长是可能阻塞着tick排队在事件队列中,如果策略逻辑耗时长建议用tagg2等多线程库。接收tick逻辑现在是基于事件的会直接由strategyEngine推送到策略

@Elric-pp
Copy link
Author

Elric-pp commented Sep 28, 2017

  1. 就是一些代码风格的配置,比如说缩进,tab 或者 空格,还有 js 的风格检测,这样子像我贡献代码的时候可以保证整个项目的代码风格保持一致
  2. 其实我可以开一个:D,不过这个不是特别重要啦,方便项目管理而已。
  3. 是的,但是模块化的方案会比较可读些
  4. 就是 CTP client 直接推送数据然后存到 redis 里,和策略逻辑分离开来,不然如果要怎么实现历史数据回测呢?

@zhangshuiyong
Copy link
Owner

刚放完假,懒洋洋的。关于第四点建议,redis数据库的数据是存在内存的,交易系统用它是非常好的。行情数据太多了,要用其他数据库做回测。回测想用ssdb数据库+cluster多进程+网页显示回测结果图形,各方面差不多准备好了,差点时间把它写了。@Elric-pp

@yuu2lee4
Copy link

yuu2lee4 commented Oct 10, 2017

用eslint做代码风格的规范统一还是有必要的,另外建议升级到node 8,这样基本支持所有的es6特性了

@zhangshuiyong
Copy link
Owner

zhangshuiyong commented Oct 10, 2017

eslint风格之后会用工具统一,node 8会升级,因为现在node.js官网node 8还不是LTS版本,10月31日才是LTS版本,所以会在10月31日之后升级到node 8 @yuu2lee4

@gucong3000
Copy link

建议添加 eslint , editorconfig 等代码风格 config , 比较方便其他人提交

我对这个很感兴趣,如果作者不介意,我愿意完成这项任务。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants