Yeah agreed it was wrong before and at least now it's correct in that it's parseable.

I really don't know which is best access=yes or access=no here, so just seeing if you had an opinion.

If I'm making a map to show paths which are open to the public vs those closed/private, I'd show this as open since it's mostly open during the daylight hours when you'd expect to find things open. So I might want access=yes to hint this to data consumers as the default, or we could be agnostic and expect all data consumers to support :conditional and make their own decision on the default.

On the other hand, I think it's also reasonable for the tags to match what's signposted, if it's signposted as "closed 6:30pm to 6am" you might expect the tag value to match as 18:30-06:00, or if it says "open 6am to 6:30pm" you might expect the tag value to match as 06:00-18:30.

This comes up a lot so maybe I could ask for wider community feedback.