Skip to content
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

Improve error handling and logging in parser #18

Merged
merged 1 commit into from
Apr 26, 2024
Merged

Conversation

ferdlestier
Copy link
Owner

@ferdlestier ferdlestier commented Apr 26, 2024

Implements improved error handling and logging in parser.py and updates documentation in README.md.

  • Error Handling and Logging in parser.py:

    • Imports the logging module and configures logging with a basic setup to write logs to parser.log.
    • Wraps the entire data parsing and analysis code block in a try-except structure to catch and log exceptions.
    • Adds logging statements at key points in the script to log information about the script's execution flow, such as successful loading of the CSV file, calculation of top daily growth and loss, and tracking of financial exposure and return on invested capital.
  • Documentation Updates in README.md:

    • Adds a new section on error handling and logging practices, explaining the importance of these improvements.
    • Provides guidelines on how to view and interpret the logs generated by parser.py, enhancing the user's ability to monitor and debug the script's execution.

For more details, open the Copilot Workspace session.

@ferdlestier ferdlestier merged commit 4d4cb5c into main Apr 26, 2024
3 of 4 checks passed
@ferdlestier ferdlestier deleted the enhance-logging branch July 23, 2024 20:36
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.

1 participant