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

[Roadmap] 1.3.2 Patch release #6571

Closed
6 tasks done
trivialfis opened this issue Jan 3, 2021 · 24 comments
Closed
6 tasks done

[Roadmap] 1.3.2 Patch release #6571

trivialfis opened this issue Jan 3, 2021 · 24 comments

Comments

@trivialfis
Copy link
Member

trivialfis commented Jan 3, 2021

We are looking to release a patch release 1.3.2.

This issue will remain open until January 11, 2021.

@dmlc/xgboost-committer

Pull requests:

@trivialfis trivialfis pinned this issue Jan 3, 2021
@trivialfis
Copy link
Member Author

@hcho3 All merged now.

@hcho3
Copy link
Collaborator

hcho3 commented Jan 12, 2021

@trivialfis Great! Why don't you give a shot at making the patch release? Use #6543 as an example.

@trivialfis
Copy link
Member Author

Thanks for the suggestion, let me do that tomorrow.

@trivialfis
Copy link
Member Author

@hcho3 @hetong007 I created a release page. Next would be submitting packages to pip and cran.

@hetong007
Copy link
Member

@trivialfis I uploaded this branch onto R-hub, and it still reports error: https://builder.r-hub.io/status/xgboost_1.3.2.1.tar.gz-f47cc590f1434dddb4c4b89cd02703dc

This error is from R-hub with platform Oracle Solaris 10, x86, 32 bit, R-release, Oracle Developer Studio 12.6. This matches the compiler of CRAN Solaris: https://cran.r-project.org/web/checks/check_flavors.html#r-patched-solaris-x86

@trivialfis
Copy link
Member Author

Em .. failing to install

@hcho3
Copy link
Collaborator

hcho3 commented Jan 14, 2021

image The link shows success?

@trivialfis
Copy link
Member Author

Looked a little bit closer:

4118#> * checking for sufficient/correct file permissions ... OK

4119#> * checking whether package ‘xgboost’ can be installed ... ERROR

4120#> Installation failed.

4121#> See ‘/export/home/XnXaLcc/xgboost.Rcheck/00install.out’ for details.

@trivialfis
Copy link
Member Author

I don't know if there's a way to retrieve that log.

@hetong007
Copy link
Member

@trivialfis
Copy link
Member Author

Got it passed: https://builder.r-hub.io/status/xgboost_1.3.2.1.tar.gz-72aa615847ba65333ecfea61c676b7b4 Will open a PR.

@trivialfis
Copy link
Member Author

#6606

@hetong007
Copy link
Member

I patched this PR and submitted it to R-hub again, but got an error in installation: https://artifacts.r-hub.io/xgboost_1.3.2.1.tar.gz-5c8a82e1e66188afea79817c398575fd/xgboost.Rcheck/00install.out

@trivialfis
Copy link
Member Author

std::ctype<char>::id ? @hetong007 what's your configuration of testing?

@hetong007
Copy link
Member

My configuration on r-hub is:

  • Visit https://builder.r-hub.io/advanced
  • Select Oracle Solaris 10, x86, 32 bit, R-release, Oracle Developer Studio 12.6 for the platform -- This is the CRAN Soalris config.
  • Upload the package, configure the email address
  • Start the check and wait.

I can submit again to see if this is a false alarm, will update this thread if ends up with a different result.

@trivialfis
Copy link
Member Author

I hope that we don't have to bisect it on R Hub ....

@trivialfis
Copy link
Member Author

trivialfis commented Jan 15, 2021

I tried the old 1.2 release, which is also failing due to type check (removed in 1.3): https://artifacts.r-hub.io/xgboost_1.2.0.1.tar.gz-1fa6b58859caf51810b85692f5e870c6/xgboost.Rcheck/00install.out

@hetong007
Copy link
Member

Seems the current CRAN error is not the same as what we have on r-hub: https://www.r-project.org/nosvn/R.check/r-patched-solaris-x86/xgboost-00install.html

@trivialfis
Copy link
Member Author

@hetong007 That was fixed in #6578

@trivialfis
Copy link
Member Author

At latest 1.3 branch, all compiler related workarounds/fixes are applied. But the linker issue you found #6571 (comment) is still there, and I'm not sure how to remove it. The std::ctype<char>::id seems to be used in std stream, but I'm not sure which part of xgboost is triggering that error.

@trivialfis
Copy link
Member Author

@hetong007 Can we just submit the package?

@hetong007
Copy link
Member

Well if a fix is not easily accessible, I can try to submit. That's at least an action trying to patch.

@trivialfis
Copy link
Member Author

trivialfis commented Jan 18, 2021

@hetong007 Thanks! Please keep me in the loop if there's any insight on the linker error. I wonder if it's reproducible on CRAN. Since of the 2 Solaris configurations on R Hub, the one that without oracle developer studio is passing.

@trivialfis
Copy link
Member Author

@hcho3 All wheels are uploaded to pypi. Many thanks for the help!

@trivialfis trivialfis unpinned this issue Jan 19, 2021
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