agrignard / ReChamp

Project for the Champs Elysées project
1 stars 3 forks source link

potential memory leak.... #50

Closed agrignard closed 4 years ago

agrignard commented 4 years ago

Ce qui devait arriver arriva...

j ai l impression qu il y a un jolie petit memory leak qui prend bien du temps a apparaitre...

voila ce que jai au bout de 30000 iterations.... Screen Shot 2020-01-23 at 8 07 48 PM

ptaillandier commented 4 years ago

A tester : pour les graphes, mettre use_cache à false: people_graph <- people_graph use_cache false;

agrignard commented 4 years ago

ok je test ca , le truc très chiant c'est que ca arrive au bout du'n certains moment donc aps evident à tracker..; ca fait quoi en gros use_cache que je comprenne quand meme ce que j'ecris?

ptaillandier commented 4 years ago

Mettre use_cache à false permet de ne pas conserver en mémoire les plus court chemin. Je ne sais pas pourquoi cela provoquerait une fuite mémoire, mais c'est ce que j'ai vu quand j'ai testé avec le profiler. J'ai testé jusqu'à 25000 pas de temp avec use_cache à false pour tous les graphes et cela avait l'air de bien fonctionner sans fuite mémoire

Le ven. 24 janv. 2020 à 16:30, Arnaud Grignard notifications@github.com a écrit :

ok je test ca , le truc très chiant c'est que ca arrive au bout du'n certains moment donc aps evident à tracker..; ca fait quoi en gros use_cache que je comprenne quand meme ce que j'ecris?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/agrignard/ReChamp/issues/50?email_source=notifications&email_token=AALPWHPMSLGDM3QERM46ASLQ7MCRRA5CNFSM4KK7RRLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ3EZTI#issuecomment-578178253, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALPWHOA42NN4ZGJ7WP2FBLQ7MCRRANCNFSM4KK7RRLA .

agrignard commented 4 years ago

Ok la bonen news c'est que tu as un profiler (faudrait que tu m'apprennes ;-) Tu utilises YourKit?

Je test ca rapidement

ptaillandier commented 4 years ago

Yep mais la version de 2016, pour lequel on a une licence GAMA.

Le ven. 24 janv. 2020 à 16:46, Arnaud Grignard notifications@github.com a écrit :

Ok la bonen news c'est que tu as un profiler (faudrait que tu m'apprennes ;-) Tu utilises YourKit?

Je test ca rapidement

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/agrignard/ReChamp/issues/50?email_source=notifications&email_token=AALPWHIOHPD6A3HWDUEBMTTQ7MELPA5CNFSM4KK7RRLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ3GMOY#issuecomment-578184763, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALPWHJS6SNKHUX63FEXCHLQ7MELPANCNFSM4KK7RRLA .

agrignard commented 4 years ago

est ce que use_Cache false aussi pour celui la? graph g <- as_edge_graph(lines); dans manage_cycle_network

ptaillandier commented 4 years ago

Oui tous

Le ven. 24 janv. 2020 à 16:49, Arnaud Grignard notifications@github.com a écrit :

est ce que use_Cache false aussi pour celui la? graph g <- as_edge_graph(lines); dans manage_cycle_network

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/agrignard/ReChamp/issues/50?email_source=notifications&email_token=AALPWHLGKGKIELK7RHYLTMLQ7MEZVA5CNFSM4KK7RRLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ3GYWY#issuecomment-578186331, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALPWHOJEZ5AQL6XKHP3ZYDQ7MEZVANCNFSM4KK7RRLA .

ptaillandier commented 4 years ago

En fait, celui là on s'en fiche

Le ven. 24 janv. 2020 à 16:57, Patrick Taillandier < patrick.taillandier@gmail.com> a écrit :

Oui tous

Le ven. 24 janv. 2020 à 16:49, Arnaud Grignard notifications@github.com a écrit :

est ce que use_Cache false aussi pour celui la? graph g <- as_edge_graph(lines); dans manage_cycle_network

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/agrignard/ReChamp/issues/50?email_source=notifications&email_token=AALPWHLGKGKIELK7RHYLTMLQ7MEZVA5CNFSM4KK7RRLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ3GYWY#issuecomment-578186331, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALPWHOJEZ5AQL6XKHP3ZYDQ7MEZVANCNFSM4KK7RRLA .

agrignard commented 4 years ago

ok je viens d'updater et je lance sur un ordi en // pour voir comment ça monte en memoire

agrignard commented 4 years ago

en fait j'avais fait tourner sur un autre ordi qui avait dans les preferences d'emmetre un warning, je close pour le moment l'issue