Closed rofinn closed 3 years ago
Alright, this was bigger than expected, but a lot of the LOC changes are tied to removing the context blocks and moving a couple files around (also adding tests and docs). I've left several issues as TODO items to avoid making this bigger.
@oxinabox Would you mind double checking that I'm using DataDeps.jl as expected?
Alright, I believe I've either addressed, responded or opened an issue for each comment left. Since there don't seem to be any new comments I'm gonna merge this and get started on the opened issues tomorrow.
Attempting to resolve the following issues: #66, #57, #51, #50, #44, #43, #31
assert
functioncontext
callsdims
keyword similar to the stats functions https://github.com/JuliaLang/Statistics.jl/blob/master/src/Statistics.jl#L164 (using special cased:rows
and:cols
dims
to improve readability)_impute!(AbstractArray{Union{T, Missing}}, imp)
dropobs
anddropvars
withImpute.filter
and adims
keywordChain
not an imputor and have it work onAssertions
andImputors