这是indexloc提供的服务，不要输入任何密码
Skip to content

[CI] Enable Python tests for github actions #143

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 110 commits into from
Dec 14, 2023
Merged

[CI] Enable Python tests for github actions #143

merged 110 commits into from
Dec 14, 2023

Conversation

@LeonhardFS
Copy link
Contributor

@LeonhardFS LeonhardFS commented Nov 18, 2023 •
edited
Loading

Enable per default all of Tuplex python tests in CI, to make sure wheels are functional.

Details:

  • Add new script to build wheels on macOS until ARM runners are available on Github incl. tests
  • Set timeout to 180s for Github actions to avoid stalling tests.
  • Fix protobuf issue when using -DBUILD_WITH_ORC where the orc libraries may have been build with different protobuf version than Tuplex causing issues when delocating the wheel.

Leonhard Spiegelberg and others added 30 commits October 14, 2023 17:25
enable apple silicon builds
4f4eab5
restrict Apple Silicon builds to Python3.9+
549a6a7
hw matrix setup
72a907f
print status
509406e
matrix magic
1a096f4
yaml fix
7b2239b
@LeonhardFS
test
c63d3db
@LeonhardFS
test
d2861b7
@LeonhardFS
use macos12 for delocate
ab572b8
@LeonhardFS
fix
2f34623
Merge branch 'master' into ls-apple-silicon
10d6c87
Merge branch 'ls-apple-silicon' of github.com:LeonhardFS/tuplex-publi…
b7ba825 
…c into ls-apple-silicon
update scripts for arm64 wheels
3b07d84
experimental transitioning to vcpkg
d981cc1
llvm 17 can be added
1b4b344
vcpkg support step by step
3496cee
wip llvm17 compatibility, remove typed pointers
15c26f5
remove old functions
28a65a0
merge in master
1318a6f
llvm17 fixes, remove old code
00d56e0
fixes
767b311
more API fixes for llvm17
19662ab
more fixing
13ca03a
compile now works, need to link with llvm libs
273b1ae
llvm and cmake tune
728346b
linking and warnings
a3b9586
everything builds now, but tests fail
5af6370
fixes and export symbols in runtime shared object
22fd6d9
compile fixes, change lllvm version to 15 to have typed pointers
356d1a1
another fix
1dd7da8
LeonhardFS and others added 26 commits December 4, 2023 18:26
@LeonhardFS
serializer fixes
117d583
@LeonhardFS
Merge branch 'ls-cibw-test' of github.com:LeonhardFS/tuplex-public in…
c0c4e83 
…to ls-cibw-test
@LeonhardFS
work for serializing lists
3f9f6c0
@LeonhardFS
more serialization fixes
7887bca
wheel build
19b7b69
workflow update
94393a3
@LeonhardFS
new test, fails right now. need to fix
f37ac07
@LeonhardFS
Merge branch 'ls-cibw-test' of github.com:LeonhardFS/tuplex-public in…
3805bdb 
…to ls-cibw-test
@LeonhardFS
lower n for faster testing on CI
8ce7fde
remove backward from build, and fix offset bug in deserializer
edd7344
update workflow + linux test file
75c48ac
fix bug with list serialize
93e642f
another test fix for option serialization/encoding
550fdbc
allow parallel notebook run
fedae6c
@LeonhardFS
fix
5c86ed0
@LeonhardFS
dir create fix
eb7aedf
cache brew dependencies
06a93ab
llvm downgrade
67d10cb
use backward to analyze ci failure
b9eba7f
build target
925d393
changes for zstd
59ae28c
add linking step for cache
3e1c154
update scripts to be lazy for macos
1b65f6c
refactor pybind conversion function
0cec731
reduce N in combined exception handling to avoid timeout of 180s
c9b25e9
@LeonhardFS
abseil link for github cache
1fc097e
LeonhardFS
LeonhardFS commented Dec 13, 2023
View reviewed changes
Copy link
Contributor Author

@LeonhardFS LeonhardFS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@LeonhardFS LeonhardFS merged commit 1f3210d into tuplex:master Dec 14, 2023
@LeonhardFS LeonhardFS deleted the ls-cibw-test branch September 30, 2025 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@LeonhardFS