Amos You

Results 12 comments of 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).