[OpenVINO Scripts] Updated with RHEL8#4296
Merged
azhogov merged 3 commits intoopenvinotoolkit:masterfrom Feb 12, 2021
Merged
Conversation
mshabunin
reviewed
Feb 11, 2021
| os=$( . /etc/os-release ; echo "${ID}${VERSION_ID}" ) | ||
| case $os in | ||
| centos7|ubuntu18.04|ubuntu20.04) [ -z "$print" ] && echo "Detected OS: ${os}" ;; | ||
| centos7|rhel8.2|ubuntu18.04|ubuntu20.04) [ -z "$print" ] && echo "Detected OS: ${os}" ;; |
Contributor
There was a problem hiding this comment.
Currently the latest version is 8.3. Is it possible to match all rhel8.x versions here? Can we add centos8 too? I understand that it is not "officially supported", but I think it can be used for package installation testing at least.
Same applies to all "rhel8.2" checks below.
Can we add centos:8 image to the self-test above (L68)?
Contributor
Author
There was a problem hiding this comment.
Commentary with rhel8.* is valid, but I don't sure about centos8
Contributor
There was a problem hiding this comment.
I just thought that it would be great to have ability to check rhel8 branch without registration:
# yum install -y gstreamer1
Updating Subscription Management repositories.
Unable to read consumer identity
This system is not registered to Red Hat Subscription Management. You can use subscription-manager to register.
Last metadata expiration check: 0:00:39 ago on Fri Feb 12 09:25:36 2021.
No match for argument: gstreamer1
Error: Unable to find a match: gstreamer1
| if [ "$os" == "ubuntu18.04" ] ; then | ||
|
|
||
| pkgs_opencv_req=(libgtk-3-0) | ||
| pkgs_opencv_req=(libgtk-3-0 libgl1) |
Contributor
There was a problem hiding this comment.
I can see that these changes are already on master branch. Why are they shown in the diff? Is it GitHub issue? Or should the branch be rebased?
azhogov
approved these changes
Feb 11, 2021
generalova-kate
approved these changes
Feb 12, 2021
a-sidorova
added a commit
to a-sidorova/openvino
that referenced
this pull request
Feb 15, 2021
* fix ss * successfully converted * successfully run moved infer and normalizer unit-tests * successfully rewritten StridedSlice infer unittests * int64 array * Successfully converter crash-when-loading, xj_feauture and toy nets (cherry-picked maxpoolV4 and tf_broadcast_ext) * successfully moved PermuteAttrs to general mechanism * successfully converted xj_feauture and crash when loading with the new rewritten SS infer * fixed get_shape_from_slice and moved to common utils * fixed extending masks and some other * some refactoring * fixed extending masks in extractor, fixed licence year and some other code clearing * corrected a couple of unittests * fox permute for 5 rank slice and 4 rank inputs/ * WIP * Added comments * fixed StridedSlice in ProposalMutation.py * rechecked shape_infer unittests added some new cases * added shape_infer unit-tests after StridedSliceNormalizer pass and Permute unit-tests * corrected unittests * Applied review comments * general permutations for inputs implemented, corrected ellipsis unrolling when shrink_axis is at the beginning, some other corrections * removed code duplication in infer and normalizer, moved 'slices' attr normalizing to StridedSliceNormalizer.py * removed some code duplication and other minor improvements * Added tests * minor corrections * [GNA] Support 1d tensors (openvinotoolkit#4270) * [OpenVINO Scripts] Updated with RHEL8 (openvinotoolkit#4296) * Updated install_NEO_OCL_driver.sh & install_openvino_dependencies.sh with rhel8 case * install_NEO_OCL_driver: Fixed conditional * script: Updated with actual revisions * [CPU] MKLDNN NCHW pooling primitive performance fix. (openvinotoolkit#4235) * [IE][VPU]: Check memory capacity after finding the corresponding device (openvinotoolkit#4314) This issue relates to multi-device mode. While we are trying to allocate a graph on one of the devices we should check memory capacity only for the corresponding device, not for the last opened as far devices may have different memory capacity or the latest opened device may have still not sent its attributes (including memory capacity) * Azure CI: Enable IB initiators as helpers * Result rename operation (openvinotoolkit#4242) * Added result rename operation * Optimize imports * Added ResultRename to package_BOM * ResultRename moved to the end of back phase, code refactoring * Revert incorrect changes * Optimize imports * Added comments and optimized imports. * DPC++ link error workaround. (openvinotoolkit#4192) * DPC++ link error workaround. OpenVINO C++ program failed to link when DPC++ compiler is used. 'make_shared_blob' causes 'unresolved external symbol' error on linking. Commented out some __clang__ specific directives to workaround the issue in "ie_blob.h". * DPC++ compatibility issue fix #2 1. Removed type-by-type template class definition for __clang__. 2. Modified TBlob() destructor. The 'unresolved reference' error occur again if I left 'virtual ~TBlob();' only. It seems it needs to be 'virtual ~TBlob() {};'. * DPC++ compatibility fix #3 - Add DPC++ conditional code Uses '__SYCL_COMPILER_VERSION' predefined macro to check if the compiler is a DPC++ or not. Added conditional directive to switch code based of the detected compiler. NOTE: User program must include <CL/sycl.hpp>, or the '__SYCL_COMPILER_VERSION' macro won't be defined and this fix won't take effect. * DPC++ compatibility issue fix #4 Changed from #ifdef to #if + logical formulas. * DPC++ compatibility issue fix #5 Added compiler check logic in src/ie_rtti.cpp * DPC++ Compatibility issue #6 - ie_parameter.cpp Added compiler check macro for DPC++ to ie_parameter.cpp as well. Co-authored-by: Yasunori Shimura <yasunori.shimura@intel.com> * Azure CI: Disable IB stop * Remove generic ie op (openvinotoolkit#4213) * Removed legacy IE shape infer * Removed GenericIE operation * Removed legacy shape infer tests * Removed legacy test with legacy IE reshape * Fixed compilation issues related to removal of GenericIE * Fixed one more compilation issue with clDNN * Fixed test for reading experimental ops * Updated tests and make IR Reader to load old experimenal and extension ops as opset6 * Change opset of some ops only if they are currently experimental/extension to avoid situation like opset1::Proposal -> opset6::Proposal * Removed more legacy code * Returned back code removed by mistake * Fixed issues related to incorrect merge with master * Merge fixes * Fixed unit tests which starts to fail because now loading the model with unknown operation is failed earlier * Removed incorrectly added code Co-authored-by: Evgeny Lazarev <elazarev.nnov@gmail.com> * [IE CLDNN] Added CTCGreedyDecoderSeqLen operation (openvinotoolkit#4119) * [CPU] Refactors jitters for nGraph interop (openvinotoolkit#4255) * [IE CLDNN] Fixed CTCGreedyDecoderSeqLenLayerTest gpu instances (openvinotoolkit#4326) * Fix comparison of constant with short float NAN values (openvinotoolkit#4299) * fix comparison of constant with short float NAN values * adjust precision, remove elvises * more templates * add ir serialization test with float16 const * remove unused prototxt * [ONNX] Remove linking libonnx from unit-test (openvinotoolkit#4298) * Remove linking libonnx from unit-test * Consider all flavors of protobuf libraries * Avoid passing on NOTFOUND properties * Set system deps for mac * Revert include dirs set up * wider range of unittests added (froze the number) * ONNX RNN/GRU enable dynamic input shape (openvinotoolkit#4241) * Add support for custom ONNX operator PriorBoxClustered (openvinotoolkit#4202) * review comments applied * Force test use bash for setupvars.sh (openvinotoolkit#4321) setupvars.sh is incomatible with sh shell. * enabled skipped unit-test * comment corrections Co-authored-by: Pavel Esir <pavel.esir@intel.com> Co-authored-by: Elizaveta Lobanova <elizaveta.lobanova@intel.com> Co-authored-by: Artyom Anokhov <artyom.anokhov@intel.com> Co-authored-by: Maksim Kutakov <maksim.kutakov@intel.com> Co-authored-by: Maksim Doronin <maksim.doronin@intel.com> Co-authored-by: Alexander Zhogov <alexander.zhogov@intel.com> Co-authored-by: Anastasia Popova <anastasia.popova@intel.com> Co-authored-by: Yasunori Shimura <63567854+yas-sim@users.noreply.github.com> Co-authored-by: Yasunori Shimura <yasunori.shimura@intel.com> Co-authored-by: Evgeny Lazarev <evgeny.lazarev@intel.com> Co-authored-by: Evgeny Lazarev <elazarev.nnov@gmail.com> Co-authored-by: Roman Lyamin <Roman.Lyamin@intel.com> Co-authored-by: Marina Kolpakova <marina.kolpakova@intel.com> Co-authored-by: Vladimir Paramuzov <vladimir.paramuzov@intel.com> Co-authored-by: Bartosz Lesniewski <bartosz.lesniewski@intel.com> Co-authored-by: Tomasz Jankowski <tomasz1.jankowski@intel.com> Co-authored-by: Katarzyna Mitrus <katarzyna.mitrus@intel.com> Co-authored-by: Bartosz Sledz <bartosz.sledz@intel.com> Co-authored-by: Andrey Somsikov <andrey.somsikov@intel.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.