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.

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

Move pass pattern to ov #7255

Merged

Conversation

ilyachur
Copy link
Contributor

PR depends on #7240

@ilyachur ilyachur added category: Core OpenVINO Core (aka ngraph) WIP work in progress labels Aug 26, 2021
@ilyachur ilyachur added this to the 2022.1 milestone Aug 26, 2021
@ilyachur ilyachur requested a review from GlebKazantaev August 26, 2021 12:37
@ilyachur ilyachur force-pushed the move_pass_pattern_to_ov branch from 63daf37 to 755212d Compare August 27, 2021 06:04
@ilyachur ilyachur force-pushed the move_pass_pattern_to_ov branch from 494091e to 70ecf71 Compare August 29, 2021 12:57
@ilyachur ilyachur marked this pull request as ready for review August 31, 2021 08:54
@ilyachur ilyachur requested review from a team August 31, 2021 08:54
@ilyachur ilyachur changed the title [WIP] Move pass pattern to ov Move pass pattern to ov Aug 31, 2021
@ilyachur ilyachur removed the WIP work in progress label Aug 31, 2021
}

OPENVINO_DEPRECATED("Use MatcherPass instead")
void add_matcher(const std::shared_ptr<pattern::Matcher>& m,
Copy link
Contributor

Choose a reason for hiding this comment

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

can we remove it finally?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It can be backward incompatible for our old users.

Copy link
Contributor

Choose a reason for hiding this comment

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

why? it's deprecated for several releases

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm, yes, it is a valid comment. So I propose to remove these deprecated methods in the separate PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

ok, let's merge then

Copy link
Contributor

Choose a reason for hiding this comment

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

It is still used by some OV components actually

@ilya-lavrenov ilya-lavrenov merged commit 9eca6ba into openvinotoolkit:master Sep 2, 2021
@ilya-lavrenov ilya-lavrenov deleted the move_pass_pattern_to_ov branch September 2, 2021 07:03
using ov::pass::BackwardGraphRewrite;
using ov::pass::GraphRewrite;
using ov::pass::MatcherPass;
using ov::pass::RecurrentGraphRewrite;
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's keep it only in legacy API.

namespace pass {
enum class PassProperty : uint32_t {
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's keep pass property in legacy API

Comment on lines +32 to +33
using ov::pass::PassProperty;
using ov::pass::PassPropertyMask;
Copy link
Contributor

Choose a reason for hiding this comment

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

keep in legacy

};
using ov::pass::pattern::Matcher;
using ov::pass::pattern::MatcherState;
using ov::pass::pattern::RecurrentMatcher;
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's keep it in legacy API

Comment on lines +98 to +108
enum class FusionType : uint32_t {
//`DIFFERENTIABLE_FUSIONS` produce ops that support autodiff
// i.e. implement `generate_adjoints`
DIFFERENTIABLE_FUSIONS = 0x1,
REGULAR_FUSIONS = 0x2,
//`FOP_FUSIONS` produce ops in the FusedOps category that might
// not be supported by all backends
FOP_FUSIONS = 0x4,
ALL_FUSIONS = 0xFFFFFFFF
};
using FusionTypeMask = ngraph::EnumMask<FusionType>;
Copy link
Contributor

Choose a reason for hiding this comment

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

let's remove this

akuporos pushed a commit to akuporos/openvino that referenced this pull request Sep 29, 2021
* Moved ngraph::Node to ov namespace

* Fixed code style

* Fixed VPU

* Fixed GNA

* Fixed tests

* Added aliases for backward compatibility

* Fix clDNN

* Try to fix build

* Fixed comment

* Renamed RTTI macros

* Add new headers

* Fixed ngraph build

* Fixed unit tests

* Try to fix Serialize
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Core OpenVINO Core (aka ngraph)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants