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

Add python3.12 #93

Open
peterdudfield opened this issue Mar 26, 2024 · 6 comments
Open

Add python3.12 #93

peterdudfield opened this issue Mar 26, 2024 · 6 comments
Labels
good first issue Good for newcomers

Comments

@peterdudfield
Copy link
Contributor

Detailed Description

Would be good if this ran for python3.12

Possible Implementation

  • run examples locally on python 3.12
  • run tests locally
  • Add to python 3.12 to CI
@peterdudfield peterdudfield added the good first issue Good for newcomers label Mar 27, 2024
@rahul-maurya11b
Copy link

@peterdudfield sir, can i take this one ?

@OyewoleRasheed
Copy link

@peterdudfield sir, I want to work on this. I'll start by running the project's examples locally using Python 3.12 to ensure compatibility and identify any potential issues. I will be glad to get the opportunity to work on this project. I'm an Electrical Engineer passionate about climate change and I am currently working with python as I am currently taking a machine learning course in my school and also learning data science with python on World Quant University. This project is something I want to contribute to and also learn more from. Thank you sir

@suleman1412
Copy link

I'd like to tackle this issue. After attempting to run the test in Python 3.12, it appears that there's an issue with compatibility. The test didn't execute as expected due to pv-site-prediction package not being supported in Python 3.12.
Any guidance or assistance would be greatly appreciated.

@shlok191
Copy link

shlok191 commented Mar 31, 2024

@peterdudfield, I think I understand what is going wrong here! :)
There are dependencies stemming from pv-site-prediction that are bound to python versions less than 3.12.

I tried to update the dependency versioning in the pyproject.toml, but unfortunately some dependencies' latest packages still do not support 3.12 yet! I've worked quite a bit on CI/CD, and I'd be happy to add 3.12 tests to the CI. It should work eventually when the dependencies catch up, but it might show you some failures in the shorter timespan.

Should I go ahead and add tests for 3.12 to CI?

@zakwatts
Copy link
Contributor

zakwatts commented Apr 2, 2024

Hi @shlok191! Thanks for pointing this out. You are correct in that currently pv-site-prediction is bound by:

[tool.poetry.dependencies] python = "^3.10,<3.12"
which would exclude version 3.12.

Can you find out which of the dependencies dont support 3.12 yet? Before we proceed with adding the 3.12 tests, let's take a step back and review our current dependencies to pinpoint which ones specifically lack support for Python 3.12. :)

@shlok191
Copy link

shlok191 commented Apr 2, 2024

@zakwatts, absolutely!

It is the pandas-stubs library that does not support python 3.12 yet. There could potentially be more (the dependency check could just be failing at the first dependency error), but from what I see it is this library only.

Here is the official error:
pandas-stubs requires Python >=3.8,<3.12, so it will not be satisfied for Python >=3.12,<4.0

I really like what you all are working on, and I would love to help out more! Let me know if there's anything else for me to possibly help out with :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

6 participants