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

memory corruption fix for distributed data parallel version before SyncUpGlobalBestSplit #3110

Merged

Conversation

imatiach-msft
Copy link
Contributor

@imatiach-msft imatiach-msft commented May 22, 2020

memory corruption was found using valgrind

see relevant issue for debugging info:
#2981

@StrikerRUS
Copy link
Collaborator

Close-reopen to retrigger Appveyor.

@StrikerRUS StrikerRUS closed this May 22, 2020
@StrikerRUS StrikerRUS reopened this May 22, 2020
@imatiach-msft imatiach-msft force-pushed the ilmat/parallel-memory-corruption branch from 7e34e36 to baa72e5 Compare May 23, 2020 19:54
@imatiach-msft
Copy link
Contributor Author

Close-reopen to retrigger travis URL checker

@imatiach-msft
Copy link
Contributor Author

the PR previously failed on a Mac build that complained about different types passed to std::max, I added a fix for it (cast) that seems to have resolved the build issue

@imatiach-msft
Copy link
Contributor Author

@guolinke might you be able to take another look at this PR? thank you!

@imatiach-msft imatiach-msft force-pushed the ilmat/parallel-memory-corruption branch from f3282ed to 69e40f8 Compare May 26, 2020 12:03
@imatiach-msft
Copy link
Contributor Author

imatiach-msft commented May 26, 2020

close reopen for CI (URL checker failed)

URL        `https://en.wikipedia.org/wiki/Gamma_distribution#Occurrence_and_applications'

Name       `gamma-distributed'

Parent URL file:///home/travis/build/microsoft/LightGBM/docs/_build/html/Parameters.html, line 239, col 220

Real URL   https://en.wikipedia.org/wiki/Gamma_distribution#Occurrence_and_applications

Check time 3.469 seconds

D/L time   1.042 seconds

Size       345.98KB

Warning    [None] Anchor `Occurrence_and_applications' not found.

           Available anchors: `coll-download-as-rl', `footer',

           `footer-copyrightico', `footer-icons', `footer-info',

           `footer-info-copyright', `footer-info-lastmod',

           `footer-places', `footer-places-about',

           `footer-places-contact',

           `footer-places-cookiestatement',

           `footer-places-developers',

           `footer-places-disclaimer',

           `footer-places-mobileview', `footer-places-privacy',

           `footer-places-statslink', `footer-poweredbyico',

           `n-aboutsite', `n-contactpage', `n-portal',

           `n-recentchanges', `p-coll-print_export',

           `p-coll-print_export-label', `p-lang', `p-lang-label',

           `p-tb', `p-tb-label', `p-wikibase-otherprojects',

           `p-wikibase-otherprojects-label', `t-cite', `t-info',

           `t-permalink', `t-print', `t-recentchangeslinked',

           `t-specialpages', `t-upload', `t-whatlinkshere',

           `t-wikibase'.

@imatiach-msft imatiach-msft reopened this May 26, 2020
@StrikerRUS StrikerRUS merged commit 8ead7cc into microsoft:master May 26, 2020
ChipKerchner pushed a commit to ChipKerchner/LightGBM that referenced this pull request Jun 10, 2020
…ncUpGlobalBestSplit (microsoft#3110)

* memory corruption fix for distributed data parallel version before SyncUpGlobalBestSplit

* updated based on comments

* updated voting and feature parallel based on comments

* fixing macos failure

* rename variable
ChipKerchner pushed a commit to ChipKerchner/LightGBM that referenced this pull request Jun 11, 2020
…ncUpGlobalBestSplit (microsoft#3110)

* memory corruption fix for distributed data parallel version before SyncUpGlobalBestSplit

* updated based on comments

* updated voting and feature parallel based on comments

* fixing macos failure

* rename variable
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants