-
Notifications
You must be signed in to change notification settings - Fork 244
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
[Exp] Add Log Expression #12448
[Exp] Add Log Expression #12448
Conversation
|
||
// template instantiations | ||
template class UnaryExpression<UnaryOperations::Absolute>; | ||
template class UnaryExpression<UnaryOperations::Logarithmic>; |
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.
template class UnaryExpression<UnaryOperations::Logarithmic>; | |
template class UnaryExpression<UnaryOperations::Logarithm>; |
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.
done :)
Co-authored-by: Máté Kelemen <[email protected]>
ping @matekelemen |
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.
thx
📝 Description
This PR removes
UnaryAbsExpression
and creates a more genericUnaryExpression
which is able to getAbs
andLog
, and can be easily extended in future for other operators if required.Finally I am also adding documentation in the same PR :)
🆕 Changelog
UnaryAbsExpression
UnaryExpression
ExpressionUtils::Log