Skip to content
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.

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

estimateResidualVariance #34

Closed
nbauer2 opened this issue Apr 4, 2014 · 9 comments
Closed

estimateResidualVariance #34

nbauer2 opened this issue Apr 4, 2014 · 9 comments
Assignees

Comments

@nbauer2
Copy link

nbauer2 commented Apr 4, 2014

Hi,
where to find this function (estimateResidualVariance). It is called in infillCritAEI.

@jakobbossek
Copy link
Contributor

At the moment this function does not exist in the repository. Maybe the author forgot to check it in?

@berndbischl
Copy link
Member

I will check it in soon!

@berndbischl
Copy link
Member

Actually, is it is going to be in mlr

@nbauer2
Copy link
Author

nbauer2 commented Apr 7, 2014

I'm afraid, I could have deleted this function accidentally as I committed new noisy infill criteria. Because I'm sure that I sow this function before. Sorry.

@bernd: could you please commit the function again or just mail it to me.

@berndbischl
Copy link
Member

Nadja, in general, you can easily check whether you deleted something by looking at the diff of you commit in on the github webpage.

@berndbischl
Copy link
Member

Please note that estimateResidualVariance should be placed as a function mlr.

Do you have the function currently?

@jakobbossek
Copy link
Contributor

Sorry, just posted a comment to the wrong issue. 😅

Please note that estimateResidualVariance should be placed as a function mlr.
Do you have the function currently?

Your earlier comments made me think you are already working on it.

@berndbischl
Copy link
Member

IIRC I assigned this to Michel in mlr

Just wanted to make sure you are not copying it here, all OK then.

@nbauer2
Copy link
Author

nbauer2 commented Apr 28, 2014

I installed the current version of mlr package and now it works.

@nbauer2 nbauer2 closed this as completed Apr 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants