Skip to content

[model] add least squares solver (review & merge)#6

Open
ryanhammonds wants to merge 3 commits intomainfrom
lstsq
Open

[model] add least squares solver (review & merge)#6
ryanhammonds wants to merge 3 commits intomainfrom
lstsq

Conversation

@ryanhammonds
Copy link
Copy Markdown
Member

Addresses #5. I made this because I was applying this package to 2d spectrograms and needed a faster solver. Allows directly solving conv weights as a linear system, rather than using iterative optimization. This should be faster than convolutional.

Additionally:

  • a dilation kwarg was added that allows the expansion/dilation of the kernel to account for learn weights at various spatial scales, e.g. dilation=2 constructs a linear system from sliding the following kernel over an input:
Screenshot 2025-04-09 at 2 50 01 PM
  • a equidistant kwarg was added that allows the removal of weight constraints when False. This is for cases when the symmetry assumption is violated.

To-Do Before Merging

  • Tests
  • Confirm predictive value of weights
  • Ensure it's actually faster the convolution

@ryanhammonds ryanhammonds changed the title Add least squares solver Add least squares solver (review & merge) Dec 12, 2025
@ryanhammonds ryanhammonds changed the title Add least squares solver (review & merge) add least squares solver (review & merge) Dec 12, 2025
@ryanhammonds ryanhammonds changed the title add least squares solver (review & merge) [model] add least squares solver (review & merge) Dec 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant