CP Legendre commited on
Commit ·
38eb34d
1
Parent(s): e21dd2d
Reduce ranking row height
Browse files- src/charts.py +1 -1
- tests/test_ranking_refinements.py +2 -1
src/charts.py
CHANGED
|
@@ -25,7 +25,7 @@ DEFAULT_PALETTE: PaletteName = "Citrus"
|
|
| 25 |
DEFAULT_BACKGROUND: PlotBackground = "Dark"
|
| 26 |
|
| 27 |
RANKING_MIN_HEIGHT_PX = 340
|
| 28 |
-
RANKING_ROW_HEIGHT_PX =
|
| 29 |
RANKING_VERTICAL_PADDING_PX = 280
|
| 30 |
MATRIX_MIN_HEIGHT_PX = 720
|
| 31 |
MATRIX_ROW_HEIGHT_PX = 36
|
|
|
|
| 25 |
DEFAULT_BACKGROUND: PlotBackground = "Dark"
|
| 26 |
|
| 27 |
RANKING_MIN_HEIGHT_PX = 340
|
| 28 |
+
RANKING_ROW_HEIGHT_PX = 22
|
| 29 |
RANKING_VERTICAL_PADDING_PX = 280
|
| 30 |
MATRIX_MIN_HEIGHT_PX = 720
|
| 31 |
MATRIX_ROW_HEIGHT_PX = 36
|
tests/test_ranking_refinements.py
CHANGED
|
@@ -86,7 +86,8 @@ def test_ranking_plot_height_scales_with_agents_and_keeps_all_y_labels():
|
|
| 86 |
|
| 87 |
figure = create_ranking_plot(df, "Score (%)", "Score", True, sort_order="Best first")
|
| 88 |
|
| 89 |
-
assert figure.layout.height >=
|
|
|
|
| 90 |
assert figure.layout.yaxis.tickmode == "array"
|
| 91 |
assert len(figure.layout.yaxis.tickvals) == rows
|
| 92 |
assert len(figure.layout.yaxis.ticktext) == rows
|
|
|
|
| 86 |
|
| 87 |
figure = create_ranking_plot(df, "Score (%)", "Score", True, sort_order="Best first")
|
| 88 |
|
| 89 |
+
assert figure.layout.height >= 340
|
| 90 |
+
assert figure.layout.height < 900
|
| 91 |
assert figure.layout.yaxis.tickmode == "array"
|
| 92 |
assert len(figure.layout.yaxis.tickvals) == rows
|
| 93 |
assert len(figure.layout.yaxis.ticktext) == rows
|