I think the lacing looks right except you have the spokes laced to the wrong holes in the rim. The hole to the left of the valve stem should go to the flange closest to you.
This does not matter so much.. It's just creating a mirror image, though the convention is left of valve to "drive" side of hub, some rims are actually offset in a way that makes this incorrect..
I believe that this is correct:
One possible issue is that you put the right side pulling spoke ahead of the left side pulling spoke in the hub flange and then continued to lace the wheel.
If you look at the hub flange the holes are not aligned, they are offset. The first spoke after the valve hole should go to the left hub flange and the second spoke after the valve hole should go to the right hub flange AFTER the first spoke in the direction you will rotate the hub later.
Starting with your "key" spoke, and the first set on that side, it's fine - but the next set of spokes, done on the opposite side of the hub, is in correct - the spokes are over-reaching. If you started key spoke to left, then the opposite side goes one hole to the left (in the hub), and then up to the 2nd hole to the left in the rim; as-is, you have it one hole to the right, and then up to the left, effectively - making spoke too short.
Hopefully that makes some measure of sense. As always, sheldonbrown.com - undo and relace carefully reading.