Skip to content

Conversation

@Saafke
Copy link
Contributor

@Saafke Saafke commented Aug 15, 2019

Merge with extra: opencv/opencv_extra#664

'Learning-based Super Resolution' project for Google Summer of Code 2019 by Xavier Weber.

This pull request completes my project and also includes all the code from my previous two PR's (#2164 and #2200). Part of this code is implemented by Fanny Monori who worked on the same project (it was hard to split the code between two PR's perfectly without problems).

This pullrequest changes

  • Adds a learning-based super-resolution module to OpenCV.
  • Adds loading code for three popular SR datasets.

This module provides an interface so that users can easily upscale images via neural networks. Four trained models are supported. I implemented EDSR and FSRCNN. Fanny Monori implemented ESPCN and LapSRN.

This PR is closely related to Fanny's PR (#2229) who provides the same basic functionality but with other additions.

Saafke added 2 commits August 15, 2019 22:08
Adds the module plus loading classes for SR data

Complete with docs, tutorials and tests.
Copy link
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

@Saafke @fannymonori Please take a look on merged changes from #2229

When building OpenCV, run the following command to build the 'dnn_superres' module:

```make
cmake -DOPENCV_EXTRA_MODULES_PATH=<opencv_contrib>/modules -Dopencv_dnn_superres=ON <opencv_source_dir>
Copy link
Member

Choose a reason for hiding this comment

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

-Dopencv_dnn_superres=ON

There is no such option.
There are similar BUILD_ options, but adding them here is redundant (=ON be default).

It is better to mention -DBUILD_EXAMPLES=ON here and in other tutorials (=OFF be default).

Copy link
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Thank you for contribution 👍

@alalek alalek merged commit 4c1fb00 into opencv:master Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants