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

Fix train_lenet_decolle_fa.py import issue #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ZhenmingYu
Copy link

Was importing standard decolle rather than decolle_fa modules.

@clee1994
Copy link

clee1994 commented Mar 9, 2022

Running this gives:

File ".local/lib/python3.7/site-packages/decolle/lenet_decolle_model_fa.py", line 109, in build_conv_stack
    readout = nn.FALinear(int(feature_height * feature_width * Nhid[i + 1]), out_channels)
AttributeError: module 'torch.nn' has no attribute 'FALinear'

I removed the "nn." in decolle/lenet_decolle_model_fa.py line 109 and it works.

@eneftci
Copy link
Member

eneftci commented Mar 9, 2022

Hi Clemens, I recommend using the branch update_21014. Do you get the same problem? I'll merge this into master soon.

@clee1994
Copy link

clee1994 commented Mar 9, 2022

Hey Emre, the branch you mentioned also has the same error, e.g. importing non-FA modules

from decolle.lenet_decolle_model import LenetDECOLLE, DECOLLELoss, LIFLayerVariableTau, LIFLayer

and using nn.FALinear

readout = nn.FALinear(int(feature_height * feature_width * Nhid[i + 1]), out_channels)

Also, I am trying to rerun the numbers from the decolle paper (e.g. DVS gesture 4.46% error) the paper mentions a hyperparameter search, but I am not sure if that matches the hyperparameters in

https://github.com/nmi-lab/decolle-public/blob/master/scripts/parameters/params_dvsgestures_torchneuromorphic.yml

(for example, the paper says burning 50ms and this config says 100 etc).

Any chance to get the hyperparameters used in the paper, or maybe even some pre-trained weights?

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

Successfully merging this pull request may close these issues.

3 participants