codecov[bot]
commented
2 years ago Codecov Report
Merging #65 (dd81331) into main (06ad9d7) will decrease coverage by
0.42%. The diff coverage is5.98%.
@@ Coverage Diff @@
## main #65 +/- ##
==========================================
- Coverage 20.09% 19.67% -0.43%
==========================================
Files 19 21 +2
Lines 1612 1677 +65
==========================================
+ Hits 324 330 +6
- Misses 1288 1347 +59 | Impacted Files | Coverage Δ | |
|---|---|---|
| ruins/apps/climate.py | 0.00% <0.00%> (ø) |
|
| ruins/components/model_scale_select.py | 0.00% <0.00%> (ø) |
|
| ruins/plotting/stripes_heatmap.py | 12.98% <ø> (ø) |
|
| ruins/components/data_select.py | 11.29% <8.82%> (+0.28%) |
:arrow_up: |
| ruins/plotting/climate_parcoords.py | 23.80% <23.80%> (ø) |
|
| ruins/plotting/weather_data.py | 8.00% <33.33%> (-0.11%) |
:arrow_down: |
| ruins/core/config.py | 78.40% <100.00%> (-0.25%) |
:arrow_down: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 06ad9d7...dd81331. Read the comment docs.
This PR should only be reviewed and merged after #64.
The scatterplot for comparing different RCPs was completely rewritten and replaced by a parallel coordinates plot. This basically includes all information the original plot was transporting, but selecting, zooming and inspecting is way easier now. I also removed the plotting of two scenarios at the same time and reworked the full application workflow to let the user add and configure new plots as they are needed. Each parallel coordinates plot goes into its own expander, which makes actual comparing of sceanrios possible. That was not easily possible for the scatterplots, as they were just too full.
@AlexDo1, you can run the new stuff locally and spot problems, @cojacoo, @jonaslenz general feedback on the 'new' climate explorer is very welcome, as I changed more or less everything. I hope for the better.