Maxpool spec update#3758

Merged

lazarevevgeny merged 30 commits intoopenvinotoolkit:masterfrom

pszmel:maxpool_spec_update

Feb 5, 2021

Contributor

pszmel commented Dec 28, 2020 •

edited

Loading

Update max pool spec

ticket: 43356

pszmel added 3 commits

December 28, 2020 14:43


          Update spec

73ebf8f


          Update outputs section

3cf4ac6


          Update rounding_type section

b1ddfad

pszmel added the category: Core label

pszmel self-assigned this

pszmel added 2 commits

December 29, 2020 11:25


          Update spec

9d221e2


          Update input section

bf1c7dc

pszmel requested review from alexey-varyzgin and rkazants

December 29, 2020 10:30

pszmel added 5 commits

December 29, 2020 11:33


          add newline


          fix one xml example

6af32dd


          fix indentation

46584be


          Merge remote-tracking branch 'upstream/master' into maxpool_spec_update

f7112d4


          Merge remote-tracking branch 'upstream/master' into maxpool_spec_update

f176a7e

pszmel marked this pull request as ready for review

January 11, 2021 12:38

pszmel requested a review from a team as a code owner

January 11, 2021 12:38


          Update detailed description

d5e9a81

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved


          Update detailed description

pszmel requested a review from rkazants

January 13, 2021 19:21

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved


          Merge remote-tracking branch 'upstream/master' into maxpool_spec_update

66cef4d

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

rkazants reviewed

View reviewed changes

Collaborator

rkazants left a comment

Please see comments.

pszmel added 5 commits

January 18, 2021 12:52


          Update spec

82989f1


          Merge remote-tracking branch 'upstream/master' into maxpool_spec_update

c9dd292


          fix typo

ed37055


          fix style

fa3e0ca


          fix example 5 description

46142dd

rkazants approved these changes

View reviewed changes

Collaborator

rkazants commented Jan 19, 2021

@lazarevevgeny, could you please review the update and merge once no concern? The specification gets much better by Piotr's effort.

rkazants requested a review from lazarevevgeny

January 19, 2021 11:17

lazarevevgeny suggested changes

View reviewed changes

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

lazarevevgeny requested a review from jdanieck

January 31, 2021 12:27

lazarevevgeny reviewed

View reviewed changes

docs/ops/pooling/MaxPool_1.md Show resolved Hide resolved

pszmel added 4 commits

January 31, 2021 15:00


           Merge remote-tracking branch 'upstream/master' into maxpool_spec_update

9191e2e


          Update spec

22bffc9


          Add Types section

e7ac30d


          Merge remote-tracking branch 'upstream/master' into maxpool_spec_update

15a3045

pszmel requested a review from lazarevevgeny

February 1, 2021 13:58

lazarevevgeny suggested changes

View reviewed changes

docs/ops/pooling/MaxPool_1.md Show resolved Hide resolved

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

docs/ops/pooling/MaxPool_1.md Outdated Show resolved Hide resolved

pszmel added 3 commits

February 2, 2021 22:45


          Refactor xml examples

e7b64d9


          Merge remote-tracking branch 'upstream/master' into maxpool_spec_update

b1e8213


          Update xml examples and add bfloat16 to supported types

055a28d

pszmel requested review from ilyachur and lazarevevgeny

February 4, 2021 10:05


          Update supported types

f8a0810

jdanieck approved these changes

View reviewed changes

lazarevevgeny approved these changes

View reviewed changes

lazarevevgeny merged commit 9695076 into openvinotoolkit:master

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

Reviewers

rkazants rkazants approved these changes

alexey-varyzgin Awaiting requested review from alexey-varyzgin

ilyachur Awaiting requested review from ilyachur

+2 more reviewers

lazarevevgeny lazarevevgeny approved these changes

jdanieck jdanieck approved these changes

Labels