-
Notifications
You must be signed in to change notification settings - Fork 63
refactor: Python.NET wrapper (general code modernization) #434
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
DaveSkender
merged 26 commits into
main
from
copilot/fix-94a0641c-58a2-4e76-a48a-4af852c8cb35
Dec 24, 2025
Merged
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
7461189
Initial plan
Copilot 901f55e
Refactor Python.NET wrapper with improved type conversions and error …
Copilot 82aade2
Fix unit tests and address Codacy issues - decimal precision, depreca…
Copilot f707ecf
ci fix: deprecated windows runner
DaveSkender d99df03
Fix staticmethod decorator causing CI test failures
Copilot 68b8a3e
Fix Codacy issues: improve pylint score from 9.30 to 9.43
Copilot 3cc4d10
Fix staticmethod decorator causing CI test failures (again)
Copilot f79198f
Fix Codacy issues: improve pylint score from 9.35 to 9.98
Copilot 3adfb2a
Fix staticmethod decorator causing CI test failures
Copilot a5bc464
Address code review feedback: fix imports, return types, and function…
Copilot 9ac634c
Fix Codacy issues and test failures: resolve function signatures and …
Copilot 5876dc9
address code review feedback
DaveSkender 1e04eff
Fix conditional syntax for post-summary checks in CI workflow
DaveSkender 45ad438
Merge main branch: resolve conflicts with timezone awareness PR #410
Copilot bf52b50
Merge branch 'main' into copilot/fix-94a0641c-58a2-4e76-a48a-4af852c8…
DaveSkender 4fd8c8c
fix: Add missing return type hints and remove trailing whitespace
Copilot c83183c
refactor: Improve vwap and ichimoku parameter validation and handling
Copilot e7507d2
Merge branch 'main' into copilot/fix-94a0641c-58a2-4e76-a48a-4af852c8…
DaveSkender f2cd50d
update DLL v2.7.1, .NET 8, and pythonnet 3.0.5
DaveSkender eb7f25f
use ruff, pyright for linters
DaveSkender 7e37b85
rabbit fixes
DaveSkender a741c56
fix: correct task group formatting in tasks.json
DaveSkender 3c2531c
review fixes
DaveSkender 0869d08
fix: simplify overload definitions in get_parabolic_sar
DaveSkender 181b8dd
Fix indentation for MACD quotes assignment
DaveSkender f280382
remove deprecated methods
DaveSkender File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,25 @@ | ||
| #!/bin/bash | ||
| set -e | ||
|
|
||
| # Create virtual environment if it doesn't exist | ||
| if [ ! -d ".venv" ]; then | ||
| echo "Creating virtual environment..." | ||
| python -m venv .venv | ||
| fi | ||
|
|
||
| # Activate virtual environment | ||
| source .venv/bin/activate | ||
|
|
||
| # Upgrade pip | ||
| echo "Upgrading pip..." | ||
| python -m pip install --upgrade pip | ||
|
|
||
| # Install core dependencies | ||
| echo "Installing core dependencies..." | ||
| pip install -r requirements.txt | ||
|
|
||
| # Install test dependencies | ||
| echo "Installing test dependencies..." | ||
| pip install -r requirements-test.txt | ||
|
|
||
| echo "✓ Dev container setup complete!" |
This file was deleted.
Oops, something went wrong.
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,63 @@ | ||
| name: CI | ||
|
|
||
| on: | ||
| push: | ||
| pull_request: | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| jobs: | ||
| linting: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - name: Checkout source | ||
| uses: actions/checkout@v6 | ||
|
|
||
| - name: Setup .NET | ||
| uses: actions/setup-dotnet@v5 | ||
| with: | ||
| dotnet-version: 10.x | ||
| dotnet-quality: ga | ||
|
|
||
| - name: Setup Python | ||
| uses: actions/setup-python@v6 | ||
| with: | ||
| python-version: 3.13 | ||
| cache: "pip" | ||
|
|
||
| - name: Create virtual environment | ||
| run: python -m venv .venv | ||
|
|
||
| - name: Install dependencies | ||
| run: | | ||
| source .venv/bin/activate | ||
| python -m pip install --upgrade pip | ||
| python -m pip install -e . | ||
| python -m pip install -r requirements-test.txt | ||
|
|
||
| - name: Ruff lint | ||
| run: | | ||
| source .venv/bin/activate | ||
| ruff check . | ||
|
|
||
| - name: Ruff format check | ||
| run: | | ||
| source .venv/bin/activate | ||
| ruff format --check . | ||
|
|
||
| - name: Pyright | ||
| run: | | ||
| source .venv/bin/activate | ||
| pyright | ||
|
|
||
| - name: Pytest | ||
| run: | | ||
| source .venv/bin/activate | ||
| pytest | ||
|
|
||
| - name: pip-audit | ||
| run: | | ||
| source .venv/bin/activate | ||
| pip-audit -r requirements.txt -r requirements-test.txt |
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.