-
Notifications
You must be signed in to change notification settings - Fork 115
Enhance functionality of IXMP4Backend
#601
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
Changes from all commits
4f2cdf8
b3b15b4
65a0ffd
cb8239d
69a281d
7683b25
cad1bb2
8b0cdfd
4c703a6
b5bbc00
cae0795
2b90397
11a4519
11abee2
990646d
704ae63
6fa0e95
0c62c27
b704722
0b7905c
78686d0
2f76c78
aef2b79
26b4515
c5d8120
aa1c222
70cbf87
6fa9fac
0ae357f
2b9ce39
8673e83
081446a
8622b0b
8a37446
448798c
326fe89
37289e9
700a4b1
82f8885
19b346c
8cd5c51
9c0d51b
234d295
d61608b
599f20c
609fda4
08c6ee9
00d2867
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -81,13 +81,28 @@ jobs: | |
| version: ${{ matrix.gams-version }} | ||
| license: ${{ secrets.GAMS_LICENSE }} | ||
|
|
||
| - uses: ikalnytskyi/action-setup-postgres@v8 | ||
| with: | ||
| # username: postgres | ||
| # password: postgres | ||
| database: ixmp_test | ||
| # port: 5432 | ||
| postgres-version: "16" | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are there limitations on which versions work?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. https://github.com/ikalnytskyi/action-setup-postgres provides versions 14 through 18, but ixmp4 is only tested on 15 and 16. We should probably document/link where to find this information somewhere. |
||
| # ssl: true | ||
| id: setup-postgres | ||
|
|
||
| - name: Set RETICULATE_PYTHON | ||
| # Retrieve the Python executable set up above | ||
| run: echo "RETICULATE_PYTHON=$(uv python find)" >> $GITHUB_ENV | ||
|
|
||
| - name: Install the package and dependencies | ||
| # TEMPORARY Use branch for https://github.com/iiasa/ixmp4/pull/208 | ||
| # [docs] → [tests] → [ixmp4,report,tutorial] | ||
| run: uv pip install .[docs] | ||
| run: | | ||
| uv pip install \ | ||
| "ixmp4 @ git+https://github.com/iiasa/ixmp4@enh/fixes-for-ixmp-support-september-2025; python_version > '3.9'" \ | ||
| .[docs] | ||
| shell: bash | ||
|
|
||
| - name: "Install libpng-dev" # for R 'png', required by reticulate | ||
| if: startsWith(matrix.os, 'ubuntu-') | ||
|
|
@@ -104,6 +119,13 @@ jobs: | |
| IRkernel::installspec() | ||
| shell: Rscript {0} | ||
|
|
||
| - name: Setup template & default-ixmp4-local Postgres schema | ||
| if: matrix.python-version != '3.9' | ||
| run: | | ||
| ixmp4 platforms add --dsn "postgresql+psycopg://postgres:postgres@localhost:5432/template1" template1 | ||
| ixmp4 platforms add --dsn "postgresql+psycopg://postgres:postgres@localhost:5432/ixmp_test" ixmp_test | ||
| IXMP4_MANAGED=false ixmp4 platforms upgrade | ||
|
|
||
| - name: Run tests | ||
| run: | | ||
| pytest ixmp \ | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note to self: clean up Python 3.9 safeguards from this file before merging.