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

Resolve ownership of edge weight information between edge and connections classes #771

Open
stiber opened this issue Jan 22, 2025 · 0 comments
Labels
GPU refactor doesn't change functionality, just improves code undergrad-capstone

Comments

@stiber
Copy link
Contributor

stiber commented Jan 22, 2025

Right now, this is tied too deeply into the ConnGrowth class, due to historical reasons and the fact that ConnGrowth needs to update the edge weights. But it perhaps makes sense for the edges to own their weights. Moreover, for other connections classes, such as ConnStatic and STDP simulations, the edges update their own weights. Also, I think that the different apparent weight matrix data actually holds different values: one may be just area of overlap during a growth simulation, while the other has signed and scaled down actual weight values.

There is of course an intersection with the GPU implementation, and the GPU implementation needs to mirror the CPU implementation (including any modifications). This should also take into account the ability to copy such information back and forth between the host and device.

@stiber stiber added GPU refactor doesn't change functionality, just improves code undergrad-capstone labels Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GPU refactor doesn't change functionality, just improves code undergrad-capstone
Projects
None yet
Development

No branches or pull requests

1 participant