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

improved error message when identical PS exists #678

Merged
merged 2 commits into from
Jan 30, 2020

Conversation

yohm
Copy link
Member

@yohm yohm commented Jan 30, 2020

When an invalid parameter is given, an error message indicating the invalid field is shown.
When an identical PS is specified, an error message indicating the duplicate of PS is shown.

Error message when identical PS exists.
image

Error message when invalid value is given.
image

When an invalid parameter is given, show error message indicating
which field is invalid.
When an identical PS is specified, an error message indicating the
duplicate of PS is shown.
@yohm yohm added the bug label Jan 30, 2020
@yohm yohm added this to the v3.8.0 milestone Jan 30, 2020
@yohm yohm self-assigned this Jan 30, 2020
@takeshi-uchitane
Copy link
Contributor

@yohm コード確認しました.マージしてください.

過去仕様だと,一部のPSが作成されてしまっていましたが,新仕様では全てのPSが作成されるような値が与えられるまでPSは作成されないのですね.

過去仕様だとパラメータ空間の空きスペースにPSを挿入できたのですが,WebUIからの操作ではそこまで複雑な作成はしないと考えますので,新仕様で良いと思います.
(PSの重複を許して空いているところだけPSを作るのはCLIでできるので.)

@yohm
Copy link
Member Author

yohm commented Jan 30, 2020

@takeshi-uchitane reviewありがとうございます。

過去仕様だと,一部のPSが作成されてしまっていましたが,新仕様では全てのPSが作成されるような値が与えられるまでPSは作成されないのですね.

はい。そこも説明しておけばよかったですね。これまでは不正な値がコンマ区切りの中に入っていても無視していました。例えばIntegerのところに1,2,fooなどと書いても、[1,2]については作られていました。
今回はIntegerに変換不能な値が一つでも含まれていたらエラーになるようになっています。

ちなみに一部だけ重複している場合には、今まで通り重複していない箇所についてPSを作ります。
例えば、すでにp0=1のPSが存在しているとして、p0に1,2,3と指定すると、p0=1はスキップし、p0=2,p0=3のPSについては新規作成します。

@yohm yohm merged commit 9daa943 into develop Jan 30, 2020
@yohm yohm deleted the error_message_when_PS_exists branch January 30, 2020 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants