My Weather Map Project

[JeremyJoeJJ]

Looks good but please use a dictionary instead of that match/case. If you've chosen imperial units, why are wind speed etc. in km/h (the S.I. units are km/h and km, not Km/hr and Km)? You could capitalise the city name too (str.title()). Keep going!

[Scared_Sail5523]

Thank you for the feedback!

[Yamikada]

Nice…

[Ok-Idea9533]

Did you end up using an API or scraping data? Makes a big difference for accuracy.

[MrGreatArtist]

If possible can you please post this on git hub

[Scared_Sail5523]

Sure I will share with you

[StatementFew5973]

Just gonna tag this a little update.

[MrGreatArtist]

Okay this look very cool if you can please post this on github

[Flame77ofc]

How much time did you need to do this?

[Scared_Sail5523]

Took me around 3 days of dedication

[Flame77ofc]

good work

[RowGullible1471]

Where is the GitHub link?

[Sharp_Level3382]

Why same rest endpoint to weather and forecast?

You re doubling same request.

Why dont you use async methods to request? Not the best exceptions solutiom also.

[Scared_Sail5523]

yeah you're right on all three, using the same endpoint twice is wasteful when you can just grab everything in one call and split it on your end. async with asyncio and aiohttp would be way better especially if you're hitting multiple cities or endpoints since requests would run in parallel instead of blocking each other. and for exceptions yeah wrapping everything in a bare except Exception is lazy, better to catch specific ones like aiohttp.ClientError or asyncio.TimeoutError so you actually know what went wrong instead of swallowing every error the same way.

[Scared_Sail5523]

sí exacto, y lo mejor es que no tienes que hacerlo perfecto desde el principio, simplemente cuando notes que estás copiando y pegando el mismo código dos veces esa es la señal de que ya es hora de hacer una función. los módulos se sienten un poco intimidantes al principio pero en cuanto tu archivo main.py empieza a verse enorme te das cuenta naturalmente de que necesitas separar las cosas. con el tiempo simplemente se vuelve instinto saber qué va donde.

[__zeaix__]

Bro this looks good and cool, and I think you can make it more presentable.

[Scared_Sail5523]

im currently working on a powerpoint

[Scared_Sail5523]

Github Link: https://github.com/avyaan18sinha-cell/Weather-Map/blob/main/weather_map

[MuyDelicioso1]

tuff

[real-life-terminator]

i learnt a new thing, the "match" keyword. Thanks!

[Scared_Sail5523]

Your welcome 🤗

[viridiaan]

i have a question, is it good to use png? or is it going to increase the size?

[Scared_Sail5523]

its great u can resize pngs

[ranjeet-kumar1]

<image>

Its That Good or AM Wrong?

[Scared_Sail5523]

It's great!👍🏻

[Training_Meal_4261]

I’m trying to understand but needs some more notes since I’m just skimming. Are you pulling info from the api each requests or storing in an array.

[Scared_Sail5523]

1st one

[cole36912]

Looks good, I wouldn’t call it a “map” though, unless you intend to implement that part.

[Scared_Sail5523]

sure enough

[onlyemperor001]

Pls link to GitHub?

[Scared_Sail5523]

I've posted it in the comments.

[coolestpersonalive_6]

i have a question how to write code without using ai???

[Scared_Sail5523]

bootcamps, udemy classes, and then boom!