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

Generalize (F)(L)ODOG normalization #3

Open
wants to merge 91 commits into
base: main
Choose a base branch
from
Open

Conversation

JorisVincent
Copy link
Contributor

@JorisVincent JorisVincent commented Mar 4, 2024

  • Documentation outlining generalization of (F)(L)ODOG normalizations
  • Refactor normalization-module, and subsequently of models-module
  • Test F,L,ODOG against MATLAB: passing
    • Automate testing using GHA, Nox

JorisVincent and others added 30 commits October 24, 2023 17:42
Also properly document `padval` default
Where possible compare components to MATLAB output; else, compare to RHS transplementation
Easier to test against subcomponents of LODOG in transplementation
... for example stimulus, replicates exactly what the MATLAB code produces

Merge branch 'dev_LODOG' into dev
extract normalizers to subroutine
reuse LODOG gaussian kernel
separate spatial masks
reuse LODOG RMS function
extract local RMS subroutine
separate read out from normalization
default spatial averaging mask is circular
rename fixture
MiaSelene and others added 25 commits February 15, 2024 17:07
Brings more in line with other doc pages
Reorganizes and removes some existing normalization
Better specification of shape for spatial normalization
Closes #3 (Gitlab)
Since it is used to prevent DivideByZero errors, makes sense to use it in divisive_normalization()
Default eps=0.0

Takes on different values in F/L/ODOG tests
feat(models): `norm_energies` method to replace `normalizers_to_RMS`
@JorisVincent JorisVincent added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request labels Mar 4, 2024
@JorisVincent JorisVincent self-assigned this Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants