MycroftAI / adapt

Adapt Intent Parser
Apache License 2.0
712 stars 154 forks source link

Feature/update pyee #75

Closed forslund closed 6 years ago

forslund commented 6 years ago

Is there a reason to keep pyee at v 1.0.1? In mycroft-core we wish to update to have the possibility to remove event handlers registered with once.

clusterfudge commented 6 years ago

Nope, just the version when I started developing adapt.The biggest delta between the two releases appears to be the introduction of asyncio support, which it attempts to use if the library is available. It shouldn't be installed by default, so the behavior should be pretty similar. It'd be interesting to see if the behavior of adapt or mycroft-core changed substantially by using the more advanced event loop features.

Anyhoo, approved.