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

Updating logging #740

Closed
wants to merge 2 commits into from
Closed

Updating logging #740

wants to merge 2 commits into from

Conversation

astjoephysics
Copy link
Collaborator

Fixes # .

Describe your changes.

Update all instances of old logging (where possible) to now use the inbuilt sorchaArguments object. All functionality remains the same as prior.

Review Checklist for Source Code Changes

  • Does pip install still work?
  • Have you written a unit test for any new functions?
  • Do all the units tests run successfully?
  • Does Sorcha run successfully on a test set of input files/databases?
  • Have you used black on the files you have updated to confirm python programming style guide enforcement?

@mschwamb
Copy link
Collaborator

mschwamb commented Jan 8, 2024

#766 replace this

@mschwamb mschwamb closed this Jan 8, 2024
@mschwamb mschwamb deleted the UpdatingLogging branch January 8, 2024 16:59
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.

2 participants