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.

Sign up for GitHub

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

Jump to bottom

CMakeLists & Setup Update #355

Merged
merged 8 commits into from
Aug 20, 2020
Merged

CMakeLists & Setup Update #355

merged 8 commits into from
Aug 20, 2020

Conversation

kiritigowda
Copy link
Collaborator

@kiritigowda kiritigowda commented Aug 18, 2020 •
edited
Loading

CMakeList Options

  • -D NEURAL_NET=OFF - turn off all NN modules
  • -D RALI=OFF - turn off rali modules

MIVisionX Setup

python MIVisionX-setup.py --directory [setup directory - optional (default:~/)]
                          --installer [Package management tool - optional (default:apt-get) [options: Ubuntu:apt-get;CentOS:yum]]
                          --opencv    [OpenCV Version - optional (default:3.4.0)]
                          --miopen    [MIOpen Version - optional (default:2.5.0)]
                          --miopengemm[MIOpenGEMM Version - optional (default:1.1.5)]
                          --protobuf  [ProtoBuf Version - optional (default:3.12.0)]
                          --rpp       [RPP Version - optional (default:0.4)]
                          --ffmpeg    [FFMPEG Installation - optional (default:no) [options:yes/no]]
                          --rali      [MIVisionX RALI Dependency Install - optional (default:yes) [options:yes/no]]
                          --neural_net[MIVisionX Neural Net Dependency Install - optional (default:yes) [options:yes/no]]
                          --reinstall [Remove previous setup and reinstall (default:no)[options:yes/no]]

@kiritigowda kiritigowda added enhancement New feature or request bugfix Bug fixes to existing features labels Aug 18, 2020
@kiritigowda kiritigowda requested a review from rrawther August 18, 2020 21:50
This was referenced Aug 18, 2020
Closed
Closed
rrawther
rrawther reviewed Aug 19, 2020
View reviewed changes
MIVisionX-setup.py Outdated Show resolved Hide resolved
rrawther
rrawther reviewed Aug 19, 2020
View reviewed changes
MIVisionX-setup.py Outdated Show resolved Hide resolved
rrawther
rrawther requested changes Aug 19, 2020
View reviewed changes
Copy link
Contributor

@rrawther rrawther left a comment

Choose a reason for hiding this comment

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

Make sure tests pass for both cases, before merging

@kiritigowda Codacy Production
Copy link
Collaborator Author

Codacy Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

See the complete overview on Codacy

@kiritigowda kiritigowda merged commit 509ece4 into ROCm:master Aug 20, 2020
@kiritigowda kiritigowda deleted the kg/cmake-setup-fix branch August 20, 2020 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rrawther rrawther rrawther requested changes

Assignees

@paveltc paveltc

@LakshmiKumar23 LakshmiKumar23

Labels
bugfix Bug fixes to existing features enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@kiritigowda @rrawther @paveltc @LakshmiKumar23