Skip to content

fix: time prompt to include UTC, convert to verbose english to help prompting#603

Merged
shakkernerd merged 2 commits into
elizaOS:mainfrom
odilitime:time-fix
Nov 26, 2024
Merged

fix: time prompt to include UTC, convert to verbose english to help prompting#603
shakkernerd merged 2 commits into
elizaOS:mainfrom
odilitime:time-fix

Conversation

@odilitime
Copy link
Copy Markdown
Collaborator

Risks

Low

Background

What does this PR do?

Pass the time/date at GMT in english to the LLM, so it's not the timezone the server is in

What kind of change is this?

Bug fixes (non-breaking change which fixes an issue)

Why are we doing this? Any context or related work?

So you can ask it what time is in specific time zones and it works (if you server's localtime is not set to UTC)

Documentation changes needed?

My changes do not require a change to the project documentation.

Copy link
Copy Markdown
Contributor

@mcp97 mcp97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM outside of unneeded comment :)

Comment thread packages/plugin-bootstrap/src/providers/time.ts Outdated
@shakkernerd shakkernerd merged commit db9a857 into elizaOS:main Nov 26, 2024
lalalune pushed a commit that referenced this pull request May 3, 2026
fix: time prompt to include UTC, convert to verbose english to help prompting
FranceFlapjack pushed a commit to FranceFlapjack/eliza that referenced this pull request May 16, 2026
fix: time prompt to include UTC, convert to verbose english to help prompting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants