Skip to content

Commit

Permalink
added validator for feasible space distribution
Browse files Browse the repository at this point in the history
Signed-off-by: Shashank Mittal <[email protected]>

validation logic fixed

added unit test

added unit test for valid distribution

requested changes made

Update pkg/webhook/v1beta1/experiment/validator/validator.go

Co-authored-by: Yuki Iwai <[email protected]>

fmt
  • Loading branch information
shashank-iitbhu committed Aug 18, 2024
1 parent 51b246f commit 328559f
Show file tree
Hide file tree
Showing 7 changed files with 132 additions and 94 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -221,6 +221,7 @@ const (
DistributionLogUniform Distribution = "logUniform"
DistributionNormal Distribution = "normal"
DistributionLogNormal Distribution = "logNormal"
DistributionUnknown Distribution = "unknown"
)

// TrialTemplate describes structure of trial template
Expand Down
160 changes: 82 additions & 78 deletions pkg/apis/manager/v1beta1/api.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions pkg/apis/manager/v1beta1/api.proto
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,7 @@ enum Distribution {
LOG_UNIFORM = 1;
NORMAL = 2;
LOG_NORMAL = 3;
DISTRIBUTION_UNKNOWN = 4;
}

/**
Expand Down
Loading

0 comments on commit 328559f

Please sign in to comment.