Add grey fill in most of the charts

[LeoLuosifen]

No description provided.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[ziqi0067]

@FedericoTartarini We have finshed all tasks and passed the tests. Please review it.

[FedericoTartarini]

@LeoLuosifen great work. However, I found the following issue

1. chart breaking - please fix it

2. wrong legend colors - please fix it

3. strange layout - I am not sure what to suggest here and in the UTCI chart since they are hard to interpret when we filter by hour. Perhaps we cannot implement the gray out feature here

4. dots are shown - when they should not be - avoid implementing the gray out feature here

5. strage behaviour - see comment 4

[FedericoTartarini]

@LeoLuosifen the new code is great but there are so many repetitions, can we try to avoid repetitions, if possible otherwise the code is going to be very hard to maintain.

[giobetti]

Dear @federico and @LeoLuosifen<https://github.com/LeoLuosifen> , thanks for all the great changes! If I can interject here, I would suggest that the grey out feature should not be part of the bar charts (i.e. Cloud cover and UTCI). They should simply show the summary statistics with the filtered data. Best, Giovanni Message ID: ***@***.***>

[LeoLuosifen]

@FedericoTartarini @giobetti Thanks for your review and suggestions. I've already tried to resolve the issues mentioned above.

[FedericoTartarini]

@LeoLuosifen amazing work. I have reviewed the changes and I am happy with the code. I am going to publish a new test version. I also discovered how to run the tests in parallel locally, but it does not work in the github action pipenv run pytest --base-url=http://127.0.0.1:8080 -vv --numprocesses 4