Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Feature - Integer support #25

Open
hkjlee109 opened this issue Jul 3, 2016 · 9 comments
Open

New Feature - Integer support #25

hkjlee109 opened this issue Jul 3, 2016 · 9 comments

Comments

@hkjlee109
Copy link
Member

hkjlee109 commented Jul 3, 2016

The traditional dice does not supports zero nor negative numbers. Our dice shall support not only the natural numbers but integers as well.

For example,

  • /dice 4d[-100, 100]
  • /dice 7d{[-100, 100], [1000, 1100]}
@jwChung
Copy link
Member

jwChung commented Jul 3, 2016

I have just added the question label meaning that your issue was reviewed but needs to be decided whether will be addressed or not.

@jwChung
Copy link
Member

jwChung commented Jul 5, 2016

@kwoolytech 저는 이것을 고려안할 이유가 없다고 생각합니다. #15 이슈와 관련되어 있는데 이 둘을 합쳐서 하나의 이슈로 정리하는 것은 어떨까요?

@myeesan
Copy link
Contributor

myeesan commented Jul 6, 2016

@kwoolytech 질문이 있습니다~

/dice 4d[-100, 100]

-100~100 까지의 범위 즉, 201 면체를 4번 던진다는 뜻인가요?

/dice 7d{[-100, 100], [1000, 1100]}

이건 [-100, 100] 범위의 주사위와 [1000, 1100]를 동시에 7번 던져서 결과는 {1, 1001}, {2, 1002}, ... , {7, 1007} 과 같은 형태로 나온다는 건가요?

@hkjlee109
Copy link
Member Author

Hi Brandon, good to see you here.
You've exactly answered yourself on the first question.
For the latter one.. actually I was expecting 7 numbers in the range of -100100 AND 10001100. (eg. {1007, -47, 73, -3, 1094, 0, 99}) but I am still open to any idea.
Seeing each element in the parenthesis as a single dice is quite interesting. I think it is worth to have a command line design discussion sometime later.

@hkjlee109
Copy link
Member Author

@jwChung I just saw your message now. Let me put this together with the issue #15.

@jwChung
Copy link
Member

jwChung commented Jul 7, 2016

저는 이것을 고려안할 이유가 없다고 생각합니다.

와 같이 말한 이유는 저는 사실 이 제안을 "[-100, 100]" "[1000, 1100]" 두 문자열을 선택대안으로 이해했습니다. 그래서 문자열 두 개이니 #15 와 같은 생각이라고 판단했는데 전혀 아니군요.

@kwoolytech #15 와 전혀 관련이 없군요. :) @myeesan 과 같이 pair 결과를 예상하는 것도 무리가 아니겠네요. 그렇다면 괄호에 대한 의미 부여를 정리할 필요가 있겠네요.

  • New Feature - Marking Dice #15 에서 {}{ 대안1, 대안2 } 과 같이 대안의 집합으로 보고 있었습니다.
  • [] 를 이 이슈에서 대안의 range [1, 100]라는 의미를 부여하고 있습니다.
  • 그러면 { [1, 100], [101, 200] } 은 어떤 의미로 해석하는 것이 좋을까요?
  • [1, 100] 표현대신 1...100 이렇게 적는 방법도 있겠네요.

@myeesan
Copy link
Contributor

myeesan commented Jul 7, 2016

@kwoolytech {[-100, 100], [1000, 1100]} 표기법에 대해 이해 했습니다. 다만, { } 내에서 한 요소만 선택하고 다시 [ ] 범위에서 하나의 수만을 선택하는 것이 일관성에서 결함이 있는 것 처럼 보여집니다.
@jwChung 그럼 #15 의 이슈와 합치는 것 보다는, #15 결정 후 고려 해야 할 이슈라고 생각되네요.

@jwChung
Copy link
Member

jwChung commented Jul 8, 2016

@myeesan 네 저도 그렇게 생각합니다. 처리되어야 할 이슈로 보고 enhancement 라벨 달아놓겠습니다.

@jwChung
Copy link
Member

jwChung commented Jul 14, 2016

1...5 를 사용하는 것이 어떨까요? 1...5 를 1,2,3,4,5로 해석된다면,
{1, 6, 7...10} 이런식의 표현이 어색하지 않겠네요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants