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

No module named 'lie_learn.representations.SO3.irrep_bases' #56

Open
EricPengShuai opened this issue Aug 8, 2021 · 4 comments
Open

Comments

@EricPengShuai
Copy link

EricPengShuai commented Aug 8, 2021

@mariogeiger In your provided MNIST example, running S2CNN encounters the following problem about lie_learn module.

Traceback (most recent call last):
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "C:\Users\peng\anaconda3\envs\pytorch_1.8\lib\site-packages\lie_learn\representations\SO3\wigner_d.py", line 5, in <module>
    from lie_learn.representations.SO3.irrep_bases import change_of_basis_matrix
ModuleNotFoundError: No module named 'lie_learn.representations.SO3.irrep_bases'

Originally posted by @EricPengShuai in #52 (comment)

@mariogeiger
Copy link
Collaborator

Hi,
I'm sorry because this repository is not maintained since a long time. I have the plan to reproduce the architecture and experiments of this repository within the e3nn library that I maintain. The only missing component is the SO3 Fourier transform.

@EricPengShuai
Copy link
Author

EricPengShuai commented Mar 9, 2022

@mariogeiger Sorry to bother you, can I implement spherical CNNs (s2cnn) with the help of e3cnn? Or do you have time to rewrite and maintain this repository?
Thanks in advance!

@mariogeiger
Copy link
Collaborator

mariogeiger commented Mar 9, 2022

Here is an implementation using e3nn:
https://github.com/e3nn/e3nn/tree/main/examples/s2cnn/mnist

There still might be an issue with this code, I remember it was not training properly last time I tried.

The fast Fourier transform is still missing for the so3 transformation, here is the current naive implementation it is using: https://github.com/e3nn/e3nn/blob/main/e3nn/o3/_so3grid.py

@nhey
Copy link

nhey commented Apr 16, 2022

@mariogeiger Thanks for the e3nn implementation. The test accuracy is about 40% for me after 20 epochs, do you have any ideas as to what the issue might be?

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