Restructure package deps, add CI, fix lint#7
Merged
Conversation
Core deps (15) include only what's needed for Operator + Reconciler + Aggregator + QA: openai, curator, faiss-cpu, voyageai, together, pandas, pyarrow, scikit-learn, etc. Heavy deps (torch, vllm, unsloth, flash-attn, wandb, etc) moved to [training] extra. Viz deps (plotly, streamlit) moved to [viz] extra. For GPU FAISS, install faiss-gpu in place of faiss-cpu at install time. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Evaluation imports (TulvingBenchEvaluator, hipporag_eval) are now wrapped in try/except so import gsw_memory doesn't crash when optional evaluation dependencies are missing. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Replace wildcard import in prompts/__init__.py with explicit imports - Remove unused variable in visualization.py - Auto-fix 130 lint issues (unused imports, f-strings without placeholders) - Run ruff format on all 30 files that needed it src/ now passes both ruff check and ruff format --check cleanly. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Runs on every push to main and PR targeting main: - Lint: ruff check on src/ (not playground) - Format: ruff format --check on src/ - Test: pytest on tests/ - Build: verify package installs and imports cleanly All three jobs run in parallel for faster feedback. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Changed pull-requests permission from read to write so the review workflow can actually post review comments on PRs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The claude-code-action requires github_token to be explicitly passed for the app token exchange to work correctly during OIDC validation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
1 participant
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.
Summary
[training],[viz]). Core now includes only what's needed for Operator + Reconciler + Aggregator + QA.import gsw_memoryno longer crashes when evaluation deps are missing.src/now passesruff checkandruff format --checkcleanly.Dependency changes
Core (always installed): openai, bespokelabs-curator, faiss-cpu, langchain-voyageai, together, scikit-learn, pandas, pyarrow, pydantic, rank-bm25, numpy, python-dotenv, pyyaml, rich, tqdm
Moved to
[training]extra: torch, vllm, unsloth, flash-attn, peft, trl, wandb, hydra-core, omegaconfMoved to
[viz]extra: plotly, streamlit, ipykernel, ipywidgetsRemoved entirely: google-genai, hdbscan, openpipe-art, editables, hatchling (not runtime deps)
Test plan
uv syncinstalls cleanly with core deps onlyimport gsw_memoryworks without optional depsuv sync(full env)Generated with Claude Code