Making a for-loop more efficient

By the way, you asked if I meant to use vmap() I’m a bit confused now as you recommended this to me earlier here:

Am I not using it correctly?