octarobotics / lci-rmf-adapter

RMF Lift and Door Adapter for LCI
MIT License
3 stars 1 forks source link

エレベーター呼び出しpayloadのvalidation処理がほしい #7

Closed MikhailBertrand closed 5 days ago

MikhailBertrand commented 5 days ago

概要

CallElevator API利用時に、payload中のoriginationdestinationの値が同じ場合は不適切な値としてエレベーター側で処理されてしまう。

これを避けるため、下記の箇所などで、publishする前に与えられたoriginationdestinationが一致しているときは送信せずに無視するような処理を加えるべきである。

https://github.com/octarobotics/lci-rmf-adapter/blob/844edae984ed39d2ebf6650f045af09b193b0304/lci_rmf_adapter/lci_rmf_adapter/lci_client.py#L515-L541

nabeshima commented 5 days ago

プルリク有難うございます。

RMFからくるLiftRequestのバリデーションは、最初に受け付けた

lci_rmf_adapter/lci_rmf_adapter/lci_rmf_adapter.py

でやっておりますので、そちらを修正することにいたしました。

以下のコミットにてサポートしましたので、mainをpullしてご利用下さい。 https://github.com/octarobotics/lci-rmf-adapter/commit/abaf30dd10810ec6dfb2bd0a2d67c8d83c770b7c

ご指摘頂き有難うございました。

MikhailBertrand commented 5 days ago

ご対応ありがとうございました。