Will Dean

Results 623 comments of Will Dean

Can you negate the controls in the DataFrame and use `validate_data` = False? Does that work? Are you able to check that this works @louismagowan Same as #413

Based on [this](https://docs.readthedocs.io/en/stable/versions.html#version-warning), it seems like this is coming from readthedocs. There doesn't seem to be any logic for the HTML in either PyMC or Arviz codebase so it seems...

Read the docs is saying "Read the Docs Addons will be enabled by default on October 7th — Read the [full announcement in our blog](https://about.readthedocs.com/blog/2024/07/addons-by-default/) to know if your project...

Found the tab in the newer readthedocs UI. https://app.readthedocs.org/dashboard/pymc-marketing/addons/edit/ Seems like it is on which is confusing ![addons](https://github.com/user-attachments/assets/f51be8b2-efd7-4cf7-9112-a9b01e58a386)

> This seems to happen a lot with `conda` envs: https://github.com/readthedocs/readthedocs.org/issues/6627 Interesting. Can we make the required environment without conda then? I saw something about static built docs as well...

Yeah, we run the plots in the docstrings. That happens upon build

The PRs should still be created. The CI will just continue to fail. Is there a way to disable CI but not the PRs by pre-commit bot?

The pre-commit hok for `no-commit-to-branch` fails upon with runs on the main. A solution might be to Add `always_run: false` or to not run the action on the main branch...

The load logic of MMM would break as it rebuilds the model upon load. Does clv not do that?

It seems that this will also break the CLV loading as well as the CLV model is built upon loading the data. https://github.com/pymc-labs/pymc-marketing/blob/7dfa9558b307ee8618fcbc1c5b92b062c1a2dfa4/pymc_marketing/clv/models/basic.py#L168-L197 What behavior to you expect here?