Jonathan Pelham
Jonathan Pelham
It passed pre-commit though? And it has the docstring spacing? Or at least it passed on my pc looks like it's failing here. I'll spend some time on my setup....
Well I pulled from main, carefully applied my changes, ran pre-commit only on the four files I changed, and then pushed to my patch-2 branch. pymc\__init__.py pymc\tests\test_model_graph.py pymc\model_graph.py pymc\docs\source\api\models.rst Could...
Looks like everything passes in Github except pre-commit itself which gives an opaque error message. `Error: The process '/opt/hostedtoolcache/Python/3.10.6/x64/bin/pre-commit' failed with exit code 1`
I took a guess it might be because networkx needs to be added to the requirements so i've done that and when it re runs hopefully that clears it all...
Can we re-run the pre-commit action with debug enabled? Or does anyone have an idea what is causing it to fail?
I tried running it twice this time before committing. I push it to my path repo and am running the same tests on it there. If that all works I...
Ok all tests passed on my patch repo so It should surely pass this time here.
> I assume the goal is for the function to be used as `pymc.model_to_networkx` (which is what is being documented) which won't work yet. For this to work you need...
Right i've fixed the trailing whitespace issue and it is [passing pre-commit tests again.](https://github.com/jonititan/pymc/runs/7838895866?check_suite_focus=true)
Great. Thanks likewise for bearing with me while I learned on the go.