'cytoolz.functoolz.Compose' object has no attribute '__module__'

[djrtwo]

When porting to cytoolz from toolz, I ran into the problem where the some previous library code was expecting Compose to have __module__ defined. I'm new to the project, but would expect the goal to be to generally to be able to move to cytoolz from toolz without any issues.

Should this be defined in Compose? If so, I can work to add it.

[pipermerriam]

Any thoughts on whether this could be addressed?

Also, here's a cute puppy to thank you for your time.

[eriknw]

Thanks for the report and the puppy! And I'm sorry for the delay; I had a 5000+ mile road trip to plan and execute that just finished.

I'm new to the project, but would expect the goal to be to generally to be able to move to cytoolz from toolz without any issues.

You're right, that is the goal, so any issue like this is greatly appreciated.

To clarify, it is Compose objects and not the Compose class that don't have __module__ attribute`, right? Taking a look now.

[djrtwo]

I believe is the Compose class. This is the original error (sorry hard to read)

I previously tried to import it and just monkey patch it but can't because it's compiled c-code.

from cytoolz import (
    Compose
)

Compose.__module__ = "cytoolz.functoolz"

error:

TypeError: can't set attributes of built-in/extension type 'cytoolz.functoolz.Compose'

I hope that helps. I'm a few weeks away from seeing the error so that's all the info I have without digging back in deeper.

[pipermerriam]

@eriknw no worries about the delay. Thanks for taking a look and for maintaining an awesome library that I love.

[pipermerriam]

ping

[llllllllll]

the __module__ is always toolz.functoolz for toolz.functoolz.Compose, is that actually useful to put in cytoolz?

[eriknw]

This is in. Sorry for the delay. I plan to do a release later today.

[pipermerriam]

Much appreciated @eriknw and @llllllllll

[eriknw]

You bet, and thanks for the puppy!

Sorry again for the excessive delay. 0.9.0 was just released if you want to give it a go.