Skip to content

Add correlate1d 1#1981

Open
goldigd05 wants to merge 5 commits intopymc-devs:v3from
goldigd05:add-correlate1d-1
Open

Add correlate1d 1#1981
goldigd05 wants to merge 5 commits intopymc-devs:v3from
goldigd05:add-correlate1d-1

Conversation

@goldigd05
Copy link
Copy Markdown

Description

Related Issue

  • Closes #
  • Related to #

Checklist

Type of change

  • New feature / enhancement
  • Bug fix
  • Documentation
  • Maintenance
  • Other (please specify):

@goldigd05
Copy link
Copy Markdown
Author

Fixes #1321

Changes

  • Added correlate1d function using convolve1d with flipped input

Type of change

  • New feature / enhancement

@goldigd05
Copy link
Copy Markdown
Author

@pre-commit-ci autofix

@jessegrabowski
Copy link
Copy Markdown
Member

Do you have a use-case in mind for correlate? I'm inclined to not merge this because of how light the wrapper is. If you need it in a model you can just do pt.flip(pt.convolve1d(x)).

@ricardoV94
Copy link
Copy Markdown
Member

ricardoV94 commented Mar 30, 2026

I think it's just API compat, but it should be called correlate: https://numpy.org/doc/2.1/reference/generated/numpy.correlate.html.

Or from scipy correlate and correlate2d (they don't seem ho have correlate1d)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants