Skip to content

Fixed attribute name in code example #2054

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

Merged
merged 1 commit into from
May 14, 2018
Merged

Conversation

kenfehling
Copy link
Contributor

hyperparam_mutations

What do these changes do?

Related issue number

@robertnishihara
Copy link
Collaborator

Good catch. I'd be inclined to just change the code to match the example and use hyperparameter_mutations everywhere. cc @richardliaw

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/5373/
Test PASSed.

@richardliaw
Copy link
Contributor

I would be in favor of shorter keywords (in fact shorter like mutations would be nice). Will defer to @ericl as he wrote the PBT impl

@richardliaw richardliaw requested a review from ericl May 14, 2018 07:55
Copy link
Contributor

@ericl ericl left a comment

Choose a reason for hiding this comment

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

Looks good. The spelling seems fine for now too

@ericl ericl merged commit 19b743c into ray-project:master May 14, 2018
alok added a commit to alok/ray that referenced this pull request May 14, 2018
* master:
  Create RemoteFunction class, remove FunctionProperties, simplify worker Python code. (ray-project#2052)
  Don't crash on duplicate actor notifications (ray-project#2043)
  Fixed attribute name in code example (ray-project#2054)
  [xray] Add Travis build for testing xray on Linux. (ray-project#2047)
  Added missing comma to code example (ray-project#2050)
  Use more CPUs for testMultipleWaitsAndGets. (ray-project#2051)
  use jobid_nil (ray-project#2044)
  Fix typo in tune. (ray-project#2046)
  Fix error in api.rst. (ray-project#2048)
  Improve shared_ptr usage (ray-project#2030)
alok added a commit to alok/ray that referenced this pull request May 15, 2018
* master:
  Create RemoteFunction class, remove FunctionProperties, simplify worker Python code. (ray-project#2052)
  Don't crash on duplicate actor notifications (ray-project#2043)
  Fixed attribute name in code example (ray-project#2054)
  [xray] Add Travis build for testing xray on Linux. (ray-project#2047)
  Added missing comma to code example (ray-project#2050)
  Use more CPUs for testMultipleWaitsAndGets. (ray-project#2051)
  use jobid_nil (ray-project#2044)
  Fix typo in tune. (ray-project#2046)
  Fix error in api.rst. (ray-project#2048)
  Improve shared_ptr usage (ray-project#2030)
alok added a commit to alok/ray that referenced this pull request May 15, 2018
* fix-a3c-torch:
  Fix shape error in conv nets
  Ensure that values are flat list
  rm all use of torch Variables
  Create RemoteFunction class, remove FunctionProperties, simplify worker Python code. (ray-project#2052)
  Don't crash on duplicate actor notifications (ray-project#2043)
  rm unnecessary Variable wrapper
  Fixed attribute name in code example (ray-project#2054)
  [xray] Add Travis build for testing xray on Linux. (ray-project#2047)
  Added missing comma to code example (ray-project#2050)
  Use more CPUs for testMultipleWaitsAndGets. (ray-project#2051)
  use jobid_nil (ray-project#2044)
  Fix typo in tune. (ray-project#2046)
  Fix error in api.rst. (ray-project#2048)
  Improve shared_ptr usage (ray-project#2030)
  replace deprecated function
  Fmt
  Fix shapes of tensors
  Rename argument name to out_size
  Use correct pytorch functions
  Use F.softmax instead of a pointless network layer
@kenfehling kenfehling deleted the patch-1 branch May 16, 2018 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants