Skip to content

feat(core): redesign Python bindings architecture for v0.5.0#16

Merged
guycipher merged 1 commit intomasterfrom
feat/python-bindings-arch
Jan 13, 2025
Merged

feat(core): redesign Python bindings architecture for v0.5.0#16
guycipher merged 1 commit intomasterfrom
feat/python-bindings-arch

Conversation

@imprvhub
Copy link
Contributor

BREAKING CHANGE: Complete restructure of Python bindings

  • Replace tidesdb.py with modular core.py and init.py structure
  • Add automated setup scripts for C foundation integration:
    • Linux/macOS: setup.sh
    • Windows: setup.ps1
  • Create samples/ directory with comprehensive operation examples:
    • basic_operations.py
    • cursor_operations.py
    • transaction_operations.py
  • Add tests/ suite validating core functionality
  • Update dependencies and documentation

Migration: Use setup.sh (Linux/macOS) or setup.ps1 (Windows) for installation and import from tidesdb package

Closes #12

BREAKING CHANGE: Complete restructure of Python bindings

- Replace tidesdb.py with modular core.py and __init__.py structure
- Add automated setup scripts for C foundation integration:
  - Linux/macOS: setup.sh
  - Windows: setup.ps1
- Create samples/ directory with comprehensive operation examples:
  - basic_operations.py
  - cursor_operations.py
  - transaction_operations.py
- Add tests/ suite validating core functionality
- Update dependencies and documentation

Migration: Use setup.sh (Linux/macOS) or setup.ps1 (Windows) for installation and import from tidesdb package

Closes #12
@imprvhub imprvhub requested a review from guycipher January 13, 2025 08:40
@imprvhub imprvhub self-assigned this Jan 13, 2025
@imprvhub imprvhub added feat request implementation documentation Improvements or additions to documentation enhancement New feature or request testing labels Jan 13, 2025
@guycipher
Copy link
Member

Looks awesome. One thing is the license headers in the files are missing.

@guycipher
Copy link
Member

Truly looks great. Awesome job @imprvhub can't wait to play with it.

@guycipher
Copy link
Member

I saw the video as well, it looks really good @imprvhub . Very easy to setup and understand. We can add onto this for sure. I will merge.

@guycipher guycipher merged commit 9a34635 into master Jan 13, 2025
@imprvhub
Copy link
Contributor Author

Looks awesome. One thing is the license headers in the files are missing.

done! #18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request feat request implementation testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Installation with pip ?

2 participants