Skip to content

Conversation

@dhsmyth
Copy link
Contributor

@dhsmyth dhsmyth commented Nov 26, 2025

  • Option to set-and-forget at client init
  • Option to override on each request in cases of part-AUTO part-MANUAL applications

Tested:

  • RithmicClient init()
  • submit_order()
  • modify_order()
  • cancel_order()

Shouldn't affect any existing systems 👍

- Option to set-and-forget at client init
- Option to override on each request in cases of part-auto part-manual applications
@rundef
Copy link
Owner

rundef commented Nov 29, 2025

Hey, yeah it looks good to me, thanks for the contribution. I'm assuming you tested on your side and everything works as expected ? (Can't test on my end rn, I don't have live access to Rithmic currently)

@dhsmyth
Copy link
Contributor Author

dhsmyth commented Dec 1, 2025

I traded Thursday/Friday with AUTO declaration during client init, debug traced that OrderPlacement.AUTO (enum 2) gets passed into functions where MANUAL was passed before, and no issues from Rithmic 👍 (unless my change caused the CME datacenter outage..)

Existing users won't notice, still defaults to MANUAL across the board as before.
Cheers

@rundef
Copy link
Owner

rundef commented Dec 1, 2025

ah ! I guess we found the reason behind the CME outage !

@rundef rundef merged commit 8694d88 into rundef:main Dec 1, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants