Skip to content

chore: fix GCC 15 warnings/errors#180

Merged
ritchie46 merged 1 commit intopola-rs:mainfrom
Voultapher:fix-warnings
Jan 22, 2026
Merged

chore: fix GCC 15 warnings/errors#180
ritchie46 merged 1 commit intopola-rs:mainfrom
Voultapher:fix-warnings

Conversation

@Voultapher
Copy link
Contributor

No description provided.

@Voultapher
Copy link
Contributor Author

I dropped the ray dependency since it doesn't support Python 3.14, and it wasn't readily apparent where it is used. That might very well be wrong.

@ritchie46
Copy link
Member

I dropped the ray dependency since it doesn't support Python 3.14, and it wasn't readily apparent where it is used. That might very well be wrong.

It is used by dask and modin. Let's wait with python 3.14 until it is supported.

@Voultapher
Copy link
Contributor Author

It be nice to have a more minimal config that can be installed as base and additional reference DBs installed as opt-in.

@ritchie46
Copy link
Member

ritchie46 commented Jan 22, 2026

Ray is not dead, they will come with python 3.14 later. We can wait for now. A minimal version I am not opposed to that. But I am opposed to removing the dependencies. It works for all tools atm, that should not be changed.

@Voultapher Voultapher changed the title chore: Migrate to Python 3.14 and fix GCC 15 warnings/errors chore: fix GCC 15 warnings/errors Jan 22, 2026
@Voultapher
Copy link
Contributor Author

I've dropped the migrate commit.

@Voultapher
Copy link
Contributor Author

Voultapher commented Jan 22, 2026

Ray is not dead, they will come with python 3.14 later. We can wait for now. A minimal version I am not opposed to that. But I am opposed to removing the dependencies. It works for all tools atm, that should not be changed.

Yep, that was the idea. Not dropping ray, but rather avoiding the if not all DBs can be installed you can bench none situtation. But it's out-of-scope for now.

@ritchie46 ritchie46 merged commit 8975df5 into pola-rs:main Jan 22, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants