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

Update import path for epyccel in Psydac to match new Pyccel structure #426

Open
jalalium opened this issue Aug 6, 2024 · 3 comments
Open
Assignees

Comments

@jalalium
Copy link
Member

jalalium commented Aug 6, 2024

Description:

It appears that a recent change in Pyccel has moved the epyccel module from /pyccel/ to /pyccel/commands. This change has caused an issue in Psydac, specifically in lines:

  • 262 in /psydac/api/basic.py
  • 418 in /psydac/api/ast/linalg.py
  • 337 in /psydac/api/glt.py
from pyccel.epyccel import epyccel

This import statement is no longer valid with the newer versions of Pyccel, and as a result, Psydac is currently not functioning properly with the latest version.

@yguclu yguclu self-assigned this Aug 12, 2024
@yguclu
Copy link
Member

yguclu commented Aug 12, 2024

Hi @jalalium, when you say "newer versions of Pyccel" are you referring to the recent commits on the devel branch?

@jalalium
Copy link
Member Author

Hi @jalalium, when you say "newer versions of Pyccel" are you referring to the recent commits on the devel branch?

This issue concerns recent commits on the devel branch.

@yguclu
Copy link
Member

yguclu commented Aug 16, 2024

Hi @jalalium, when you say "newer versions of Pyccel" are you referring to the recent commits on the devel branch?

This issue concerns recent commits on the devel branch.

All right. Then Psydac will have a problem with the future releases of Pyccel.

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

No branches or pull requests

2 participants