JuliaAstro / SPICE.jl

Julia wrapper for NASA NAIF's SPICE toolkit
MIT License
54 stars 14 forks source link

Fixing deprecations in previously not included functions #3

Closed Brinky5678 closed 6 years ago

Brinky5678 commented 6 years ago

Dear Maintainers,

Within the "Spice.jl" module definition, I included the files "constants.jl", "math.jl", "pck.jl", and "spk.jl". I required the use of the functions already created in these files, yet were previously not exported. Additionally, some deprecation warnings were fixed in those files regarding array definitions.

Please consider my changes.

With kind regards,

Dennis Brinkman

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 83


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/math.jl 0 1 0.0%
src/pck.jl 0 3 0.0%
src/spk.jl 0 3 0.0%
<!-- Total: 0 7 0.0% -->
Files with Coverage Reduction New Missed Lines %
src/b.jl 2 95.45%
<!-- Total: 2 -->
Totals Coverage Status
Change from base Build 82: -0.4%
Covered Lines: 148
Relevant Lines: 240

💛 - Coveralls
coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 85


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/m.jl 7 8 87.5%
src/s.jl 21 36 58.33%
<!-- Total: 38 54 70.37% -->
Totals Coverage Status
Change from base Build 82: 25.7%
Covered Lines: 201
Relevant Lines: 229

💛 - Coveralls
codecov-io commented 6 years ago

Codecov Report

Merging #3 into master will increase coverage by 25.68%. The diff coverage is 68%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master       #3       +/-   ##
===========================================
+ Coverage   62.08%   87.77%   +25.68%     
===========================================
  Files          18       15        -3     
  Lines         240      229       -11     
===========================================
+ Hits          149      201       +52     
+ Misses         91       28       -63
Impacted Files Coverage Δ
src/b.jl 100% <ø> (ø) :arrow_up:
src/k.jl 100% <ø> (+100%) :arrow_up:
src/SPICE.jl 90.9% <ø> (ø) :arrow_up:
src/t.jl 100% <100%> (+100%) :arrow_up:
src/c.jl 93.1% <100%> (+0.24%) :arrow_up:
src/p.jl 100% <100%> (ø)
src/s.jl 52.5% <58.33%> (+52.5%) :arrow_up:
src/m.jl 87.5% <87.5%> (ø)
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e37f139...c7fec5e. Read the comment docs.

helgee commented 6 years ago

Thank you for your contribution! Could I bother you to move the functions you need to the respective alphabetical file? I orginally wanted to group them by functionality but lost track due to the large number of functions in SPICE.

If you don't have the time I will merge and move them myself.

Brinky5678 commented 6 years ago

No Problem. Somewhere this week I will move everything to be in the correct alphabetical order. Additionally, I will also include some tests of the functions to improve the code coverage.

Brinky5678 commented 6 years ago

Updated the branch, including additional tests. Not sure about the codecov/patch, maybe I don't fully understand how that features works yet.

Brinky5678 commented 6 years ago

That was indeed the problem! It is working now.

helgee commented 6 years ago

Please activate the test and then this is good to go 👍 Thank you!

(BTW: You might want to add the email address you use for your Git commits to your GitHub account. This way they become associated to your account and become clickable in the interface.)