This attempt is clever. However, I find it only works if the length of s is even. It assumes that whether the length of s is even or odd, '_' should be added at the end.

As a demonstration:

solution('python') => ['py', 'th', 'on'], which is correct
But solution('pythonista') => ['py', 'th', 'on', 'is', 'ta'], which is incorrect