voiio / Sam

Sam – cuz your company is nothing without Sam
https://code.voiio.de/Sam/
BSD 3-Clause "New" or "Revised" License
9 stars 1 forks source link

Ref #36 -- Add time of day as additional instruction #47

Closed mostafa-anm closed 7 months ago

codecov[bot] commented 7 months ago

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 59.94%. Comparing base (521a2b8) to head (caecd67).

:exclamation: Current head caecd67 differs from pull request most recent head f4e4323. Consider uploading reports for the commit f4e4323 to get more accurate results

Files Patch % Lines
sam/slack.py 0.00% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #47 +/- ## ========================================== - Coverage 60.41% 59.94% -0.47% ========================================== Files 11 11 Lines 389 387 -2 ========================================== - Hits 235 232 -3 - Misses 154 155 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

herrbenesch commented 7 months ago

@mostafa-anm is there a test yet for the system prompt? you might created/update it.

codingjoe commented 7 months ago

You can simply use sam.config.TIMEZONE. It'll have what you need.

mostafa-anm commented 7 months ago

@mostafa-anm is there a test yet for the system prompt? you might created/update it.

@herrbenesch Sure, I will look into it in here! https://github.com/voiio/Sam/issues/54

@codingjoe Thnx for suggesting the utils sam.config.TIMEZONE