the Izhikevich_2007 example neuron model bug

[max-talanov]

Dear all.
Thank you for the great impact involved in creation of the brian2 and documentation and examples.
But I have found the bug in the example Izhikevich_2007:
The line 55 contains the formula of the Neuron:
dv/dt = (ge * (Ee-vr) + El - v) / taum : volt
While original example STDP contains the following formula:
dv/dt = (ge * (Ee-v) + El - v) / taum : volt
And the original article of Song, Miller and Abbott (2000) page 25:

That makes much more sense as ge influence is multiplied to Ee - current voltage rather than reset voltage constant.
In the scope of the Izhikevich_2007 example the bug possibly does not make much difference, but if one tries to reuse the code in the network with combination of STDP and static synapses STDP synaptic influence could be much lower than static.
Could you please fix this minor issue in the example.
Thank you so much in advance.
Dr. Max Talanov.

[DHRUVJ2003]

Hello,
I am interested in solving this issue. Can you please state how can I reproduce the issue? Will changing the formula in Izhikevich_2007 solve the issue??
I am new to open source and to brian as well.

Thanks

[max-talanov]

Hello.
Yes please update the formula.

[mstimberg]

Closed via #1485 – thanks to @max-talanov for reporting and to @DHRUVJ2003 for fixing 🙏 .

The updated example is here (will be in the "stable" version of the docs with the next release): https://brian2.readthedocs.io/en/latest/examples/frompapers.Izhikevich_2007.html

[DHRUVJ2003]

Thank you 🙏 ..... It is an honour to contribute to brian