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.

Sign up for GitHub

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

Jump to bottom

Include distance range edges in regoperator #38

Closed
stestoll opened this issue Oct 3, 2020 · 0 comments
Closed

Include distance range edges in regoperator #38

stestoll opened this issue Oct 3, 2020 · 0 comments
Assignees
@luisfabib
Labels
bug Something isn't working
Milestone
0.12.0

Comments

@stestoll
Copy link
Collaborator

stestoll commented Oct 3, 2020

Currently, the L matrix returned by regoperator for an N-element distance vector returns an N-2 by N matrix. This ignores the distance range edges and leads to edge artifacts, because P smoothness is not imposed at the edges. But it should. Therefore, the L matrix needs to be N by N, with [-2,1,0…] in the first row for ghe second-order difference case etc.
@stestoll stestoll added the bug Something isn't working label Oct 3, 2020
@stestoll stestoll added this to the 0.12.0 milestone Oct 3, 2020
luisfabib added a commit that referenced this issue Oct 5, 2020
@luisfabib
regoperator: include distance range edges (#38)
d983f6d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@luisfabib luisfabib

Labels
bug Something isn't working
Projects
None yet
Milestone
0.12.0
Development

No branches or pull requests
2 participants
@stestoll @luisfabib