Do you need an interactive session? If not, you should definitely use sbatch instead of srun. If there is an issue with login1, for example, or with the connection between login1 and your desktop, the job will be terminated.
You are using
in your script. This variable is set because you have modules loaded. If you don’t need them, just use module purge to cleanly unload all your modules. For the record, you have defined the modules to be loaded when logging in in your .bash_profile. If you no longer need them, clean them up there too.