Skip to content

Rename at aliases to pt #6572

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.

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

Closed
ricardoV94 opened this issue Mar 7, 2023 · 4 comments · Fixed by #6577
Closed

Rename at aliases to pt #6572

ricardoV94 opened this issue Mar 7, 2023 · 4 comments · Fixed by #6577

Comments

@ricardoV94
Copy link
Member

Description

We used to do import aesara.tensor as at, and for ease we just changed it to pytensor.tensor as at. Since we don't depend directly on any Aesara or Aesara-related library we should go with the new default pytensor.tensor as pt.

@Raj-Parekh24
Copy link
Contributor

Hey @ricardoV94,

I am currently getting acquainted with the repository, I have raised PR for fixes in documentation issue #6569. For this I have set-up the local environment.

Can I take this issue to get myself acquainted with the code base.

@shreyas3156
Copy link
Contributor

Hi, @Raj-Parekh24 I actually just made the changes for this issue and am in the process of submitting a PR.

@Raj-Parekh24
Copy link
Contributor

Ok @shreyas3156,

Thanks for the update.

@shreyas3156
Copy link
Contributor

Hi @ricardoV94, I have created a PR implementing the changes that would close this issue. It would be great if you could review it and provide a feedback as this is my first contribution to this repo.

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

Successfully merging a pull request may close this issue.

3 participants