Solving bug on optimization plot

[cetagostini]

Description

Running optimization notebook again and solving bug.

Related Issue

• Closes #
• Related to #

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

Modules affected

• MMM
• CLV

Type of change

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

---

📚 Documentation preview 📚: https://pymc-marketing--1174.org.readthedocs.build/en/1174/

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[review-notebook-app]

View / edit / reply to this conversation on ReviewNB

juanitorduz commented on 2024-11-06T15:44:14Z
----------------------------------------------------------------

Line #1.    print(np.sum(list(mmm.optimal_allocation_dict.values())))

Can you please comment what is this cell for?

[review-notebook-app]

View / edit / reply to this conversation on ReviewNB

juanitorduz commented on 2024-11-06T15:44:15Z
----------------------------------------------------------------

Can you maybe make these plots less "tall"?

[juanitorduz]

Thanks @cetagostini ! Just a couple of small suggestions

[cetagostini]

Done @juanitorduz modify the cell to have a more explicit print and make the plot less tall.

[juanitorduz]

Thanks @cetagostini !