Clean up marimo progress bar

[fonnesbeck]

Description

Fix progbar typo: extra "s" was being added to "draws"

Checklist

• Checked that the pre-commit linting/style checks pass
• Included tests that prove the fix is effective or that the new feature works
• Added necessary documentation (docstrings and/or example notebooks)
• If you are a pro: each commit corresponds to a relevant logical change

Type of change

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

[read-the-docs-community]

Documentation build overview

📚 pymc | 🛠️ Build #31764733 | 📁 Comparing 0c37e55 against latest (f89e6c8)

🔍 Preview build

Show files changed (1 files in total): 📝 1 modified | ➕ 0 added | ➖ 0 deleted

File	Status
glossary.html	📝 modified

[jessegrabowski]

I think the actual problem is that the code assumes self.step_name will be singular, but here it is defined as plural. What if we just remove the s there?

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.49%. Comparing base (f89e6c8) to head (0c37e55).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8184   +/-   ##
=======================================
  Coverage   91.49%   91.49%           
=======================================
  Files         124      124           
  Lines       19868    19868           
=======================================
  Hits        18179    18179           
  Misses       1689     1689           

Files with missing lines	Coverage Δ
pymc/progress_bar/progress.py	97.29% <100.00%> (-0.04%)	⬇️
pymc/progress_bar/rich_progress.py	98.90% <100.00%> (+0.02%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[fonnesbeck]

@jessegrabowski as long as we are comfortable with the header in the sampler stats table using "draw" instead of "draws" as well.

[jessegrabowski]

@jessegrabowski as long as we are comfortable with the header in the sampler stats table using "draw" instead of "draws" as well.

We could have a standard for that as well? step_name stores the stem, so we add +ing for the name, +s when needed, or whatever else?

[ricardoV94]

tuneing and tunes? I think it's too cute.

If there's a need for two names, just specify the two? We have just two cases now anyway (mcmc/smc) so not any meaningful amount of work

[fonnesbeck]

I think "draw" and "stage" are currently the only values used for action. Is there anything other than MCMC and SMC that uses this particular progbar?

[ricardoV94]

I think "draw" and "stage" are currently the only values used for action. Is there anything other than MCMC and SMC that uses this particular progbar?

No

[fonnesbeck]

Shape errors in DEMetropolis can't be related to these changes.

[ricardoV94]

Have to cherry pick some patches from main to v6, give me a sec

[ricardoV94]

@fonnesbeck do you want this to go to main for a patch-release? If so change the target of the branch

[fonnesbeck]

Probably should, yes.

[ricardoV94]

@fonnesbeck I changed the branch and rebased