Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add read/write of text formats #23

Merged
merged 7 commits into from
Oct 24, 2020

Conversation

magister-ludi
Copy link
Contributor

The changes in this branch add reading and writing of netpbm "plain" formats to the package and tests for the new methods.

Changes also fix #22, by adding tests for PBM binary formats and fixing a bug and a deprecation warning uncovered by the tests.

I'm happy to make any changes that will help this code conform with the existing code. Opinions are welcome on whether it's worth submitting changes so that FileIO.jl uses Netpbm.jl for all netpbm I/O. I can do that if this PR is merged.

Copy link
Member

@timholy timholy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, this a great! Thanks for fleshing this package out!

Yes, I'd support making this the default reader.

src/Netpbm.jl Show resolved Hide resolved
@magister-ludi magister-ludi merged commit 2aac7d8 into JuliaIO:master Oct 24, 2020
@magister-ludi magister-ludi deleted the textformats branch October 25, 2020 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

An error and a deprecation warning while loading "PBMBinary" format
2 participants