search

furiosa-ai / device-api

APIs that offers NPU devices' information and allow to control the devices
Apache License 2.0
5 stars 8 forks source link

Implement metadata access: alive, atr_error, heartbeat #45

Closed libc-furiosa closed 2 years ago

libc-furiosa commented 2 years ago

Implement some management APIs for #44

libc-furiosa commented 2 years ago

@sukyoungjeong-furiosa 네, 저도 같은 지점에서 비슷한 생각을 했었습니다. 일전에 논의된 바와 같이, API 레벨에서는 Result로 에러를 명확하게 표현하고 API를 사용하는 측에서 용도에 맞게끔 처리하는 것이 좋을 것 같습니다. 별도 PR에서 정비 부탁드리겠습니다. 감사합니다.

libc-furiosa commented 2 years ago

@sukyoungjeong-furiosa #46 의 변경사항을 적용하여 코드를 업데이트 하였습니다. (Option -> DeviceResult)

한번만 더 확인 부탁드립니다. 감사합니다.

libc-furiosa commented 2 years ago

@sukyoungjeong-furiosa 아 참, 그 작업을 마지막에 하려고 미뤄두고 있었습니다. 깜빡할 뻔 했네요. 짚어주셔서 감사합니다. 문서 작성하고 다시 리뷰 요청 드릴게요.

libc-furiosa commented 2 years ago

@sukyoungjeong-furiosa 업데이트 하였습니다. 리뷰 부탁드립니다. (관사 선택이 어렵네요)

sukyoungjeong-furiosa commented 2 years ago

수고하셨습니다!