Skip to content

gaussian grbm initialization#71

Open
jquetzalcoatl wants to merge 13 commits intodwavesystems:mainfrom
jquetzalcoatl:feature/gaussian-rbm-init
Open

gaussian grbm initialization#71
jquetzalcoatl wants to merge 13 commits intodwavesystems:mainfrom
jquetzalcoatl:feature/gaussian-rbm-init

Conversation

@jquetzalcoatl
Copy link
Copy Markdown

@jquetzalcoatl jquetzalcoatl commented Mar 16, 2026

grbm weights and biases initialization set to Gaussian N(0,1/number of nodes)

Hinton guide suggests 0.01 as standard deviation. See https://www.cs.toronto.edu/~hinton/absps/guideTR.pdf

Moreover, having it set to Gaussian with this dependence on the number of nodes makes the energy extensive and initializes the gRBM in a paramagnetic phase similar to that describen in the Random Energy model paper

https://journals.aps.org/prb/abstract/10.1103/PhysRevB.24.2613

See #48

@kevinchern
Copy link
Copy Markdown
Collaborator

@jquetzalcoatl IIRC, Hinton's recommendation pertains to zero-one-valued RBMs (bipartite with hidden units). Would it make sense to translate the $0.01$ to the spin-valued equivalent?

@jquetzalcoatl
Copy link
Copy Markdown
Author

@kevinchern The REM reference is for spin models i.e., {-1,1}. Ultimately, the initialization pertains to whether the model is ergodic. In this sense, the support only set an offset energy.

I believe the main motivation for initializing with 0.01 in Hinton's guide is to start in a paramagnetic phase, which ties nicely with the REM/SK spin glass model

Copy link
Copy Markdown
Collaborator

@kevinchern kevinchern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a release note to go with this?

jquetzalcoatl and others added 2 commits March 16, 2026 16:04
Co-authored-by: Kevin Chern <32395608+kevinchern@users.noreply.github.com>
@jquetzalcoatl
Copy link
Copy Markdown
Author

added release note

Copy link
Copy Markdown
Collaborator

@kevinchern kevinchern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Following motivation from references, should h be initialized to 0?

jquetzalcoatl and others added 5 commits March 17, 2026 10:46
Co-authored-by: Kevin Chern <32395608+kevinchern@users.noreply.github.com>
Co-authored-by: Kevin Chern <32395608+kevinchern@users.noreply.github.com>
Co-authored-by: Kevin Chern <32395608+kevinchern@users.noreply.github.com>
Co-authored-by: Kevin Chern <32395608+kevinchern@users.noreply.github.com>
Copy link
Copy Markdown
Collaborator

@kevinchern kevinchern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests are failing but otherwise LGTM. Thanks for the much-needed PR @jquetzalcoatl !!

@VolodyaCO offered to take a look at the tests

jquetzalcoatl and others added 3 commits March 17, 2026 10:59
Co-authored-by: Kevin Chern <32395608+kevinchern@users.noreply.github.com>
Co-authored-by: Kevin Chern <32395608+kevinchern@users.noreply.github.com>
Co-authored-by: Kevin Chern <32395608+kevinchern@users.noreply.github.com>
@kevinchern kevinchern requested a review from VolodyaCO March 17, 2026 18:01
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

Successfully merging this pull request may close these issues.

2 participants