Skip to content

Bug in summing "LinOpConv"s with different indices #36

Description

@ThomasDeb

I encountered a bug when trying to sum two "LinOpConv" operators whose dimensions along which the convolution is performed do not match. I'm attaching a code snippet illustrating the problem. I don't think it is possible to rewrite this sum as a single convolution operator in the general case, so an easy fix would be to change the "plus_" method of LinOpConv accordingly (from testing isa(G,'LinOpConv') to something like isa(G,'LinOpConv') && isequal(sort(this.index), sort(G.index))). However I'm not sure I fully understand all the subcases of the "plus_" method so I'm not confortable trying to fix the problem completely myself
GlobalBioIm_bug
f.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions