ecosys icon indicating copy to clipboard operation
ecosys copied to clipboard

Update circleDetection.gsql

Open louisza opened this issue 5 years ago • 3 comments

Optimise memory usage of the algorithm - (based on my experience with this algorithm eating up 10x memory compared to idle state)

Cleaning up previously visited list accumulators greatly improves memory usage for this algorithm. This is done by removing the edgeTupleList from the source in the post-Accum step as the new paths are already forwarded to the next vertexes in the path. To my understanding, at that point, there is no need to store the paths in source vertex.

louisza avatar Oct 28 '20 09:10 louisza

@louisza Thank you for your contribution! Our algorithm team will review today and get back to you with any questions and/or go forwards with the pull request. We appreciate your input and are excited to have you here in the community!

JonHerke-TG avatar Oct 28 '20 12:10 JonHerke-TG

Please remove me

Sincerely, Jeff Haber Sent from my iPhone

On Oct 28, 2020, at 5:53 AM, HerkTG [email protected] wrote:

 @louisza Thank you for your contribution! Our algorithm team will review today and get back to you with any questions and/or go forwards with the pull request. We appreciate your input and are excited to have you here in the community!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

JeffHMB avatar Oct 28 '20 13:10 JeffHMB

@JeffHMB you will need to "unstar" tigergraph ecosys if you would not like to receive updates. Unfortunately, you will need to log into your account to do this.

JonHerke-TG avatar Jan 28 '21 20:01 JonHerke-TG