Skip to content
This repository has been archived by the owner on Dec 1, 2021. It is now read-only.

Learning rate schedule #82

Merged
merged 10 commits into from
Dec 27, 2018
Merged

Conversation

ruimashita
Copy link
Contributor

@ruimashita ruimashita commented Dec 17, 2018

Change yaml filed name from learning_rate_setting to learning_rate_schedule, and change selection item's name more clearly.
Also refactoring blueoil/generate_lmnet_config.py and blueoil/templates/lmnet/classification.tpl.py and blueoil/templates/lmnet/classification.tpl.py to handle condition easily.

Motivation and Context

In order to easy understand setting yaml field and item name, changed these more clearly.

Description

  • Change yaml filed name from learning_rate_setting to learning_rate_schedule
  • Change selection item's name more clearly. from tune1, tune2, tune3, fixed to 2-step-decay, 3-step-decay, 3-step-decay-with-warmup, constant.
  • Change test/config/make_yml_config.py and test/config/*.ymlaccordingly.
  • Refactoring blueoil/generate_lmnet_config.py and blueoil/templates/lmnet/classification.tpl.py and blueoil/templates/lmnet/classification.tpl.py to handle condition easily.
  • update document: https://deploy-preview-82--blueoil.netlify.com/usage/init.html

How has this been tested?

blueoil.sh init generated config files can blueoil.sh train, for each learning_rate_schedule.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature / Optimization (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@ruimashita ruimashita changed the title [WIP] Learning rate schedule Learning rate schedule Dec 26, 2018
@ruimashita
Copy link
Contributor Author

@lm-jira @lm-konda
Please review, I changed the text more readable in ea8f63a

Copy link
Contributor

@lm-jira lm-jira left a comment

Choose a reason for hiding this comment

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

It's okay for me

@ruimashita ruimashita merged commit 8529940 into blue-oil:master Dec 27, 2018
@ruimashita ruimashita deleted the learning-rate-schedule branch December 27, 2018 09:25
@ruimashita
Copy link
Contributor Author

@lm-konda @lm-jira
Thanks 👍

ruimashita added a commit that referenced this pull request Jun 26, 2019
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.

3 participants