cjdoris / ARFFFiles.jl

Load and save ARFF files
MIT License
5 stars 2 forks source link

fix #13 #14

Closed jbrea closed 2 years ago

jbrea commented 2 years ago

@cjdoris Did you have a chance at looking at this?

jbrea commented 2 years ago

Ah, yes, thanks. (for unknown reasons I didn't see your comment on github).

On Thu, Nov 11, 2021 at 01:34:52AM -0800, Christopher Rowley wrote:

@cjdoris commented on this pull request.

@@ -972,11 +972,19 @@ end return (pos, done) end

+_zero(::Any, ::Any, ::Any, ::Any) = 0 +function _zero(::AbstractVector{<:Union{<:AbstractString, Missing}}, r, i, nrows)

I believe this should be r.pools[r.colkindidxs[i]][1]

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/cjdoris/ARFFFiles.jl/pull/14#pullrequestreview-803588936

cjdoris commented 2 years ago

Just tried it on some datasets, works great thanks!

cjdoris commented 2 years ago

I'm registering a new version now.