Skip to content

MultiViewICA #500

Open
Open

Description

@hugorichard

I have developped recently a new method called MultiViewICA.
https://proceedings.neurips.cc/paper/2020/file/de03beffeed9da5f3639a621bcab5dd4-Paper.pdf
It does the same thing as SRM but yields significantly better results.
I currently have a standalone implementation of it here:
https://github.com/hugorichard/multiviewica
In practice, I use SRM to reduce the data and MultiViewICA is applied on the reduced data.
I believe it would be nice to have it directly in Brainiak so that users can use it more easily.
If you think this is a good idea, I'll make a pull request.

Activity

  1. mihaic commented on Apr 23, 2021

    @mihaic
    Member

    Hello, @hugorichard! I think that's a great idea. Aside from python-picard, which looks fine, I see no new dependencies.

    Let's hear also from our SRM team. @TuKo, @qihongl?

  2. hugorichard commented on Apr 23, 2021

    @hugorichard
    ContributorAuthor

    Actually I would like to make https://github.com/hugorichard/multiviewica
    a dependency as other people are already relying on this code. Would that be fine with you ?

  3. mihaic commented on Apr 23, 2021

    @mihaic
    Member

    In that case, what would the BrainIAK contribution be?

  4. hugorichard commented on Apr 23, 2021

    @hugorichard
    ContributorAuthor

    This integration of SRM and MultiViewICA

  5. hugorichard commented on Apr 23, 2021

    @hugorichard
    ContributorAuthor

    so that people can just do MultiViewICA.fit() and it performs first SRM and then MultiViewICA on reduced data.

  6. mihaic commented on Apr 23, 2021

    @mihaic
    Member

    I see. It seems to me less compelling than incorporating MultiViewICA, but let's hear from others as well...

  7. lcnature commented on May 3, 2021

    @lcnature
    Contributor

    Hi @hugorichard. Thanks for thinking of contributing to BrainIAK! Since the major code is not going to come into BrainIAK, how do you think about making a new example notebook in funcalign folder that shows how to use SRM and MultiViewICA together?

  8. hugorichard commented on May 3, 2021

    @hugorichard
    ContributorAuthor

    Hi @lcnature ! I think that is a great idea. If you think that's interesting, I'll work on that.

  9. mihaic commented on May 5, 2021

    @mihaic
    Member

    Yes, an example would be great!

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