-
Notifications
You must be signed in to change notification settings - Fork 42
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
I thinkYour code is taking up unnecessary 6GB memory in selftraining.py #20
Comments
hello,Have you verified your conjecture? |
yes,I try my code and get the same result,and it's evident right
…------------------ 原始邮件 ------------------
发件人: "One J"<[email protected]>;
发送时间: 2020年3月28日(星期六) 晚上10:39
收件人: "OasisYang/SSG"<[email protected]>;
抄送: "朱方琪"<[email protected]>; "Author"<[email protected]>;
主题: Re: [OasisYang/SSG] I thinkYour code is taking up unnecessary 6GB memory in selftraining.py (#20)
you don't need cluster_list on line 366,it takes up 6GB,beacuse cluster.components_ is a is a matrix shape of (15xxx,16552),you can see this in site-packages/sklearn/cluster/dbscan_.py.
I think eps_list can have the same effect.
I don't know if I am right, please tell me
hello,Have you verified your conjecture?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
hi, I'm new to this. So I'm wondering what eps_list is and what you did in detail. By the way, I can't find cluster_list used on line 366 in selftraining.py, did I do something wrong? |
作者改了代码,你debug一下代码就知道怎么做了 |
非常感谢您的回答,但我确实是个小白,所以现在还是一头雾水 |
you don't need cluster_list on line 366,it takes up 6GB,beacuse cluster.components_ is a is a matrix shape of (15xxx,16552),you can see this in site-packages/sklearn/cluster/dbscan_.py.
I think eps_list can have the same effect.
I don't know if I am right, please tell me
The text was updated successfully, but these errors were encountered: