Skip to content

Example discrete markov chain weather#702

Closed
juanitorduz wants to merge 6 commits into
pymc-devs:mainfrom
juanitorduz:example-discrete-markov-chain-weather
Closed

Example discrete markov chain weather#702
juanitorduz wants to merge 6 commits into
pymc-devs:mainfrom
juanitorduz:example-discrete-markov-chain-weather

Conversation

@juanitorduz

Copy link
Copy Markdown

Example of the feature added in #693

Example provided by @ricardoV94

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@juanitorduz juanitorduz self-assigned this Jul 1, 2026
@juanitorduz juanitorduz added the docs Improvements or additions to documentation label Jul 1, 2026
@juanitorduz juanitorduz requested a review from ricardoV94 July 1, 2026 09:22
@codecov-commenter

codecov-commenter commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.11%. Comparing base (86fac3c) to head (216153e).
⚠️ Report is 18 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main     #702       +/-   ##
===========================================
+ Coverage   51.60%   91.11%   +39.50%     
===========================================
  Files          73       90       +17     
  Lines        8003     8418      +415     
===========================================
+ Hits         4130     7670     +3540     
+ Misses       3873      748     -3125     

see 30 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread notebooks/discrete_markov_chain_marginalization.ipynb
Comment thread notebooks/discrete_markov_chain_marginalization.ipynb
@ricardoV94

ricardoV94 commented Jul 1, 2026

Copy link
Copy Markdown
Member

Looks pretty good, left just a suggestion to make it more atemporal

@juanitorduz

Copy link
Copy Markdown
Author

Thanks @ricardoV94 , I removed "old" and "new" mentions. I hope I did not miss anything

@juanitorduz

Copy link
Copy Markdown
Author

sorry @ricardoV94 I had forgot to push the "now" removals

@@ -0,0 +1,1223 @@
{

@jessegrabowski jessegrabowski Jul 1, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Add author + watermark


Reply via ReviewNB

@jessegrabowski

Copy link
Copy Markdown
Member

Looks great, just needs provenance.

I would consider using a PyMC model to generate the data (from a prior draw) then use pm.observe to estimate it, since that's the workflow we want to promote. Not a deal-breaker for me but I'd like to stop suggesting users need to write their models twice, once in numpy then again in pymc.

@maresb

maresb commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Would be cool to use real weather data!

@ricardoV94

Copy link
Copy Markdown
Member

Would be cool to use real weather data!

Do you have some?

@juanitorduz this should be in pymc-examples, sorry if it wasn't clear. Notebooks here are invisible

@juanitorduz

Copy link
Copy Markdown
Author

Would be cool to use real weather data!

Do you have some?

@juanitorduz this should be in pymc-examples, sorry if it wasn't clear. Notebooks here are invisible

ok (for now I will keep synthetic data). Do we wanna have a copy here ? I will open a PR on pymc-examples 👍

@ricardoV94

Copy link
Copy Markdown
Member

nah no copy here

@juanitorduz

Copy link
Copy Markdown
Author

216153e

Added in 216153e

@ricardoV94

Copy link
Copy Markdown
Member

@juanitorduz this looks good, wanna move it to pymc-examples?

@juanitorduz

Copy link
Copy Markdown
Author

@juanitorduz this looks good, wanna move it to pymc-examples?

yes! I will so it sometime this week 🙏

@juanitorduz

Copy link
Copy Markdown
Author

Closing in favor of pymc-devs/pymc-examples#889

@juanitorduz juanitorduz closed this Jul 7, 2026
@juanitorduz juanitorduz deleted the example-discrete-markov-chain-weather branch July 7, 2026 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants