Skip to content

MemoizedBmi has incorrect set_value_at_indices kwarg #155

@BSchilperoort

Description

@BSchilperoort

MemoizedBmi has the following method set_value_at_indices

    def set_value_at_indices(self, var_name, indices, src):
        return self.origin.set_value_at_indices(var_name, indices, src)

However, the BMI spec (and the OptionalDestBmi & BmiClient) has the following method signature:

    def set_value_at_indices(self, name, inds, src):

If a user uses kwargs with the memoized bmi this will go wrong.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions