Skip to content

Enforce empty dots where applicable #434

Closed

Description

@hfrick

grid_regular.parameters() doesn't enforce empty dots but should. Let's do a check for those situations across the package.

dials/R/grids.R

Lines 64 to 73 in a8a8e92

grid_regular.parameters <- function(
x,
...,
levels = 3,
original = TRUE,
filter = NULL
) {
# test for NA and finalized
# test for empty ...
params <- x$object

Activity

  1. added a commit that references this issue on Apr 8, 2026
  2. hfrick commented on Apr 8, 2026

    @hfrick
    MemberAuthor

    On encode_unit():

    make_max_entropy_grid() and make_latin_hypercube_grid() call encode_unit() on a parameter set which may contain qual and quant parameters like this:

    dials/R/space_filling.R

    Lines 492 to 499 in e564926

    # Get back to parameter units
    sf_grid <- map2(
    params,
    sf_grid,
    encode_unit,
    direction = "backward",
    original = original
    )

    which means that original gets passed to either method and encode_unit.qual_param() currently swallows it silently.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    featurea feature request or enhancement

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions