feat: Add India weather connector for FarmVibes.AI - #236
feat: Add India weather connector for FarmVibes.AI#236Nikhil Chaudhary (nik464) wants to merge 1 commit into
Conversation
Gap fixed: FarmVibes.AI only supported US/NOAA weather data. This adds Indian coordinate support via Open-Meteo with: - 4 major Indian crop types (wheat, rice, mustard, sugarcane) - Crop-specific weather risk assessment - IST timezone support - Agri variables: ET0, radiation, precipitation
|
Nikhil Chaudhary (@nik464) please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
Renato L. de F. Cunha (renatolfc)
left a comment
There was a problem hiding this comment.
Thanks for the contribution. This is not wired into FarmVibes yet: the weather file is neither packaged nor registered as an operation or workflow. The current implementation also has an off-by-one date range, incorrect timezone handling, and silently reports low risk for sugarcane and unknown crops. Please rebase onto current main, add a real op/workflow contract, correct the date and timezone behavior, reject unsupported crops explicitly, and cover those paths with focused tests.
Gap fixed: FarmVibes.AI only supported US/NOAA weather data. This adds Indian coordinate support via Open-Meteo with: