Having interrupts turned off between iterations of loop()
is no good idea.
As I already have pointed out to you in your other thread
You should only disable interrupts for very short times, e.g. only for a couple of instructions to transfer highly volatile variables.
Instead of using noInterrupts()
I'd rather use ATOMIC_BLOCK
for such cases.