lava-nc / lava

A Software Framework for Neuromorphic Computing
https://lava-nc.org
Other
529 stars 136 forks source link

Tag consistency for NcProcessModels #805

Open ngr-francesco opened 8 months ago

ngr-francesco commented 8 months ago

User story

I am working on the Brian2Lava project as part of the INRC. In order for our pipeline to work as intended it would be optimal if the @tag for all NcProcessModels was consistent. Instead, in some cases the tag is 'ucoded' (in the case of LIF) while in some others it is 'ucode' (in the case of Dense). Having a consistent tag would make our life easier since we wouldn't have to rely on:

PhilippPlank commented 7 months ago

Thank you for raising this issue. This house keeping is of course necessary, but probably not of immediate priority from our side. If you would like to change it, feel free to submit a PR :)

jlubo commented 1 month ago

Hi, as a simple fix for this, it should suffice to add the "ucoded" tag to all process models in "lava/proc/dense/ncmodels.py".