-
Notifications
You must be signed in to change notification settings - Fork 86
update config of example and README #608
base: master
Are you sure you want to change the base?
Conversation
@LucienZhang Test dose not passed. Please check CI :) |
This PR needs Approvals as follows.
Please choose reviewers and requet reviews! Click to see how to approve each reviewsYou can approve this PR by triggered comments as follows.
See all trigger commentsPlease replace [Target] to review target
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@LucienZhang
Thank you for the PR.
This PR include some different changes.
- Fix the documentation.
- Change config values related with CIFAR 10 image size
- Other config changes.
Can you separate PR?
And please remove change in license header.
@@ -9,79 +9,77 @@ | |||
# | |||
# Unless required by applicable law or agreed to in writing, software | |||
# distributed under the License is distributed on an "AS IS" BASIS, | |||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | |||
# WITH WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why you change this?
I recommend you to create issues. |
|
1 similar comment
|
the shape of cifar-10 should 32x32, and updated the description in README.