Skip to content

new fun splitVecParams NOT DONE #151

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

new fun splitVecParams NOT DONE #151

wants to merge 4 commits into from

Conversation

@berndbischl
Copy link
Member

@berndbischl berndbischl commented Dec 22, 2016

No description provided.

@coveralls
Copy link

coveralls commented Dec 22, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-0.4%) to 94.444% when pulling 90eb0b3 on split_vec_par into c0013d7 on master.

@jakobbossek
Copy link
Contributor

jakobbossek commented Dec 22, 2016

Maybe rename to splitVectorParams? Seems to be more consistent with naming throughout ParamHelpers.

@coveralls
Copy link

coveralls commented Dec 23, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-0.06%) to 94.761% when pulling 1ad9f0e on split_vec_par into c0013d7 on master.

@jakob-r jakob-r changed the title new fun splitVecParams NOLT DONE new fun splitVecParams NOT DONE Jan 5, 2017
@jakob-r jakob-r mentioned this pull request Jan 5, 2017
Open
@jakob-r
Copy link
Member

jakob-r commented Jan 5, 2017

What's the status here?

@jakob-r
Copy link
Member

jakob-r commented May 12, 2017

We could need an update here to work on mlr-org/mlrMBO#257

jakob-r added 2 commits August 9, 2017 11:32
@jakob-r
Merge branch 'master' into split_vec_par
3b6f0fd 
# Conflicts:
#	NAMESPACE
#	R/getTypeStrings.R
#	man/getTypeStrings.Rd
@jakob-r
roxygen
22d3285
@jakob-r
Copy link
Member

jakob-r commented Aug 9, 2017

@berndbischl What's the status?

@jakob-r
Copy link
Member

jakob-r commented Nov 7, 2017

@berndbischl Why is this not done? Looks fine to me.

@jakob-r jakob-r added this to the v1.12 milestone Jun 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v1.12

Development

Successfully merging this pull request may close these issues.

5 participants

@berndbischl @coveralls @jakobbossek @jakob-r