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

Is there any probability to develop syops for spikingjelly>=0.0.0.13? Thanks! #6

Open
AndyCao1125 opened this issue Sep 5, 2023 · 6 comments

Comments

@AndyCao1125
Copy link

No description provided.

@iCGY96
Copy link
Owner

iCGY96 commented Sep 5, 2023

We are working on upgrading the syops-counter to be compatible with spikingjelly>=0.0.0.13. We appreciate your patience and interest in our project.

@NneurotransmitterR
Copy link

Hi! The README says SyOPs requires SpikingJelly <= 0.0.0.0.12, I notice that in SpikingJelly <= 0.0.0.0.12 spikingjelly.activation_based is called spikingjelly.clock_driven but in the example spikingjelly.activation_based is used. I think that is somewhat confusing and I hope you can clarify it. Thanks.

@Taotaonoangry
Copy link

Hi! The README says SyOPs requires SpikingJelly <= 0.0.0.0.12, I notice that in SpikingJelly <= 0.0.0.0.12 spikingjelly.activation_based is called spikingjelly.clock_driven but in the example spikingjelly.activation_based is used. I think that is somewhat confusing and I hope you can clarify it. Thanks.

Same question, have you solved yet?

@NneurotransmitterR
Copy link

I just used spikingjelly.clock_driven instead. In their source code:

try:
    from spikingjelly.clock_driven.neuron import MultiStepIFNode, MultiStepLIFNode, IFNode, LIFNode, MultiStepParametricLIFNode, ParametricLIFNode
except:
    from spikingjelly.activation_based.neuron import MultiStepIFNode, MultiStepLIFNode, IFNode, LIFNode, MultiStepParametricLIFNode, ParametricLIFNode

I think the developers were to make SyOPs compatible with higher versions of SpikingJelly but for some reason they haven't implemented it. In SpikingJelly >= 0.0.0.0.14 the multi-step node classes don't exist any more, so SyOPs must be rewritten to be compatible with SJ's multi-step mode.

@titizheng
Copy link

I just used spikingjelly.clock_driven instead. In their source code:

try:
    from spikingjelly.clock_driven.neuron import MultiStepIFNode, MultiStepLIFNode, IFNode, LIFNode, MultiStepParametricLIFNode, ParametricLIFNode
except:
    from spikingjelly.activation_based.neuron import MultiStepIFNode, MultiStepLIFNode, IFNode, LIFNode, MultiStepParametricLIFNode, ParametricLIFNode

I think the developers were to make SyOPs compatible with higher versions of SpikingJelly but for some reason they haven't implemented it. In SpikingJelly >= 0.0.0.0.14 the multi-step node classes don't exist any more, so SyOPs must be rewritten to be compatible with SJ's multi-step mode.

same issue, have you solve it?

@NneurotransmitterR
Copy link

I am still planning on it; for simple demos you can just revert to SpikingJelly<=0.0.0.0.12 in your project.

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

5 participants