Skip to content

Add event logging - #208

Merged
allison-li-1016 merged 9 commits into
mainfrom
allison/add-event-logging
Apr 14, 2026
Merged

Add event logging#208
allison-li-1016 merged 9 commits into
mainfrom
allison/add-event-logging

Conversation

@allison-li-1016

@allison-li-1016 allison-li-1016 commented Nov 12, 2025

Copy link
Copy Markdown
Contributor

Estimated time to review: small

Addresses #191

Description of changes:

  • added CLI option for logging events
  • added an 'EVENTS' output file logging events that occur at given tick intervals
  • added logging information to cytotoxic module to track lysis events & proliferation module to track division events and cycle length
  • edit tests to include new mocks for event logging

Discussion:

  • We can consider logging division as an event as a proxy for cell cycle durations
  • I am concerned about the events queue object getting exponentially larger if we start tracking frequent events, we should discuss clearing the queue once the events are logged

@allison-li-1016
allison-li-1016 marked this pull request as draft November 12, 2025 22:08
@allison-li-1016
allison-li-1016 marked this pull request as ready for review November 13, 2025 00:42
@allison-li-1016
allison-li-1016 requested review from a team, Jannetty, cainja, daniellevahdat, jacob-evarts, jessicasyu, kristaphommatha and pohaoc2 and removed request for a team November 13, 2025 00:42
@Jannetty

Copy link
Copy Markdown
Member

Recording that we spoke about making sure to clear the events queue object each time it writes to file

@kristaphommatha kristaphommatha left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@allison-li-1016

Copy link
Copy Markdown
Contributor Author

@allison-li-1016 consider adding eventLog object in the Utils class to enforce logs to be of a certain format....

@allison-li-1016
allison-li-1016 merged commit 303cbdf into main Apr 14, 2026
6 checks passed
@allison-li-1016
allison-li-1016 deleted the allison/add-event-logging branch April 14, 2026 19:12
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