Skip to content
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

feat: Adapt skore to be compliant on python >=3.9, <3.13 #471

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

thomass-dev
Copy link
Collaborator

@thomass-dev thomass-dev commented Oct 10, 2024

  • Remove requirements files
  • Remove use of requirements in Makefile and CI
  • Add support of python 3.9 and 3.10
  • Fix code and tests

@thomass-dev thomass-dev linked an issue Oct 10, 2024 that may be closed by this pull request
@thomass-dev thomass-dev marked this pull request as draft October 10, 2024 09:50
@thomass-dev thomass-dev force-pushed the 390-adapt-skore-to-be-compliant-on-python-39 branch 3 times, most recently from 08d7551 to 64a499a Compare October 10, 2024 12:32
@thomass-dev thomass-dev changed the title Adapt skore to be compliant on python >=3.9 Adapt skore to be compliant on python >=3.9, <3.14 Oct 10, 2024
@thomass-dev thomass-dev changed the title Adapt skore to be compliant on python >=3.9, <3.14 Adapt skore to be compliant on python>=3.9,<3.14 Oct 10, 2024
@thomass-dev thomass-dev changed the title Adapt skore to be compliant on python>=3.9,<3.14 Adapt skore to be compliant on python in [3.9, 3.14[ Oct 10, 2024
@thomass-dev thomass-dev changed the title Adapt skore to be compliant on python in [3.9, 3.14[ feat: Adapt skore to be compliant on python in python >=3.9, <3.14 Oct 10, 2024
@thomass-dev thomass-dev changed the title feat: Adapt skore to be compliant on python in python >=3.9, <3.14 feat: Adapt skore to be compliant on python in python >=3.9, <3.13 Oct 10, 2024
@thomass-dev thomass-dev changed the title feat: Adapt skore to be compliant on python in python >=3.9, <3.13 feat: Adapt skore to be compliant on python >=3.9, <3.13 Oct 10, 2024
@thomass-dev thomass-dev force-pushed the 390-adapt-skore-to-be-compliant-on-python-39 branch from 64a499a to 0eac73c Compare October 10, 2024 12:43
@thomass-dev thomass-dev marked this pull request as ready for review October 10, 2024 12:47
@tuscland
Copy link
Member

How hard is it to support 3.13?

skore/src/skore/cli/cli.py Outdated Show resolved Hide resolved
@thomass-dev thomass-dev marked this pull request as draft October 10, 2024 13:17
@thomass-dev thomass-dev force-pushed the 390-adapt-skore-to-be-compliant-on-python-39 branch 3 times, most recently from c82f211 to c8d8aed Compare October 10, 2024 13:35
@thomass-dev thomass-dev changed the title feat: Adapt skore to be compliant on python >=3.9, <3.13 feat: Adapt skore to be compliant on python >=3.9, <3.14 Oct 10, 2024
@thomass-dev thomass-dev force-pushed the 390-adapt-skore-to-be-compliant-on-python-39 branch 2 times, most recently from 7b6c833 to 2c7dfe7 Compare October 10, 2024 14:00
@thomass-dev
Copy link
Collaborator Author

thomass-dev commented Oct 10, 2024

How hard is it to support 3.13?

Vega is not compatible vega/altair#3620 vega/altair#3587 😢 .

@thomass-dev thomass-dev changed the title feat: Adapt skore to be compliant on python >=3.9, <3.14 feat: Adapt skore to be compliant on python >=3.9, <3.13 Oct 10, 2024
@thomass-dev thomass-dev force-pushed the 390-adapt-skore-to-be-compliant-on-python-39 branch 3 times, most recently from d1e0abc to 7ed4d4a Compare October 10, 2024 14:20
@thomass-dev thomass-dev force-pushed the 390-adapt-skore-to-be-compliant-on-python-39 branch from 7ed4d4a to 22f26cc Compare October 10, 2024 14:21
@thomass-dev thomass-dev marked this pull request as ready for review October 10, 2024 14:40
Copy link
Contributor

@rouk1 rouk1 left a comment

Choose a reason for hiding this comment

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

Wonderful job

@rouk1 rouk1 merged commit 4fe2b30 into main Oct 10, 2024
10 checks passed
@rouk1 rouk1 deleted the 390-adapt-skore-to-be-compliant-on-python-39 branch October 10, 2024 15:08
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.

Adapt skore to be compliant on python >=3.9
3 participants