-
Notifications
You must be signed in to change notification settings - Fork 17
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
Labels
❓ Question
Further information is requested
Comments
I ready do not think anybody cares... Cannot imagine that we parse the time from this output. My vote is to change it. |
mathomp4
added a commit
that referenced
this issue
Sep 19, 2024
7 tasks
mathomp4
added a commit
that referenced
this issue
Sep 19, 2024
…-print Fixes #3034. Convert Cap time print to colons
If we want to go real crazy, we could also change the date to be 2019-08-08
to be more ISOy. Agree with Atanas that likely no one cares except me.
…On Thu, Sep 19, 2024 at 8:48 AM Atanas Trayanov ***@***.***> wrote:
I ready do not think anybody cares... Cannot imagine that we parse the
time from this output. My vote is to change it.
—
Reply to this email directly, view it on GitHub
<#3034 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKXORYVNFYCVJUG2GOID4GTZXLBYFAVCNFSM6AAAAABOPZGWZWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRQHA4TKMZQHA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Per @briardew, he was wondering about these prints from Cap:
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:
MAPL/gridcomps/Cap/MAPL_CapGridComp.F90
Lines 1852 to 1853 in c7926cd
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. 😄
The text was updated successfully, but these errors were encountered: