Skip to content

Move initializers to WeightInitializers #212

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Open
MartinuzziFrancesco opened this issue Mar 8, 2024 · 2 comments · May be fixed by #213
Open

Move initializers to WeightInitializers #212

MartinuzziFrancesco opened this issue Mar 8, 2024 · 2 comments · May be fixed by #213

Comments

@MartinuzziFrancesco
Copy link
Collaborator

Following issue 21 in WeightInitializers.jl we should do something similar for the initializers that we have here
@avik-pal
Copy link
Member

avik-pal commented Mar 9, 2024

For centralized maintenance, I think it makes sense to move those initializers to WeightInitializers as well?

@MartinuzziFrancesco
Copy link
Collaborator Author

that's a good point, I'll finish dealing with issue 21 and then we can move them there

@MartinuzziFrancesco MartinuzziFrancesco linked a pull request Mar 12, 2024 that will close this issue
Move initializers to WeightInitializers.jl #213
Open
@MartinuzziFrancesco MartinuzziFrancesco changed the title Add type check and conversion in initializers Move initializers to WeightInitializers Oct 15, 2024
@MartinuzziFrancesco MartinuzziFrancesco mentioned this issue Oct 15, 2024
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move initializers to WeightInitializers.jl SciML/ReservoirComputing.jl
2 participants
@MartinuzziFrancesco @avik-pal