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

Save models from xgb.cv #795

Closed
wants to merge 84 commits into from
Closed

Save models from xgb.cv #795

wants to merge 84 commits into from

Conversation

vykhand
Copy link

@vykhand vykhand commented Feb 1, 2016

#793 Please note that this change have not been tested. I could not install xgboost from the most recent repository due to some compilation errors - missing logging.h.

@pommedeterresautee
Copy link
Member

try this to update your repository including submodule, with the following commands:

git submodule init
git submodule update

BTW, great idea

@pommedeterresautee pommedeterresautee changed the title If prediction = T, xgb.cv would also save fitted models for all the f… Save models from xgb.cv Feb 1, 2016
@vykhand
Copy link
Author

vykhand commented Feb 1, 2016

Hi! Thanks for kind words. I can't figure out what is wrong. I did
git submodule init git submodule update

before the pull request but after commit. and then again after I got the appveyor error message/.

How do I fix it now?

@pommedeterresautee
Copy link
Member

@pommedeterresautee
Copy link
Member

regarding AppVeyor, I don't think it is related to you.

@vykhand
Copy link
Author

vykhand commented Feb 1, 2016

Ah, sorry I thought you were talking about AppVeyor issue, the compilation issue is different - I am getting the exact same message as

#797

I don't think it could be related to submodules. I am getting it when installing from github from my repo and as well from dmlc.

@vykhand
Copy link
Author

vykhand commented Feb 8, 2016

Hi Guys,

I am fairly new to the process of contributing to XGB, or to any other SW to be honest.
I fail to re-install xgb from my own git to test my change, due to some problems with compilation.
Is there any other way to test?
Otherwise, may be someone would be so kind and test my change?

With respect, Andrey

phunterlau and others added 27 commits February 25, 2016 11:32
Add "mean absolute error" to metrics
Fix model save problem in YARN
cause this code test pickle the booster, so change bst2 -> bst3
[DIST] Add Distributed XGBoost on AWS Tutorial
temp compatibility with sklearn
the description field was set to what should be the long_description field -- making a bit of a mess on PyPi
Plugging in gp_xgboost_gridsearch
@tqchen tqchen closed this Feb 28, 2016
@tqchen
Copy link
Member

tqchen commented Feb 28, 2016

closing this for now, see #892 for new guideline in enhancing training API

@lock lock bot locked as resolved and limited conversation to collaborators Jan 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.