Skip to content

Conversation

@AmanDevelops
Copy link
Contributor

Issue Referenced

closes #47

Describe the Changes

Change 1: assistant.py

added a query processor for calling the weather_reporter function

Change 2: commands.py

implementation of weather_reporter function:

Type of Change

  • New feature (non-breaking change which adds functionality)

Additional Setup Requirements

No Response

Necessary Conditions

  • I have commented on my code, particularly in hard-to-understand areas.
  • My changes generate no new warnings.
  • The changes do not involve any Private data leak/reveal.
  • Secrets, if any, are kept hidden via an environment file, whose structure has been explained via documentation.

Please make sure to check the following milestones before submitting the bug report!

Copy link
Owner

@vihar-s1 vihar-s1 left a comment

Choose a reason for hiding this comment

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

Please share a short video demonstrating the feature

@AmanDevelops
Copy link
Contributor Author

AmanDevelops commented Jan 30, 2025

Please share a short video demonstrating the feature

here is the required video demonstration @vihar-s1

screen-recorder-fri-jan-31-2025-00-39-43.webm

@vihar-s1
Copy link
Owner

vihar-s1 commented Feb 1, 2025

sorry for the delay @AmanDevelops...was busy with office work...will review your PRs by EOD today

@vihar-s1 vihar-s1 merged commit 761d9e4 into vihar-s1:main Feb 1, 2025
5 checks passed
@AmanDevelops AmanDevelops deleted the weather branch February 5, 2025 18:45
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.

[FEAT]: Weather Forecasting

2 participants