fal-ai / dbt-fal

do more with dbt. dbt-fal helps you run Python alongside dbt, so you can send Slack alerts, detect anomalies and build machine learning models.
https://fal.ai/dbt-fal
Apache License 2.0
853 stars 72 forks source link

Please make fal compatible with Python 3.11 #871

Closed garcia-alex closed 1 year ago

garcia-alex commented 1 year ago

Context We would like fal to run with the rest of our pipeline which is currently on Python 3.11.

Is your feature request related to a problem? Please describe. Currently if you poetry add fal to an existing poetry virtual env that is using python 3.11, it fails:

poetry add fal    
Using version ^0.9.1 for fal

Updating dependencies
Resolving dependencies... (0.0s)

The current project's Python requirement (>=3.11,<4.0) is not compatible with some of the required packages Python requirement:
  - fal requires Python >=3.7.2,<3.11, so it will not be satisfied for Python >=3.11,<4.0

Because no versions of fal match >0.9.1,<0.10.0
 and fal (0.9.1) requires Python >=3.7.2,<3.11, fal is forbidden.
So, because pipeliner depends on fal (^0.9.1), version solving failed.

  • Check your dependencies Python requirement: The Python requirement can be specified via the `python` or `markers` properties

    For fal, a possible solution would be to set the `python` property to "<empty>"

    https://python-poetry.org/docs/dependency-specification/#python-restricted-dependencies,
    https://python-poetry.org/docs/dependency-specification/#using-environment-markers

Describe the solution you'd like I'd like fal and dbt-fal to be compatible with python 3.11 so I can poetry install it with no issues.

Describe alternatives you've considered There are lots of alternatives - e.g. simply creating a separate Python process in the pipeline that is not integrated with dbt but this is less elegant.

Additional context N/A

Is there an existing feature request for this?