Enjoying your algo tutorials, but bemused by the downloadable code in your Friday Stop risk manager.
First, it's set to trigger on a Monday!
if (_tradingTime.DayOfWeek == DayOfWeek.Monday
Second, for live trading it's set to use the PC time rather than the trader server time. Why would you ever want to do that? You'll end up going flat at some random time depending on your timezone.
Given that naive customers might use this for live trading without noticing these oddities, you might want to release a fix?
Finally, VS 2019 flags the _localVariable underscore notation as a naming violation for a private variable in namespace scope, preferring LocalVariable.
There's also a redundant variable - account.
Hope this is useful
[Created by a customer]
