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

[SOT][3.11] Increase ref of func to avoid segmentation fault #58424

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

SigureMo
Copy link
Member

@SigureMo SigureMo commented Oct 26, 2023

PR types

Bug fixes

PR changes

Others

Description

修复 3.11 分支没有 INCREF func 导致错误释放进而导致的段错误

PCard-66972

@paddle-bot
Copy link

paddle-bot bot commented Oct 26, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Oct 26, 2023
@SigureMo SigureMo requested a review from zrr1999 October 26, 2023 14:52
@SigureMo SigureMo changed the title [SOT] Increase ref of func to avoid segmentation fault [SOT][3.11] Increase ref of func to avoid segmentation fault Oct 26, 2023
@SigureMo SigureMo merged commit 672752c into PaddlePaddle:develop Oct 26, 2023
28 checks passed
@SigureMo SigureMo deleted the sot/fix-311-func-not-inc branch October 26, 2023 17:03
@madhavi1102
Copy link

@SigureMo Appreciate if you can help me how to avoid segmentation fault error. I am trying to train text detection db_resnet50 model on icdar dataset but each time training is stopped with error Python segmentation Fault.

What all conditions should I pre check before training.
What is the solution to avoid this issue.

Waiting for your response,

Thank you

@paddle-bot paddle-bot bot removed the contributor External developers label Nov 3, 2023
danleifeng pushed a commit to danleifeng/Paddle that referenced this pull request Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants