Skip to content

Conversation

juanitorduz
Copy link
Contributor

@juanitorduz juanitorduz commented Aug 16, 2025

Closes #1584

I made very few modifications to the original content.

It would be nice to have this one ready before the Euro Scipy talk :)


📚 Documentation preview 📚: https://pytensor--1585.org.readthedocs.build/en/1585/

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@juanitorduz juanitorduz marked this pull request as draft August 16, 2025 14:26
@juanitorduz juanitorduz self-assigned this Aug 16, 2025
@juanitorduz juanitorduz marked this pull request as ready for review August 16, 2025 18:28
@@ -0,0 +1,3311 @@
{
Copy link
Member

@jessegrabowski jessegrabowski Aug 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I recall that Ricardo had an objection to my quip about this graphic being "upside down"


Reply via ReviewNB

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any suggested changes @ricardoV94 ?

@@ -0,0 +1,3311 @@
{
Copy link
Member

@jessegrabowski jessegrabowski Aug 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This section is about logp inference, but the content is about rng seeding


Reply via ReviewNB

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the seeding part and just left a small pymc model and show where PyTensor appears. Is this ok?

@@ -0,0 +1,3311 @@
{
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the seeding part and just left a small pymc model and show where PyTensor appears. Is this ok?

@@ -0,0 +1,3311 @@
{
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any suggested changes @ricardoV94 ?

@jessegrabowski
Copy link
Member

Removed the seeding part and just left a small pymc model and show where PyTensor appears. Is this ok?

I actually think showing that we fully transform a forward graph x ~ N(0, 1 | y) to logp(x ,y) using only graph inspection is a good thing to do. It's extremely impressive, and I think most people don't even realize it's happening.

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

Successfully merging this pull request may close these issues.

Add intro notebook from workshop
2 participants