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

warboy*2 should be equivalent(1) to warboy(1)*2? #52

Closed hyunsik closed 1 year ago

hyunsik commented 1 year ago

Background

warboy*2 와 같이 (N) 이 없는 경우 multi-core 모드로 동작합니다. multi-core 는 조금 더 특별한 경우기 때문에 명시적으로 특별한 notation 이 필요하다고 생각합니다.

Proposal

변경전:

변경후:

@libc-furiosa @sukyoungjeong-furiosa 논의를 시작해보고자 하는 이슈이니 편히 의견 주시면 감사하겠습니다.

sukyoungjeong-furiosa commented 1 year ago

제안 감사합니다 ㅎㅎ 원래 (*)를 고려했었는데, 말씀주신대로 separator가 여러 의미가 되면 nom parser combinator 쓰기가 안좋았던 기억이네요.

warboy처럼 parenthesis 없이 사용할때 mode를 변경하는것에는 동의합니다. 용례가 어느쪽이 많은지에 따라 2pe를 default 해도 될것같습니다. DeviceConfig::default() 하면 fused가 나와서 어느 한쪽으로 통일하면 좋을것같네요.

notation 관련해서는, warboy x 2 보다 warboy*2 로 쓰는게 너무 편해 보이네요.

hyunsik commented 1 year ago

@sukyoungjeong-furiosa 네 * 는 현재로 유지하고 default mode가 변경되면 좋을 것 같습니다.