rofinn / FilePaths.jl

A type based approach to working with filesystem paths in julia
Other
82 stars 14 forks source link

add FileIO support #58

Closed johnnychen94 closed 3 years ago

johnnychen94 commented 3 years ago

Partially address https://github.com/JuliaIO/FileIO.jl/issues/315 by normalizing the SystemPath to string. This is probably not a good fix as I never used FilePath before... However, requiring all IO backends to provide FilePath support is perhaps not practical at all.

cc: @jkrumbiegel

rofinn commented 3 years ago

Yeah, we don't want to stringify the filepath, but rather loosen the type constraints for whichever backend is being too restrictive. See https://github.com/JuliaIO/FileIO.jl/pull/259/files. Once there's a MWE it should be pretty easy to figure out what needs to be updated.