run() is called repeatedly. RollLong and RollShort are global values associated with your asset. The more you divide it by 365, the more it approaches zero. So your script basically set them to a negligible value.

Consider using a fixed value. You can configure it in your asset list and leave it alone in your script.