Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

LOD bug still present #38

Open
pareeohnos opened this issue Nov 14, 2021 · 1 comment
Open

LOD bug still present #38

pareeohnos opened this issue Nov 14, 2021 · 1 comment
Labels
bug Something isn't working
Milestone
1.0.0 Beta

Comments

@pareeohnos
Copy link
Owner

There is still a bug with the LOD calculations that is messing up entries, and it is extremely obvious when trying to enter an EP setup. It seems the app is loading yesterdays LOD values. One possibility is that TWS is not reporting the latest values when fetching historical data, so we need to change the time unit from days to something smaller. As small as possible would be best
@pareeohnos pareeohnos added the bug Something isn't working label Nov 14, 2021
@pareeohnos pareeohnos added this to the 1.0.0 Beta milestone Nov 14, 2021
@pareeohnos
Copy link
Owner Author

Implemented a (hopeful) fix. When the app loads or new feeds are requested, it now requests 2 sets of historical data

  • The last 20 days for ADR
  • The last n seconds since the market opened

The LOD is then calculated from the second set of data, which should be most accurate as it will be every second since the market opened

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
1.0.0 Beta
Development

No branches or pull requests
1 participant
@pareeohnos