Skip to content

Commit

Permalink
Merge pull request VowpalWabbit#340 from arielf/master
Browse files Browse the repository at this point in the history
Additions of newer options to vw-hypersearch
  • Loading branch information
JohnLangford committed Jul 12, 2014
2 parents c552fc7 + 0007de7 commit 71c11e2
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
9 changes: 7 additions & 2 deletions utl/vw-hypersearch
Original file line number Diff line number Diff line change
Expand Up @@ -463,14 +463,17 @@ sub is_integer_option($) {
my $expects_integer =
($opt =~ qr{^-*
bs?
|autolink
|batch_sz
|bootstrap|B
|(?:csoaa|wap)(?:_ldf)?
|cb
|cb(?:ify)?
|num_children
|holdout_(?:period|after)
|initial_pass_length
|lda
|log_multi
|lrq
|nn
|oaa
|ect
Expand All @@ -479,7 +482,9 @@ sub is_integer_option($) {
|ring_size
|examples
|searn
|search
|total
|top
|ngram
|skips
$}x);
Expand Down Expand Up @@ -535,7 +540,7 @@ sub process_args {

# Some vw option arguments must be integers, if we want to
# optimize those, we must bisect while rounding to an integer
my ($option_before_pct) = ($vw_command_line =~ /(\S+)\s+%/);
my ($option_before_pct) = ($vw_command_line =~ /(\S+)\s+\S*%/);
$IntegerOpt = is_integer_option($option_before_pct);

if ($vw_command_line =~ /--passes/ and
Expand Down
3 changes: 2 additions & 1 deletion vowpalwabbit/searn_entityrelationtask.cc
Original file line number Diff line number Diff line change
Expand Up @@ -283,4 +283,5 @@ namespace EntityRelationTask {
}
delete hinfo;
}
}
}

0 comments on commit 71c11e2

Please sign in to comment.