Modern Tooling + Parallel Vault loading#55
Closed
JavierGOrdonnez wants to merge 9 commits intomfarragher:mainfrom
Closed
Modern Tooling + Parallel Vault loading#55JavierGOrdonnez wants to merge 9 commits intomfarragher:mainfrom
JavierGOrdonnez wants to merge 9 commits intomfarragher:mainfrom
Conversation
- Replace setup.py + requirements*.txt with a single pyproject.toml using hatchling as build backend - Add [tool.uv] dev-dependencies for uv-managed dev workflow - Add [tool.ruff] config for linting (E/W/F/I/UP/B/C4/SIM) and formatting (replaces any prior flake8/black/isort setup) - Add [tool.ty] config for static type checking with ty - Add [tool.pytest.ini_options] and [tool.coverage] sections - Update CI to use astral-sh/setup-uv and uv sync/run, upgrade actions/checkout to v4, add dedicated lint and typecheck jobs uv.lock will be generated automatically on first `uv sync`. https://claude.ai/code/session_01G13wzGLvmoog4E73fUairt
- Fix F401: use explicit re-export syntax in __init__.py (as api, etc.) - Fix I001: sort all import blocks to isort standard - Fix C416/C404/C405: replace unnecessary comprehensions with simpler equivalents (dict(), list(), set literals) - Fix SIM118: remove .keys() in dict membership tests - Fix UP032: convert .format() calls to f-strings - Fix B006: replace mutable default arg with None sentinel in html_processing._remove_main_formatting - Fix B028: add stacklevel=2 to warnings.warn in api.py - Fix E722: replace bare except with except Exception in md_utils.py - Fix W291: suppress intentional trailing whitespace in test string literal via per-file-ignores in pyproject.toml - Run ruff format over all 23 files (consistent style) - Remove --cov flags from pytest addopts (added by CI, not hardcoded) https://claude.ai/code/session_01G13wzGLvmoog4E73fUairt
…n-malformed-file Add skipping avoid crash on malformed file
JavierGOrdonnez
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is a follow-up PR from #54 (so it includes those changes - modern python tooling: uv, ruff, ...) and additionally introduces parallelism in the Vault generation:
On a 12-core AMD processor:
connect()and 1.5x ongather()connect()andgather()