DummyObservationBuilder fix - Githubissues

flatland-association / flatland-rl

The Flatland Framework is a multi-purpose environment to tackle problems around resilient resource allocation under uncertainty. It is designed to be a flexible and method agnostic to solve a wide range of problems in the field of operations research and reinforcement learning.

https://www.flatland-association.org/

MIT License

24 stars 8 forks source link

DummyObservationBuilder fix #23

Open aiAdrian opened 1 year ago

aiAdrian commented 1 year ago

The DummyObservationBuilder returns true instead of a List: get_many

Objectives

` class DummyObservationBuilder(ObservationBuilder):

def __init__(self):
    super().__init__()

def reset(self):
    pass

def get_many(self, handles: Optional[List[int]] = None) -> bool:
    return True

def get(self, handle: int = 0) -> bool:
    return True`

...

Todos

[ ] Check where the DummyObservationBuilder is used in Flatland
[ ] Correct the get method, that for each Agent a List (np.array) is returned
[x] Correct the get_many method, such as it returns a list of [ get ( ...) , ... ] for each agent one observation

Minimal testing requirements

[ ] Test the use of DummyObservationBuilder is all found Flatland parts
[x] Write a unit test