Timestamp on combiner log

[aitmlouk]

Is your feature request related to a problem? Please describe.
Why not adding a Timestamp to the combiner log like reducer, then at least it will help to track errors for long training

Describe the solution you'd like
A clear and concise description of what you want to happen.
Instead of having this:
combiner_1 | COMBINER(FEDn_Combiner_addi):0 COMBINER: waiting for model updates: 0 of 4 completed.

it's better to have this:
combiner_1 | COMBINER(FEDn_Combiner_addi): 09/24/2021 09:38:58 AM 0 COMBINER: waiting for model updates: 0 of 4 completed.

Describe any drawbacks (if any)
None

Contact Details
An e-mail address in case we need to contact you for further details regarding this request.

[ahellander]

Do we currently log this to a file on the combiner? Best regards, Andreas Hellander Associate Professor, Scientific Computing 24 sep. 2021 kl. 11:52 skrev Addi Ait-Mlouk ***@***.******@***.***>>: Is your feature request related to a problem? Please describe. Why not adding a Timestamp to the combiner log like reducer, then at least it will help to track errors for long training Describe the solution you'd like A clear and concise description of what you want to happen. Instead of having this: combiner_1 | COMBINER(FEDn_Combiner_addi):0 COMBINER: waiting for model updates: 0 of 4 completed. it's better to have this: combiner_1 | COMBINER(FEDn_Combiner_addi): 09/24/2021 09:38:58 AM 0 COMBINER: waiting for model updates: 0 of 4 completed. Describe any drawbacks (if any) None Contact Details An e-mail address in case we need to contact you for further details regarding this request. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#284>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AASBHZQGHKDVHGWRL3QRKGTUDRC6JANCNFSM5EVSQ2PA>. Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. När du har kontakt med oss på Uppsala universitet med e-post så innebär det att vi behandlar dina personuppgifter. För att läsa mer om hur vi gör det kan du läsa här: http://www.uu.se/om-uu/dataskydd-personuppgifter/ E-mailing Uppsala University means that we will process your personal data. For more information on how this is performed, please read here: http://www.uu.se/en/about-uu/data-protection-policy

[aitmlouk]

We have this on reducer and client-log but not on combiners

[Wrede]

Merged in #296