Skip to content

fix: keep unique items visible in analysis - #17

Merged
Roman-Andr merged 3 commits into
mainfrom
dev
Aug 19, 2026
Merged

fix: keep unique items visible in analysis#17
Roman-Andr merged 3 commits into
mainfrom
dev

Conversation

@Roman-Andr

Copy link
Copy Markdown
Owner

Impact

  • keeps «Уникальные товары» beside «Дешевле всех» in the visible center of the analysis dashboard
  • moves the less critical average-price card to the right edge
  • preserves count and currency number formats after the layout change
  • synchronizes uv.lock with project version 0.3.1

For the supplied data copy.xlsm, the verified values are 149 cheapest-everywhere items and 2415 unique items.

Root cause

The unique-items metric existed in cells G5:H5, but the wide dashboard pushed it outside the initial viewport on narrower Excel windows. It therefore appeared to be missing even though its value was present.

Validation

  • .venv/bin/ruff check .
  • .venv/bin/mypy
  • UV_CACHE_DIR=/tmp/pharmparser-uv-cache uv lock --check
  • UV_CACHE_DIR=/tmp/pharmparser-uv-cache uv run pytest --ignore=tests/integration/test_gui.py — 327 passed
  • focused workbook and grid tests — 24 passed

The remaining GUI tests require Xvfb and are covered by CI.

@Roman-Andr
Roman-Andr merged commit af6cc76 into main Aug 19, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant