Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

Add timezone subcommand #1089

Merged
merged 9 commits into from
Feb 15, 2022
Merged

Add timezone subcommand #1089

merged 9 commits into from
Feb 15, 2022

Conversation

dchassin
Copy link
Member

@dchassin dchassin commented Jan 19, 2022 •
edited
Loading

This PR fixes issue #835 and adds the timezone subcommand to find the timezone for a location.

Current issues

  1. Only the most commonly used ISO timezones have been added. The remaining ones can be easily added should the need arise.

Code changes

  • Fix gldcore/timestamp.cpp
  • Add gldcore/scripts/gridlabd-timezone

Documentation changes

Test and Validation Notes

  • Add gldcore/scripts/autotest/test_timezone.glm

@dchassin dchassin added bug Request a fix or change to the code enhancement Request an enhancement or new feature labels Jan 19, 2022
@dchassin dchassin added this to the HiPAS 4.2 Final Release milestone Jan 19, 2022
@dchassin dchassin requested a review from aivanova5 January 19, 2022 19:55
@dchassin dchassin self-assigned this Jan 19, 2022
@dchassin dchassin linked an issue Jan 19, 2022 that may be closed by this pull request
Add ISO timezones to tzinfo.txt #835
Closed
@dchassin dchassin changed the title Develop add timezone subcommand Add timezone subcommand Jan 19, 2022
@dchassin dchassin mentioned this pull request Jan 19, 2022
Merged
23 tasks
@aivanova5 aivanova5 merged commit a595ba3 into develop Feb 15, 2022
@aivanova5 aivanova5 deleted the develop-add-timezone-subcommand branch February 15, 2022 00:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@aivanova5 aivanova5 aivanova5 approved these changes

Assignees

@dchassin dchassin

Labels
bug Request a fix or change to the code enhancement Request an enhancement or new feature
Projects
None yet
Milestone
HiPAS Patch Release
Development

Successfully merging this pull request may close these issues.

Add ISO timezones to tzinfo.txt
2 participants
@dchassin @aivanova5