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

Serde support #54

Open
volks73 opened this issue Jan 7, 2023 · 1 comment · May be fixed by #57
Open

Serde support #54

volks73 opened this issue Jan 7, 2023 · 1 comment · May be fixed by #57

Comments

@volks73
Copy link
Contributor

volks73 commented Jan 7, 2023

I was starting to play around with this crate and others in the "ndarray family" to become more familiar with using Rust for CV and ML applications. In my adventure, I have a need to (de)serialize images to pass between processes. Constantly decoding/encoding into an image format seems excessive, but maybe this is the ultimate solution. So, I saw that the ndarray crate has an optional serde feature, but the core, ImageBase type in this crate does not implement the Serialize or Deserialize traits and there is no similar optional feature.

Is there interest in adding Serde support to this crate as a similar optional feature so ImageBase can be (de)serialized?

@xd009642
Copy link
Member

xd009642 commented Jan 7, 2023

Sure, such a PR would be accepted (add serde as a feature that optionally enables ndarray serde feature).

@volks73 volks73 linked a pull request Jan 7, 2023 that will close this issue
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 a pull request may close this issue.

2 participants