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

Odd time print format from Cap #3034

Closed
mathomp4 opened this issue Sep 19, 2024 · 3 comments
Closed

Odd time print format from Cap #3034

mathomp4 opened this issue Sep 19, 2024 · 3 comments
Assignees
Labels
❓ Question Further information is requested

Comments

@mathomp4
Copy link
Member

Per @briardew, he was wondering about these prints from Cap:

CAP: INFO: Read CAP restart properly, Current Date =   2019/08/08
CAP: INFO:                            Current Time =   03/00/00 

Namely: Why is the time represented by slashes. My only guess is that it was that way long ago in MAPL and, well, we just never changed it.

The code would be simple to change:

call lgr%info('Read CAP restart properly, Current Date = %i4.4~/%i2.2~/%i2.2', CUR_YY, CUR_MM, CUR_DD)
call lgr%info(' Current Time = %i2.2~/%i2.2~/%i2.2', CUR_H, CUR_M, CUR_S)

The question is: Should we? I appeal to @atrayano and @bena-nasa to know if they know of anyone or any scripts that depend on this format?

I'll even mention at @rtodling in case he knows of an ADAS script that depends on this output for any reason.

And, why not, I'll mention @tclune because everyone else has been. 😄

@mathomp4 mathomp4 added the ❓ Question Further information is requested label Sep 19, 2024
@mathomp4 mathomp4 self-assigned this Sep 19, 2024
@atrayano
Copy link
Contributor

I ready do not think anybody cares... Cannot imagine that we parse the time from this output. My vote is to change it.

@briardew
Copy link

briardew commented Sep 19, 2024 via email

@mathomp4
Copy link
Member Author

Just for @tclune benefit, I did communicate to @briardew that MAPL3 will be using ISO time format.

I suppose if we wanted, we could conver this logging output to ISO...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❓ Question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants