This is a useful feature and I think it should be implemented. The current implementation appears sound. Since this is exposing the existing internal Carpet get_walltime function to end-users, it would be nice to have some documentation for this quantity so that users can interpret the output. This could take the form of a comment in param.ccl. Perhaps something like “Current wall clock time, as number of seconds since 00:00:00 UTC, January 1, 1970”.
Thank you. I have updated the pull request. If you approve this change, please click on the “Approve” button on the top right of the Bitbucket website.