Skip to content

Commit

Permalink
filter on es
Browse files Browse the repository at this point in the history
  • Loading branch information
shaowei su committed Nov 26, 2022
1 parent a0b2cb1 commit a05ff34
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -343,7 +343,7 @@ func (g *General) ConvertTrials(ts []trialsv1beta1.Trial) []*suggestionapi.Trial
if t.IsMetricsUnavailable() {
continue
}
if !t.IsObservationAvailable() {
if !t.IsObservationAvailable() && t.IsEarlyStopped() {
continue
}
trial := &suggestionapi.Trial{
Expand Down
4 changes: 1 addition & 3 deletions pkg/suggestion/v1beta1/skopt/base_service.py
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ def getSuggestions(self, trials, current_request_number):
Get the new suggested trials with skopt algorithm.
"""
logger.info("-" * 100 + "\n")
logger.info("New GetSuggestions call\n")
logger.info("New GetSuggestions call with current request number: {}\n".format(current_request_number))
skopt_suggested = []
loss_for_skopt = []
if len(trials) > self.succeeded_trials or self.succeeded_trials == 0:
Expand Down Expand Up @@ -113,8 +113,6 @@ def getSuggestions(self, trials, current_request_number):

else:
logger.error("Succeeded Trials didn't change: {}\n".format(self.succeeded_trials))
logger.error("No new suggestions could be generated, return early..\n")
return []

logger.info("Running Optimizer ask to query new parameters for Trials\n")

Expand Down

0 comments on commit a05ff34

Please sign in to comment.