No, for daily bars starting and closing at 17:00 ET, you need this:
BarOffset = (ET+17)*60;
And the short trade problem was indeed tricky, I had to look twice until I saw the reason. You're shifting the series by 1 and use 80 bars, so your lookback period must be 1+80 = 81 at least. The default is 80.
The too short lookback period caused the first value of the series to be wrong, therefore the short trade condition was never met. The computer can not detect this kind of error, so you get no error message.