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

Error cell [14] issubclass() arg 1 must be a class #6

Open
DanD364 opened this issue May 10, 2023 · 3 comments
Open

Error cell [14] issubclass() arg 1 must be a class #6

DanD364 opened this issue May 10, 2023 · 3 comments

Comments

@DanD364
Copy link

DanD364 commented May 10, 2023

Hello,

Thank you very much for this amazing work.
I am trying to understand a bit better differentiable physics for fluid dynamics and this would help me a lot.

However when I am trying to run your code there is an error on cell 14 that I was unable to solve. Maybe due to an update of HfArgumentParser? Everything ran well on the previous cells (except for a small typo on cell 5 where there were some spaces after the '%' sign but that's straight forward to fix)

Do you have an idea on how to fix it and run your code?

Thank you very much

Fluid_error
@NickGeneva
Copy link
Member

Hi @DanD364

This looks like a problem with the HfArgParser and some Python magic it uses. For Python 3.9 I have to hack around a solution to get it to function. By now its likely out of date for python 3.10, so maybe try downgrading to python 3.9?

@edwardsmith999
Copy link

Thank you for an interesting paper and making the package available.

I have a student trying to reproduce the cylinder results on Colab, but as it is now 3.10 by default it returns the above error.
To go to Python3.9 downgrading is needed. Trying the technique here:
https://stackoverflow.com/questions/68657341/how-can-i-update-google-colabs-python-version
3.8 replaced by 3.10 causes a

WARNING: The following packages were previously imported in this runtime:
      [cycler,kiwisolver]
     You must restart the runtime in order to use newly installed versions"

and a crash when "restart runtime" is clicked.

Given downgrading Python seems to be impossible, would it be possible to fix for later versions of Python?

@edwardsmith999
Copy link

Just created a pull request which seems to fix for Python 3.10

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

3 participants