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
Added some Q #4
Added some Q #4
Conversation
Please add docstrings |
Pull Request Test Coverage Report for Build 101
💛 - Coveralls |
Pull Request Test Coverage Report for Build 114
💛 - Coveralls |
Codecov Report
@@ Coverage Diff @@
## master #4 +/- ##
===========================================
- Coverage 99.75% 82.79% -16.97%
===========================================
Files 21 23 +2
Lines 414 523 +109
===========================================
+ Hits 413 433 +20
- Misses 1 90 +89
Continue to review full report at Codecov.
|
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.
Looks good! Just some minor changes
src/e.jl
Outdated
handleerror() | ||
permutedims(r) | ||
|
||
end |
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.
Newline missing
src/q.jl
Outdated
|
||
- [NAIF Documentation](https://naif.jpl.nasa.gov/pub/naif/toolkit_docs/C/cspice/qxq_c.html) | ||
""" | ||
qxq(q1,q2...) = qxq(collect(q1),collect(q2)) |
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.
Remove this definition
test/m.jl
Outdated
vecgood = [1., 2., 3.] | ||
@test mxvg(matrix, vecgood) == [6.,20.] | ||
vecgood = [1., 2., 3.] | ||
@test mxvg(matrix, vecgood) == [6.,20.] |
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.
Just use general Julia syntax, which is matrix * vecgood
test/q.jl
Outdated
|
||
dq = qxq(q, qav) | ||
|
||
#dq = [-0.5 * x for x in dq] |
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.
Do not comment out code, delete it. Git will take care of 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.
Or do something like # TODO: Finish this
test/q.jl
Outdated
|
||
qK = [0.0, 0.0, 0.0, 1.0] | ||
|
||
qIJ=qxq(qI, qJ) |
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.
Always spaces around =
for assignment
test/r.jl
Outdated
end | ||
|
||
|
||
end |
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.
Newline missing
No description provided.