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

Fix wallTimeSecs() for Windows #161

Merged

dawson6 merged 2 commits into main from windowsFix

Jul 23, 2024

Collaborator

liu15 commented Jul 10, 2024 •

edited

Loading

Also fix periodic reporting output for Windows.


          Fix wallTimeSecs() for Windows

fe9624a

liu15 requested a review from dawson6

July 10, 2024 19:30


          Improved windows reporting

d2b6678

dawson6 approved these changes

View reviewed changes

Member

dawson6 left a comment

Will pull and test this later. Approving now.

Collaborator Author

liu15 commented Jul 16, 2024

How about you merge it after you have had a chance to test it?

Member

dawson6 commented Jul 16, 2024 via email

I can do that.

________________________________ From: Ben Liu ***@***.***> Sent: Tuesday, July 16, 2024 9:57 AM To: LLNL/ATS ***@***.***> Cc: Dawson, Shawn A. ***@***.***>; Review requested ***@***.***> Subject: Re: [LLNL/ATS] Fix wallTimeSecs() for Windows (PR #161) How about you merge it after you have had a chance to test it? — Reply to this email directly, view it on GitHub<https://urldefense.us/v2/url?u=https-3A__github.com_LLNL_ATS_pull_161-23issuecomment-2D2231400263&d=DwMCaQ&c=pKoAVQro6qDbLoK0T8588B4mZJhJhC4e6QXJy0XnJec&r=LK15r9M5QEXJEXaQ_801XVi5NXL1NFGMThXvmDL4PJM&m=wvbYo8BXC3fgd__aqwgYYM_-zJQJ-GY55NjjU1BeN2JfKssZam-zomxAz87Zjhdm&s=MoEaELz8KDb79UctzY-2Gyo5oobk31_eCMGtFs56U-0&e=>, or unsubscribe<https://urldefense.us/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AB7SCJTWI64GNEJGDTQKBULZMVGIHAVCNFSM6AAAAABKVRKW2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZRGQYDAMRWGM&d=DwMCaQ&c=pKoAVQro6qDbLoK0T8588B4mZJhJhC4e6QXJy0XnJec&r=LK15r9M5QEXJEXaQ_801XVi5NXL1NFGMThXvmDL4PJM&m=wvbYo8BXC3fgd__aqwgYYM_-zJQJ-GY55NjjU1BeN2JfKssZam-zomxAz87Zjhdm&s=EivRjMdpdd2N9AUCa71UqYRmsyRLPXtAGchno_rh940&e=>. You are receiving this because your review was requested.Message ID: ***@***.***>

dawson6 merged commit 346ed95 into main

18 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet