search

mathpaquette / IQFeed.CSharpApiClient

IQFeed.CSharpApiClient is fastest and the most well-designed C# DTN IQFeed socket API connector available
MIT License
120 stars 43 forks source link

Examples #143

Closed jphiggs closed 1 year ago

jphiggs commented 2 years ago

Hi Mathieu, it's been a while. You may recall I helped implement L2 a few years ago and was going to create an example for #141. As I was reviewing the current examples, I found that they're blowing up on the 6.2 ProtocolVersion specified in IQFeedDefault.cs. The AdminMessageHandler is throwing because the response string starts with 'E'. That's as far as I got for now. For some reason IQFeed is returning an error. The response string is "E,6.2 is not a valid protocol.,\r"

mathpaquette commented 2 years ago

@jphiggs its been a while! Glad to see you again around! Are you able to dig a little bit further ? I dont have access to market depth anymore. Math

jphiggs commented 2 years ago

Yes I will gladly, but it's the current examples that are failing with 6.2 and I haven't looked at any of this code in a long while. If you can't afford the time to figure out that one, maybe I'll create a new set of examples with a console menu targeting .NET 6 and also a set of the same examples for WPF .

jphiggs commented 2 years ago

I finally got around to fixing the L2 example. Just need to comment/uncomment the appropriate lines to switch between requesting/handling MBP and MBO data. I also left the 6.1 and prior handlers there (commented out) in case someone still wants to use those.

I also created a new menu driven console demo project including Lookup and Streaming examples. It targets the .NET 6 runtime and has separate choices for MBP and MBO L2 data. It eliminates the need to re-compile to run different examples (unless you want to change the ticker symbol). Everything works fine for me.

I created a new branch (Demo.Net6Protocol6.2) with 2 commits but could not push to the origin for some reason. There's not a pull request yet.

Let me know what you think.

mathpaquette commented 2 years ago

Sure thanks let me take a look !

Sent from my iPhone

On Jun 28, 2022, at 1:44 PM, John Higgins @.***> wrote:



I finally got around to fixing the L2 example. Just need to comment/uncomment the appropriate lines to switch between requesting/handling MBP and MBO data. I also left the 6.1 and prior handlers there (commented out) in case someone still wants to use those.

I also created a new menu driven console demo project including Lookup and Streaming examples. It targets the .NET 6 runtime and has separate choices for MBP and MBO L2 data. It eliminates the need to re-compile to run different examples (unless you want to change the ticker symbol). Everything works fine for me.

I created a new branch (Demo.Net6Protocol6.2) with 2 commits but could not push to the origin for some reason. There's not a pull request yet.

Let me know what you think.

— Reply to this email directly, view it on GitHubhttps://github.com/mathpaquette/IQFeed.CSharpApiClient/issues/143#issuecomment-1169034102, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABBICNZPZP7OX37QQ2DKLVDVRM2XNANCNFSM5YVYBBPA. You are receiving this because you commented.Message ID: @.***>

mathpaquette commented 2 years ago

@jphiggs sorry ive been sick for few days. where's the PR?