Add PyPI publishing support#48
Merged
fcostaoliveira merged 9 commits intoupdate.redisearchfrom Feb 12, 2026
Merged
Conversation
fcostaoliveira
commented
Feb 11, 2026
fcostaoliveira
commented
- Configure pyproject.toml for PyPI package 'vector-benchmark'
- Add console script entry point 'vector-db-benchmark'
- Add GitHub Actions workflow for automated PyPI publishing
- Update help text examples to use 'vector-db-benchmark' and 'redis-*'
- Add init.py files to datasets and experiments for proper packaging
- Improve download messages to show destination folder
- Add Redis Performance Team to authors
- Add package metadata: description, keywords, classifiers, homepage, etc.
- Configure pyproject.toml for PyPI package 'vector-benchmark' - Add console script entry point 'vector-db-benchmark' - Add GitHub Actions workflow for automated PyPI publishing - Update help text examples to use 'vector-db-benchmark' and 'redis-*' - Add __init__.py files to datasets and experiments for proper packaging - Improve download messages to show destination folder - Add Redis Performance Team to authors - Add package metadata: description, keywords, classifiers, homepage, etc.
Resolved conflicts: - benchmark/dataset.py: Kept improved bz2 extraction with .bz2 extension removal - run.py: Kept updated help text with vector-db-benchmark and redis-* examples
- Move run.py from packages to include list - run.py is a module file, not a package directory - Fixes Poetry error: '/code/benchmark does not contain any element'
- Update pyproject.toml to properly separate packages from data files - Move datasets and experiments from packages to include list - Specify exact files to include to avoid large dataset files - Update Dockerfile to copy source files before poetry install - Fix .dockerignore to allow README.md and LICENSE (needed by Poetry) - Fix empty continuation line warning in Dockerfile - Verified Docker build succeeds and container runs correctly
JoanFM
approved these changes
Feb 11, 2026
- Change working directory from /app to /code for consistency with Poetry .pth file - Update entrypoint script to use /code paths - Change default CMD from 'run.py' to 'vector-db-benchmark' command - Simplify entrypoint to just ensure results directory permissions - Now uses the Poetry-installed console script instead of direct Python execution
- Update GitHub Actions workflow to use vector-db-benchmark instead of run.py - Update docker-test.sh to use vector-db-benchmark command - Add --entrypoint python for Python-specific tests - Update volume mount paths from /app to /code - Ensures tests work with Poetry-installed console script
benoitdion
reviewed
Feb 11, 2026
| README.md | ||
| LICENSE | ||
| # README.md and LICENSE are needed for Poetry package metadata | ||
| !README.md |
|
what is the actual point of PR? |
- Remove original author from PyPI authors field - Update description to mention this is a fork - Keeps Redis Performance Team as the package maintainer - Original attribution preserved in LICENSE and git history per Apache 2.0
🐳 Docker Build Validation✅ Docker build successful! Platforms tested:
Git SHA: Docker Hub Status: ✅ Docker Hub credentials configured Image details:
Tests performed:
The Docker image is ready for deployment! 🚀 |
🐳 Docker Build Validation✅ Docker build successful! Platforms tested:
Git SHA: Docker Hub Status: ✅ Docker Hub credentials configured Image details:
Tests performed:
The Docker image is ready for deployment! 🚀 |
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
3 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.