Skip to content

Installation tutorials rework#18195

Merged
alalek merged 3 commits intoopencv:masterfrom
mshabunin:linux-tutorial
Oct 7, 2020
Merged

Installation tutorials rework#18195
alalek merged 3 commits intoopencv:masterfrom
mshabunin:linux-tutorial

Conversation

@mshabunin
Copy link
Copy Markdown
Contributor

@mshabunin mshabunin commented Aug 26, 2020

Changes:

  • Created "Installation overview" and "Configuration reference" tutorials.
  • Updated "Linux installation" tutorial.
  • Added scripts for automatic checks of installation for Ubuntu 18
  • General tutorials: moved "Author" and "Version" information from listing into tutorials, marked some tutorials "Obsolete"
  • Minor CSS doxygen changes

Demo

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or other license that is incompatible with OpenCV
  • The PR is proposed to proper branch ❔
  • There is reference to original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

@alalek
Copy link
Copy Markdown
Member

alalek commented Aug 31, 2020

BTW, There is something wrong with new logo.
This style can fix that:

#projectlogo
{
	text-align: center;
	vertical-align: center;
	border-collapse: separate;
	padding-left: 0.5em;
}

Copy link
Copy Markdown
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.

Great update!

Comment on lines +192 to +193
`CV_ENABLE_INTRINSICS`
`CV_DISABLE_OPTIMIZATION`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

TBD?

@mshabunin mshabunin marked this pull request as ready for review October 7, 2020 17:35
@mshabunin
Copy link
Copy Markdown
Contributor Author

@alalek , @asmorkalov , I think the PR can be considered ready for review. There are still some unfinished things in "Config reference", but I believe the focus of the PR should be on "Installation overview" and "Installation in Linux'. We can restructure and update config reference later with Windows/OSX/ARM/Android installation tutorials updates.

@mshabunin
Copy link
Copy Markdown
Contributor Author

Looks like Javadoc build is broken due to some reasons.

Copy link
Copy Markdown
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.

Well done! Thank you 👍

@alalek alalek merged commit 46ccde8 into opencv:master Oct 7, 2020
a-sajjad72 pushed a commit to a-sajjad72/opencv that referenced this pull request Mar 30, 2023
Installation tutorials rework

* Doc: general installation, config reference, linux installation

* Doc: addressed review comments

* Minor fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: documentation Documentation fix or update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants