Skip to content

Conversation

@ad
Copy link
Collaborator

@ad ad commented Jun 14, 2024

Related to #3

Implements error handling and recursion limit adjustment in lsofgraph.py to address the "maximum recursion depth exceeded" error, and updates the README.md with a note on handling large datasets.

  • Recursion Limit Adjustment: Adds sys.setrecursionlimit(15000) at the beginning of the if __name__ == '__main__': block in lsofgraph.py to preemptively increase the recursion limit before processing starts.
  • Error Handling: Wraps the main execution block in a try-except statement to catch RuntimeError related to recursion depth, providing a more graceful error message suggesting an increase in the recursion limit.
  • Documentation Update: Adds a note in README.md under the "Generate graph" section, advising on the potential for a "maximum recursion depth exceeded" error when handling large datasets and suggesting the workaround of increasing the recursion limit.

@akme
Copy link
Owner

akme commented Jun 14, 2024

LGTM

@akme akme merged commit 990ef96 into master Jun 14, 2024
@akme akme deleted the ad/fix-recursion-error branch June 14, 2024 14:25
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.

3 participants