-
-
Notifications
You must be signed in to change notification settings - Fork 51
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.
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
aggregationPeriod not configurable #211
Comments
I have no idea how I didn't notice this before! Will fix ASAP. |
Fix already on the way :-) PR in a minute
(I Also need this and have it ready)
… Am 05.02.2025 um 22:26 schrieb Graeme Holliday ***@***.***>:
I have no idea how I didn't notice this before! Will fix ASAP.
—
Reply to this email directly, view it on GitHub <#211 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACZTFXUL34QRH64QLMNHCCD2OJ6Y5AVCNFSM6AAAAABWR27KISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMZYGA3DAMRQGA>.
You are receiving this because you are subscribed to this thread.
|
Thanks! I beat you to it though haha |
Sold !
Ok, yours is better. ;-)
… Am 05.02.2025 um 22:36 schrieb Graeme Holliday ***@***.***>:
Closed #211 <#211> as completed via #212 <#212>.
—
Reply to this email directly, view it on GitHub <#211 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACZTFXWIVMSO5IMHAGFREHL2OJ74BAVCNFSM6AAAAABWR27KISVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJWGIYDCMBWGYZTQMA>.
You are receiving this because you commented.
|
Great! Thankyou! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The acceptAggregationPeriod of the newly created DXFeed Channels is hardcoded to a very high value.
This prevents any realtime streaming.
Changing it to something more useful as in i.e. "0.1" solves it, would be even better to have it configurable, though
The text was updated successfully, but these errors were encountered: