Add type annotations about cmdx HOT 5 OPEN

Heya @Muream,

The python 3 syntax is more comfortable to use though considering cmdx supports older versions of Maya, the comment based syntax is probably the only viable option

Indeed, at least until Maya 2020 is deprecated, so for at least another 3 years.

Would you be open for me to add that in?

Yes, on 1 condition.

• Line length

Sounds petty, but that has been my issue with these things in the past. I don't benefit from type hints personally, but if there's one thing I do benefit from it's multiple side-by-side editor views. From what I gather, the parsing mechanic behind defining these as comments is that the need to be on one line? What ends up happening then is that these lines grow beyond 80 chars wide, to 200+ depending on how many arguments there are. Even that small Python 2 example is dangerously close at 76 chars.

from cmdx.

Haha yes, I always have at least two editors side by side so I hear you

It's perfectly possible to have type annotations per line in a multiline function definition like so

def createNode(
    type,  # type: Union[str, MTypeId]
    name=None,  # type: Optional[str]
    parent=None,   # type: Optional[DagNode]
):
    # type: (...) -> Node
    ...

Another option is to use a stub file. It's generally recommended to keep the annotations within the source file as that's just one file to maintain instead of two

But in this case, I think it might be the way to go since typing needs to be imported for more advanced type annotations (like generics) and that won't be available for python 2 either without having it as a dependency which I'm pretty sure you'd rather not have

from cmdx.

I like the idea of a stub file, in that it would have little (to none?) effect on those that don't use typing. But it does sound rather tedious on your part, having to write (or generate?) this file.

I'd be up for seeing the start of a PR with these comment-based type-hints, to see what kind of effect/damage it would have on readability. Maybe it's not that bad? If it is, then there's always the stub file.

from cmdx.

Cool, generating a stub is easy with mypy and after that it's as much work cleaning it up as adding annotations to the source file itself and is actually likely to give me more freedom to implement things like Generics

I'll make a proof of concept PR in the next few days with a few different styles of annotations 👍

from cmdx.

Awesome!

from cmdx.