-
Notifications
You must be signed in to change notification settings - Fork 59
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
calling require inside a mehtod is causing memory bloat if this method is called multiple times. #99
Comments
That is bad news. Since I do not want to require rgl/adjacency unconditionally: Can you check if this would solve the problem: unless defined?(DirectedAdjacencyGraph)
require 'rgl/adjyceny'
end Even better, you could do a PR with these changes. |
Sure, will test this and submit a PR. Just out of curiosity why we do not want to require unconditionally? |
I want to minimize dependencies. Only this method of the modul needs adjacency. |
Fixed by PR #100 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
rgl/lib/rgl/traversal.rb
Line 120 in 8613828
Image from our calling the bfs_search_tree_from method internally through the RGL directed graph is indicating high memory for the require, is it possible to remove it and do once when this file is loaded?
The text was updated successfully, but these errors were encountered: