-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat: SA-429 Add options support #62
Conversation
bbee9fe
to
fd07169
Compare
8dd95a2
to
f2e1a3d
Compare
f2e1a3d
to
c0325c7
Compare
e2d2aa1
to
eb1271b
Compare
isinstance(strike, (float, int)) | ||
or ( | ||
isinstance(strike, str) | ||
and re.match(r"^(SPOT|FWD)([+-]\d{1,2}%)?$", strike) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also support ATM here to match v9?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not think any of the strings are supported atm when creating the option directly. But looks like it should be supported with the get_option
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have raised an error
eb1271b
to
4ffaa64
Compare
4ffaa64
to
76ba8ca
Compare
No description provided.