Skip to content

New value 149 for FOG#74

Open
HansChaudry wants to merge 1 commit into
null8626:mainfrom
HansChaudry:main
Open

New value 149 for FOG#74
HansChaudry wants to merge 1 commit into
null8626:mainfrom
HansChaudry:main

Conversation

@HansChaudry

Copy link
Copy Markdown

While testing client.get("New York"), I encountered a ValueError because weather code 149 is not recognized by the Kind enum.

The upstream API response identifies weather code 149 as smoky haze. At the time, smoke from Canadian wildfires was affecting the northeastern United States.

As wildfire smoke events become more common, this code may appear more frequently. I propose adding 149 as an associated value for Kind.FOG, since it is the closest existing category. A dedicated SMOKE or HAZE enum value could also be considered if greater specificity is preferred.

Error:
Screenshot 2026-07-17 at 9 11 49 AM

Raw json from curl -L "https://wttr.in/New+York?format=j1"

Screenshot 2026-07-17 at 9 13 09 AM

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 duplication

Metric Results
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

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.

1 participant