chenmoneygithub commented 2 weeks ago

What does this PR do?

Add buffering time to mlflow logger, tested with unit test along with manual testing.

What issue(s) does this change relate to?

[x] Have you read the contributor guidelines?
[ ] Is this change a documentation change or typo fix? If so, skip the rest of this checklist.
[x] Was this change discussed/approved in a GitHub issue first? It is much more likely to be merged if so.
[ ] Did you update any related docs and document your change?
[ ] Did you update any related tests and add any new tests related to your change? (see testing)
[ ] Did you run the tests locally to make sure they pass?
[ ] Did you run pre-commit on your change? (see the pre-commit section of prerequisites)

mvpatel2000 commented 2 weeks ago

qq: does this actually need to be configurable? or should you just pick a good default value?

Hm yea @chenmoneygithub can we set a default at least to something you'd like? maybe 5s?

chenmoneygithub commented 2 weeks ago

I am putting 10s for now.