Skip to content

Ising - Gaussian RBM - no biases#63

Open
Francill99 wants to merge 8 commits intoDsysDML:developfrom
Francill99:develop
Open

Ising - Gaussian RBM - no biases#63
Francill99 wants to merge 8 commits intoDsysDML:developfrom
Francill99:develop

Conversation

@Francill99
Copy link
Contributor

The choice of not applying gradient to biases is due to the fact that for theoretical study we need no biases

@codecov-commenter
Copy link

codecov-commenter commented Jan 30, 2026

Codecov Report

❌ Patch coverage is 30.63063% with 154 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
rbms/ising_gaussian/implement.py 20.65% 73 Missing ⚠️
rbms/ising_gaussian/classes.py 31.18% 64 Missing ⚠️
rbms/ising_gaussian/functional.py 46.87% 17 Missing ⚠️
Files with missing lines Coverage Δ
rbms/ising_gaussian/__init__.py 100.00% <100.00%> (ø)
rbms/map_model.py 100.00% <100.00%> (ø)
rbms/ising_gaussian/functional.py 46.87% <46.87%> (ø)
rbms/ising_gaussian/classes.py 31.18% <31.18%> (ø)
rbms/ising_gaussian/implement.py 20.65% <20.65%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nbereux nbereux self-requested a review February 2, 2026 11:09
@nbereux
Copy link
Contributor

nbereux commented Feb 2, 2026

can you add an __init__.py to the folder please ?

@Francill99
Copy link
Contributor Author

I added init.py analogously to other folders, let me know if there are changes to do

Copy link
Contributor

@nbereux nbereux left a comment

Choose a reason for hiding this comment

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

Looks fine to me ! Just the typos to change and check the formula and we're good to go !

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.

4 participants