-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
ptype complex64_t added #1140
ptype complex64_t added #1140
Conversation
lambday
commented
Jun 3, 2013
- complex64_t datatype added, with (only required) support for Math, SGVector, SGMatrix, SGSparseVector, SGSparseMatrix
- two methods added in SGVector, get_real() and get_imag(), for complex64_t
- two unit tests added, for Math complex functions and SGVector complex
@@ -1738,6 +1817,11 @@ void Parameter::add(SGMatrix<SGSparseVector<floatmax_t> >* param, | |||
SG_FREE(*(SGString<float64_t>**) m_parameter); break; | |||
case PT_FLOATMAX: | |||
SG_FREE(*(SGString<floatmax_t>**) m_parameter); break; | |||
case PT_COMPLEX64: | |||
SG_SERROR("TParameter::delete_cont(): Implementation " |
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.
What is this supposed to do?
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.
@karlnapf I haven't added support for SGString<complex64_t>, that's why kept anything related to SGString<complex64_t> this way. Shall I add it?
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.
No you dont have to add it, but please change the error message to "Parameters of strings of complex64_t are not supported"
wow, nice patch and a lot of stuff The parameter things should be carefully unit-tested (best with serialisation) |
@karlnapf thanks. I'm working on the parameter unit-tests for complex. Will add soon. |
static inline float64_t sinh(float64_t x) | ||
{ | ||
return ::asinh(x); | ||
return ::asinh(x); //TODO is this correct? |
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.
Ouch!
Unit-tests! :)