Skip to content

Conversation

@juanitorduz
Copy link
Contributor

Attempts to address #51

I really want to have the linear nodes hehe. So this is a draft PR to try to port the changes added in pymc-devs/pymc#5044 . The code-base has changed a bit so I need to catch up in understanding the code more. If someone could give me some guidance it would be much appreciated. I feel this is a relatively "low-hanging-fruit" but as always, the devil is in the details.

@juanitorduz juanitorduz marked this pull request as draft March 17, 2023 14:40
@aloctavodia
Copy link
Member

Thanks for taking care of this! I am not able to provide feedback right now. But I will do it ASAP.

@juanitorduz
Copy link
Contributor Author

No rush! It will take me some time to go through the code and make sense of it which is the best approach in contrast to the brute force way hehe.

@juanitorduz juanitorduz mentioned this pull request Mar 18, 2023
@juanitorduz
Copy link
Contributor Author

I will close this one to work on a new one as I have gained some useful learnings in #76 (Also I do not feel like resolving conflicts 😅 )

@juanitorduz juanitorduz closed this Apr 3, 2023
@aloctavodia
Copy link
Member

Perfect

@juanitorduz juanitorduz deleted the issue_51_linear_fit_draft branch April 3, 2023 12:43
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.

2 participants