Amos You
Amos You
encountered this error as well. the way i fixed was 1. uninstalling ffmpeg from the conda environment and only using ffmpeg-python 2. installing ffmpeg on my system (the script runs...
@vroulet most of the classes / states in transform.py and combine.py don't have proper docstrings. would it still be worthwhile to remove :members: for those files (many of those classes...
@vroulet i removed the :members: from all the states with the exception of SAMState in sam.py since SAMState uses the chex.dataclass decorator instead of inheriting from NamedTuple. for consistency should...
@vroulet apologies on the delay! been quite a month for me but we're back. ready for another review :)
apologies for the delay! It looks like a lot of restructuring happened since my last changes. I can open a new pull request that does these changes on the new...
happy to help out with the wrapper version if there's still interest!
I took a look at using [sphinx-toolbox](https://sphinx-toolbox.readthedocs.io/en/latest/extensions/more_autodoc/autonamedtuple.html) to autodoc the namedtuples, but I wasn't able to resolve the duplicate fields showing up. It seems that this is a known issue...
this is what it looks like if you remember the :members: line in each of the autoclass directives
revisiting this! so removing :members: from the autoclass directives isn't a viable solution since it also removes fields that are accurately documented like for LookaheadState i've tried using the autodoc-skip-member...
oh that looks good! hmm I just tried commenting out :members: again and the entire docstring for LookaheadState is gone from the docs (this has never happened before).