Add tests for transformers (!27) · Merge requests · bob / bob.pipelines · GitLab

Amir MOHAMMADI added 1 commit 5 years ago

added 1 commit

3c842a5f - introduce SampleBatch

Compare with previous version

Amir MOHAMMADI added 1 commit 5 years ago

added 1 commit

4edc65de - introduce SampleBatch

Compare with previous version

Amir MOHAMMADI mentioned in issue #14 (closed) 5 years ago

mentioned in issue #14 (closed)

Tiago de Freitas Pereira resolved all threads 5 years ago

resolved all threads

Tiago de Freitas Pereira @tiago.pereira started a thread on the diff 5 years ago

bob/pipelines/wrappers.py

         # if being wrapped with DaskWrapper, add ToDaskBag to the steps
         if DaskWrapper in bases:
             valid_params = ToDaskBag._get_param_names()
             params = {k: kwargs.pop(k) for k in valid_params if k in kwargs}

Tiago de Freitas Pereira mentioned in merge request !29 (closed) 5 years ago

mentioned in merge request !29 (closed)

Tiago de Freitas Pereira @tiago.pereira 5 years ago

Owner

That's very nice. LGTM

thanks

Tiago de Freitas Pereira merged 5 years ago

merged

Tiago de Freitas Pereira mentioned in commit 5 years ago

mentioned in commit 617867b5

Please register or sign in to reply