Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Rename the observations variable in the evaluation util to avoid shadowing #1288

Merged
merged 6 commits into from
Apr 11, 2023
Merged

Rename the observations variable in the evaluation util to avoid shadowing #1288

merged 6 commits into from
Apr 11, 2023

Commits on Jan 18, 2023

  1. Rename the observations variable in the evaluation util to avoid shad… 

    …owing

This enables a callback in evaluate_policy to have access to the
observation vector that is fed to the environment step function,
which is currently shadowed by the output observation.
    @npit
    npit committed Jan 18, 2023
    Configuration menu
    Copy the full SHA
    3d3f454 View commit details
    Browse the repository at this point in the history

Commits on Apr 3, 2023

  1. Merge branch 'master' into fix-rename

    @araffin
    araffin authored Apr 3, 2023
    Configuration menu
    Copy the full SHA
    f5cedca View commit details
    Browse the repository at this point in the history

Commits on Apr 9, 2023

  1. Update changelog

    @npit
    npit committed Apr 9, 2023
    Configuration menu
    Copy the full SHA
    118a4d5 View commit details
    Browse the repository at this point in the history

Commits on Apr 11, 2023

  1. Merge branch 'master' into fix-rename

    @araffin
    araffin committed Apr 11, 2023
    Configuration menu
    Copy the full SHA
    1eb3aa8 View commit details
    Browse the repository at this point in the history

  2. Add test

    @araffin
    araffin committed Apr 11, 2023
    Configuration menu
    Copy the full SHA
    308578f View commit details
    Browse the repository at this point in the history

  3. Move assignment outside of the loop

    @araffin
    araffin committed Apr 11, 2023
    Configuration menu
    Copy the full SHA
    067c5df View commit details
    Browse the repository at this point in the history