-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Closed
Labels
.NETIssue or Pull requests regarding .NET codeIssue or Pull requests regarding .NET codepythonPull requests for the Python Semantic KernelPull requests for the Python Semantic Kernelsk team issueA tag to denote issues that where created by the Semantic Kernel team (i.e., not the community)A tag to denote issues that where created by the Semantic Kernel team (i.e., not the community)
Milestone
Description
Looking at the current Time skills in C# and Python, sometimes the code use the local culture, in other case the format is hard-coded and could cause unexpected results (e.g. when using the planner), e.g. month-day order being reversed only for some functions.
It would help adding a config to the skill constructor, so that a user can explicitly set which format/locale/culture setting to use in all functions.
See also comments on PR #927
/cc @tim-gill
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
.NETIssue or Pull requests regarding .NET codeIssue or Pull requests regarding .NET codepythonPull requests for the Python Semantic KernelPull requests for the Python Semantic Kernelsk team issueA tag to denote issues that where created by the Semantic Kernel team (i.e., not the community)A tag to denote issues that where created by the Semantic Kernel team (i.e., not the community)
Type
Projects
Status
Sprint: Done