Skip to content

supported types in runs/function.py, tuple as supported type missing #373

Closed

Description

@bstrang

model dependent attributes for trace arff

for key in model.cv_results_:
    if key.startswith('param_'):
        # supported types should include all types, including bool, int float
        supported_types = (bool, int, float, six.string_types)
        if all(isinstance(i, supported_types) or i is None for i in model.cv_results_[key]):
            type = 'STRING'
        else:
            raise TypeError('Unsupported param type in param grid')

        # we renamed the attribute param to parameter, as this is a required
        # OpenML convention
        attribute = ("parameter_" + key[6:], type)
        trace_attributes.append(attribute)
return trace_attributes

Activity

  1. added a commit that references this issue on Dec 18, 2017
    6147c4f
  2. added a commit that references this issue on Mar 13, 2018
    f4387d6
  3. janvanrijn commented on Mar 13, 2018

    @janvanrijn
    Member

    @bstrang please be informed that the PR is merged. Still tuple values are not allowed, but you can insert them as list of integers (and they will be treated as tuples)

  4. mfeurer commented on Mar 14, 2018

    @mfeurer
    Collaborator

    @bstrang @janvanrijn can this be closed?

  5. janvanrijn commented on Mar 14, 2018

    @janvanrijn
    Member

    Please leave it open for one more week, I'll try to think of the suggested unit test that we both couldn't remember anymore

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

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions