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

访问会越界 #149

Closed
wants to merge 6 commits into from
Closed

访问会越界 #149

wants to merge 6 commits into from

Conversation

LuckyQinn
Copy link
Contributor

访问sums_ptr会越界,应该改为channels

@nihui
Copy link
Member

nihui commented Sep 27, 2017

pull request 里头除了 reduction.cpp 以外的文件不要修改

@nihui
Copy link
Member

nihui commented Sep 27, 2017

还有,请把改动做在一个 commit 里头...
谢谢

@LuckyQinn
Copy link
Contributor Author

是的,不好意思,第一次用github,很多操作还不懂,误提交了很多上去
我还想着怎么可以撤销pull request呢

@nihui
Copy link
Member

nihui commented Sep 27, 2017

要不我先关掉这个 pull request,重新来一次?

@LuckyQinn
Copy link
Contributor Author

可以啊,你直接修改就好了,我就不pull request 了,还没研究透怎么用

@nihui
Copy link
Member

nihui commented Sep 27, 2017

合并好了

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

Successfully merging this pull request may close these issues.

2 participants