Improve stubs in av.container

[laggykiller]

No description provided.

[WyattBlue]

Should be template: Stream | None = None

[laggykiller]

I see that either codec_name or template is needed. Should I type it as an overload function instead?

Edit: Nevermind, typing as overload function would make it impossible to type:

    @overload
    def add_stream(
        self,
        codec_name: str,
        rate: Fraction | int | float | None = None,
        template: None = None,
        options: dict[str, str] | None = None,
    ) -> Stream: ...
    @overload
    def add_stream(
        self,
        codec_name: None,
        rate: Fraction | int | float | None = None,
        template: Stream,
        options: dict[str, str] | None = None,
    ) -> Stream: ...

The second overload function template argument must be provided, but it would cause non-default argument coming after default argument, which is invalid

[WyattBlue]

Yes, overload is appropriate here

[laggykiller]

Sorry I edited the previous comment, I found out that typing as overloading might be impossible. Any thoughts?