search

mottosso / cmdx

Fast and persistent subset of maya.cmds
https://mottosso.com/cmdx
BSD 2-Clause "Simplified" License
193 stars 36 forks source link

fix hang when adding nested compound attributes #38

Closed benblo closed 3 years ago

benblo commented 4 years ago

fixes https://github.com/mottosso/cmdx/issues/5

mottosso commented 3 years ago

Could we add just one test to this, to make sure it works on all Maya's, and that it won't break again in the future?

benblo commented 3 years ago

I could, though last time I used tests wasn't very fun ;) I don't really have time to spend on this right now, I tend to work in bursts so next time I can focus on cmdx, I'll do it right, setup the docker thing you mentioned etc and finish this.

Btw, I recently created wrappers for some APIs (eg playbackOptions), that are exposed as commands although to me they are much better suited as a class with properties (all it does is expose get/set accessors). Much of my pains with Maya come from not having proper autocompletion for cmds, not having proper typing information, and spending way too much time reading the docs to figure out the keyword combinations, so I've now taken the habit of whenever I need a new API, first wrapping it into something I find more sensible (I feel like most of it could be automated). I don't know if this is of any interest to you? maybe in a different module, since I feel this is not necessarily the focus of cmdx.

mottosso commented 3 years ago

I could, though last time I used tests wasn't very fun ;)

Haha, I understand exactly. xD I've added a test just now.

I don't know if this is of any interest to you? maybe in a different module, since I feel this is not necessarily the focus of cmdx.

Not for cmdx, that's more in the territory of PyMEL I think; i.e. all-encompassing mess. What's keeping you from using that?