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

Use defaults to find parameter_names #1518

Open
Christian-B opened this issue Dec 18, 2024 · 1 comment
Open

Use defaults to find parameter_names #1518

Christian-B opened this issue Dec 18, 2024 · 1 comment

Comments

@Christian-B
Copy link
Member

As we are moving to using spynnaker/pyNN/models/defaults.py for all models

Why not have that class also discover the data needed for
AbstractHasParameterNames.get_parameter_names

This will avoid issue like in

Where some params are included and others are not

@Christian-B Christian-B changed the title Use defaults to find __PARAM_NAMES Use defaults to find parameter_names Dec 18, 2024
@Christian-B
Copy link
Member Author

PyNN specification divide the init values init default_parameters and default_initial_values (State variables)

These are set by the
@default_parameters and or @default_initial_values

If one is used the reset of the init defaults become the other.
If both are specified the rest become neither

there are two ways that the methods to fill these can be called.

@defaults tag on the class
implementing default_parameters and default_initial_values to call get_map_from_init as AbstractPyNNModel does

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

1 participant