-
Notifications
You must be signed in to change notification settings - Fork 278
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
add optional local_parser_constants to Parser #1773
Conversation
Hi, I've run clang-format and found that the code formatting is good. Thanks for fixing the formatting. |
- add parse_julia function that uses "im" for I - implemented as described in symengine#1462
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 like the idea and everything looks sane.
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.
Thanks for the PR. Everything looks good except for a minor comment.
symengine/parser.h
Outdated
@@ -7,6 +7,7 @@ namespace SymEngine | |||
{ | |||
|
|||
RCP<const Basic> parse(const std::string &s, bool convert_xor = true); | |||
RCP<const Basic> parse_julia(const std::string &s, bool convert_xor = true); |
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 don't think we need a special function for julia, but we should add a function like basic_parse2
to expose sending local_parser_constants
to parse
C++ function.
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 don't see a string->basic map type in the cwrapper, should I add a CMapStrBasic
for this?
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.
Yes. See also #1378
parse_julia
function that uses "im" forI