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.

Sign up for GitHub

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

Jump to bottom

adding logging module #34

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

adding logging module #34

wants to merge 1 commit into from

Conversation

wjsutton
Copy link
Contributor

@wjsutton wjsutton commented Oct 9, 2024

Logging module:

  • Records inputs, outputs, actions to timestamped log files under folder "logs"
  • Excludes data structure sent to VDS
  • implemented on all current modules and main.py

@wjsutton wjsutton linked an issue Oct 9, 2024 that may be closed by this pull request
Logging #16
Open
@stephenlprice stephenlprice self-requested a review October 9, 2024 14:55
@stephenlprice
Copy link
Collaborator

great job @wjsutton. I'll review this before our next meeting

@stephenlprice stephenlprice added the enhancement New feature or request label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stephenlprice stephenlprice Awaiting requested review from stephenlprice stephenlprice is a code owner

@joeconstantino joeconstantino Awaiting requested review from joeconstantino joeconstantino is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@wjsutton wjsutton

Labels
enhancement New feature or request
Projects
langtab
Status: In review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Logging
2 participants
@wjsutton @stephenlprice