Skip to content
This repository has been archived by the owner on Feb 6, 2023. It is now read-only.

Cut protobuf 2 #238

Merged
merged 2 commits into from
Aug 6, 2019
Merged

Cut protobuf 2 #238

merged 2 commits into from
Aug 6, 2019

Conversation

beam2d
Copy link
Member

@beam2d beam2d commented May 30, 2017

No description provided.

@gwtnb
Copy link
Member

gwtnb commented May 30, 2017

To cut protobuf 2, we have to edit some other lines. For example, https://github.com/pfnet/chainer-test/blob/5084edbf8af6e89bb06e314fded6fb5d954d22a2/run_test.py#L61

@unnonouno unnonouno changed the base branch from _v2 to master July 19, 2017 13:20
@unnonouno unnonouno force-pushed the _v2-protobuf-3-only branch from 3322d59 to 15a43ff Compare July 19, 2017 13:21
@unnonouno
Copy link
Member

I updated this PR

@unnonouno unnonouno force-pushed the _v2-protobuf-3-only branch from 15a43ff to 46d777a Compare July 31, 2017 05:34
@niboshi niboshi self-assigned this Aug 6, 2019
@kmaehashi
Copy link
Member

protobuf 2 support has dropped in Chainer v2.0 chainer/chainer#3090

@niboshi
Copy link
Member

niboshi commented Aug 6, 2019

Jenkins, test this please

@chainer-ci
Copy link
Member

Jenkins CI test (for commit 1899c64, target branch master) succeeded!

@niboshi niboshi merged commit 8c75286 into master Aug 6, 2019
@niboshi niboshi deleted the _v2-protobuf-3-only branch August 6, 2019 09:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants