Tim-Salzmann / l4casadi

Use PyTorch Models with CasADi for data-driven optimization or learning-based optimal control. Supports Acados.
MIT License
310 stars 19 forks source link

support for python 3.8 #31

Closed mandralis closed 2 months ago

mandralis commented 2 months ago

I need to run l4casadi in python 3.8 since it makes it easier to use with ROS2 foxy. Is it possible to make it backwards compatible with Python 3.8 ?

Thanks !

Tim-Salzmann commented 2 months ago

Hi Ioannis,

Thanks for reaching out. Did you check that it is indeed not compatible with 3.8 as of now?

I would think that if it is not compatible it should only be a couple of lines to change.

Let me know

Best Tim

Tim-Salzmann commented 2 months ago

I just gave it a try, and it seems to work for Python 3.8. Updates the requirements.

mandralis commented 2 months ago

Hi Tim,

Thank you very much for your response. My problem is that when I run CUDACXX=/usr/local/cuda-12/bin/nvcc pip3 install l4casadi --no-build-isolation I get an error:

ERROR: Could not find a version that satisfies the requirement l4casadi (from versions: none) ERROR: No matching distribution found for l4casadi

Best, Ioannis

On Mon, 13 May 2024 at 00:00, Tim Salzmann @.***> wrote:

I just gave it a try, and it seems to work for Python 3.8.

— Reply to this email directly, view it on GitHub https://github.com/Tim-Salzmann/l4casadi/issues/31#issuecomment-2106797452, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH3DQVGHZV4YCYQ4NP4XD2LZCBQJFAVCNFSM6AAAAABHTN3X22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWG44TONBVGI . You are receiving this because you authored the thread.Message ID: @.***>

mandralis commented 2 months ago

It worked when building from source. For some reason I have to run the following to get it to work: export TORCH_CUDA_ARCH_LIST="8.0 8.6 8.9 9.0"

On Mon, 13 May 2024 at 08:01, Ioannis Mandralis @.***> wrote:

Hi Tim,

Thank you very much for your response. My problem is that when I run CUDACXX=/usr/local/cuda-12/bin/nvcc pip3 install l4casadi --no-build-isolation I get an error:

ERROR: Could not find a version that satisfies the requirement l4casadi (from versions: none) ERROR: No matching distribution found for l4casadi

Best, Ioannis

On Mon, 13 May 2024 at 00:00, Tim Salzmann @.***> wrote:

I just gave it a try, and it seems to work for Python 3.8.

— Reply to this email directly, view it on GitHub https://github.com/Tim-Salzmann/l4casadi/issues/31#issuecomment-2106797452, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH3DQVGHZV4YCYQ4NP4XD2LZCBQJFAVCNFSM6AAAAABHTN3X22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWG44TONBVGI . You are receiving this because you authored the thread.Message ID: @.***>