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.

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

Update combined.py #7541

Merged
merged 3 commits into from
Aug 1, 2023
Merged

Update combined.py #7541

merged 3 commits into from
Aug 1, 2023

Commits on Jul 11, 2023

  1. Update combined.py

    from my understanding, the `check_repeated_memory_variable` validator will raise an error if any of the variables in the `memories` list are repeated. However, the `load_memory_variables` method does not check for repeated variables. This means that it is possible for the `CombinedMemory` instance to return a dictionary of memory variables that contains duplicate values. This code will check for repeated variables in the `data` dictionary returned by the `load_memory_variables` method of each sub-memory. If a repeated variable is found, an error will be raised.
    mzamini92 authored Jul 11, 2023
    Configuration menu
    Copy the full SHA
    af847db View commit details
    Browse the repository at this point in the history

Commits on Aug 1, 2023

  1. Configuration menu
    Copy the full SHA
    f2522e4 View commit details
    Browse the repository at this point in the history
  2. cr

    hwchase17 committed Aug 1, 2023
    Configuration menu
    Copy the full SHA
    1d8cf46 View commit details
    Browse the repository at this point in the history