search

arviz-devs / arviz-plots

ArviZ modular plotting
https://arviz-plots.readthedocs.io
Apache License 2.0
2 stars 1 forks source link

Add neutral element concept to aes generation #35

Closed OriolAbril closed 5 months ago

OriolAbril commented 5 months ago

Closes #33

📚 Documentation preview 📚: https://arviz-plots--35.org.readthedocs.build/en/35/

codecov-commenter commented 5 months ago

Codecov Report

Attention: Patch coverage is 90.90909% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 76.82%. Comparing base (cfa1ebb) to head (9cd8522).

Files Patch % Lines
src/arviz_plots/plot_collection.py 90.47% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #35 +/- ## ========================================== + Coverage 76.65% 76.82% +0.16% ========================================== Files 15 15 Lines 1281 1299 +18 ========================================== + Hits 982 998 +16 - Misses 299 301 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

OriolAbril commented 5 months ago

I have to update _get_aes_dict_from_dt and add tests for it.