Fix CSV.jl deprecations

[stillyslalom]

Fixes

julia> model2 = PR(["water", "acetone"])
┌ Warning: `lazystrings` keyword argument is deprecated; use `stringtype=PosLenString` instead
└ @ CSV C:\Users\alexa\.julia\packages\CSV\owrEo\src\context.jl:270
┌ Warning: `lazystrings` keyword argument is deprecated; use `stringtype=PosLenString` instead
└ @ CSV C:\Users\alexa\.julia\packages\CSV\owrEo\src\context.jl:270
┌ Warning: `lazystrings` keyword argument is deprecated; use `stringtype=PosLenString` instead
└ @ CSV C:\Users\alexa\.julia\packages\CSV\owrEo\src\context.jl:270
┌ Warning: `lazystrings` keyword argument is deprecated; use `stringtype=PosLenString` instead
└ @ CSV C:\Users\alexa\.julia\packages\CSV\owrEo\src\context.jl:270
┌ Warning: `lazystrings` keyword argument is deprecated; use `stringtype=PosLenString` instead
└ @ CSV C:\Users\alexa\.julia\packages\CSV\owrEo\src\context.jl:270

[longemen3000]

Nice!, can you up the package version from 0.2.2 to 0.2.3?

[longemen3000]

Mmm, the errors are relevant

[stillyslalom]

Yep, working on a fix.

…

On Fri, Oct 8, 2021 at 1:50 PM Andrés Riedemann ***@***.***> wrote: Mmm, the errors are relevant — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#46 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABA73GELYFPQ5XBSTKWUOZLUF44OFANCNFSM5FT2CPDQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[stillyslalom]

Tests pass on my machine.

[codecov-commenter]

Codecov Report

Merging #46 (89d6e84) into master (1b0b910) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
- Coverage   83.51%   83.41%   -0.10%     
==========================================
  Files         104      104              
  Lines        6319     6319              
==========================================
- Hits         5277     5271       -6     
- Misses       1042     1048       +6     

Impacted Files	Coverage Δ
src/utils/database.jl	89.39% <100.00%> (-0.37%)	⬇️
src/models/eos/Activity/COSMOSAC/utils.jl	90.90% <0.00%> (-9.10%)	⬇️
...property_solvers/multicomponent/bubble_pressure.jl	93.75% <0.00%> (-3.13%)	⬇️
src/models/combiningrules.jl	98.55% <0.00%> (-1.45%)	⬇️

---

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 1b0b910...89d6e84. Read the comment docs.

[longemen3000]

test passes on julia nightly, but fails on julia 1.6, this is unrelated (happens because we have the manifest in the repo, a thing that we shouldn't). so, im gonna merge and remove the manifest myself