BharatSahAIyak / kisai-bot

0 stars 0 forks source link

Weather based crop advisory #135

Open prtkjakhar opened 5 months ago

prtkjakhar commented 5 months ago

Description

Buttons of different crops below weather widget which show a modal based advisory for that crop.

Need clarity on how can we show crops to farmers, different crops for each farmer based on region? API for this data? @ChakshuGautam @aashutosh-samagra @Amruth-Vamshi

aashutosh-samagra commented 4 months ago
ChakshuGautam commented 4 months ago
aashutosh-samagra commented 4 months ago
rachitavya commented 4 months ago

Maintaining API automation scraping here: https://github.com/BharatSahAIyak/upcar-advisory

Response for this PDF: response.json

KDwevedi commented 4 months ago

PDFS

rachitavya commented 4 months ago

Common/Unique fields in first 3 PDFs There is a same pattern in every PDF:

  1. Introduction: No use for us
  2. Next 1st week weather forcast
  3. 8th day - 14th day weather forecast
  4. General Advisory
  5. Crop Specific advisories in bullets
  6. References: No use for us

    Discrepancies between current schema and PDF information

Main discrepancies lies in weather_details: Previously: Daily weather data in numbers for parameters like min_temp, max_temp, min_rh, max_rh, wind_direction, cloud_cover, etc. Now in upcar we have: Two weeks data in paragraph having:

parsing errors (not exhaustive is fine)

  • Values for weather details are hallucinations of GPT. e.g. It is giving values for rh_max even if there is no mention of rh_max in the PDF.
  • More detailed advisories in PDF, less advisories coming in response to us.
KDwevedi commented 4 months ago

Current Status:

KDwevedi commented 4 months ago

@rachitavya

Get Accurate Data

rachitavya commented 4 months ago

@prtkjakhar Can you elaborate on what basis we put the weather data on the bot. On the basis of location ? If yes, then how do we select the parameter from the apis. Do we select district or we need lattitude / longitude ?

I wanted to know about the flow so that we can figure out what to do for missing districts in imd APIs.

cc: @aashutosh-samagra @KDwevedi

prtkjakhar commented 4 months ago

@rachitavya if you are talking about the weather data we get inside message then I send lat. long. via socket. If you are talking about weather page then @Amruth-Vamshi has provided a bff api that takes lat. long. and returns weather data.

rachitavya commented 4 months ago

https://github.com/BharatSahAIyak/upcar-advisory

This repo is updated for:

Will share the API endpoint after deploying.

To do:

@ChakshuGautam @aashutosh-samagra @KDwevedi

KDwevedi commented 4 months ago

API response might be unsuitable for weather:

Should re-evaluate weather pipeline

KDwevedi commented 4 months ago

blocked on weather APIs being down @aashutosh-samagra @ChakshuGautam

aashutosh-samagra commented 4 months ago

@rachitavya can you point me to the latest JSON which has the weather based crop advisory, i checked one JSON which had too concise information and relevant crops/lines were getting dropped.

rachitavya commented 4 months ago

@aashutosh-samagra https://github.com/BharatSahAIyak/upcar-advisory/blob/dev/latest.json

aashutosh-samagra commented 4 months ago

Weather APIs are now up @KDwevedi @rachitavya , we can schedule a call to discuss

aashutosh-samagra commented 4 months ago

@aashutosh-samagra https://github.com/BharatSahAIyak/upcar-advisory/blob/dev/latest.json

@rachitavya I checked this JSON only, we can relook at it since the information is too concise and relevant crops/lines are getting dropped.

prtkjakhar commented 4 months ago

Managing new tickets on the repo itself

aashutosh-samagra commented 4 months ago

Enhancements in UPCAR weather advisory API -