Skip to content

Multipage imread#3575

Merged
opencv-pushbot merged 3 commits intoopencv:masterfrom
Ashod:multipage_imread
Jan 12, 2015
Merged

Multipage imread#3575
opencv-pushbot merged 3 commits intoopencv:masterfrom
Ashod:multipage_imread

Conversation

@Ashod
Copy link
Copy Markdown
Contributor

@Ashod Ashod commented Jan 3, 2015

Added support to load multi-page images via imreadmulti API.

The mechanism is universal as BaseImageDecoder is augmented with a new function to allow for easy support of multipage decoding.
Multipage support is added to Tiff decoder, and is fully functional.

Complete tests added for both imread and imreadmulti (test images added to opencv_extra repo).

NOTES:

  • The API could be alternatively named imread_multi (note the underscore). I leave this open to suggestions.
  • I didn't include C API changes here as I'd like to get feedback on the interface. Specifically, should CvSeq be used for the output Mat array? Or should another approach be used?
  • I didn't include 2.4 branch changes as I'd like get initial approval on the change set and API before I invest the necessary effort on 2.4 branch.

@Ashod Ashod force-pushed the multipage_imread branch from 92343af to 4739648 Compare January 3, 2015 17:48
@Ashod
Copy link
Copy Markdown
Contributor Author

Ashod commented Jan 8, 2015

Anyone willing to review this, please?

@vpisarev
Copy link
Copy Markdown
Contributor

looks great! 👍

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.

3 participants