Skip to content
This repository was archived by the owner on Aug 16, 2023. It is now read-only.

[ADD] odoo-shippable: add env var to avoid numpy weird errors#401

Merged
moylop260 merged 1 commit intoVauxoo:masterfrom
vauxoo-dev:master-fix-numpy-weird-errors-fer
Apr 19, 2021
Merged

[ADD] odoo-shippable: add env var to avoid numpy weird errors#401
moylop260 merged 1 commit intoVauxoo:masterfrom
vauxoo-dev:master-fix-numpy-weird-errors-fer

Conversation

@fernandahf
Copy link
Copy Markdown
Contributor

Description of the issue/feature this PR addresses:

Currently, if numpy is available in the modules even if you are not using it
Odoo try to compile and the system is down only for a type of processor

Currently we know 2 server reproducing the error:

B&F-production
Runbot

More info about:

numpy/numpy#17674
numpy/numpy#17759

It is reproducing in the following MR:

https://git.vauxoo.com/vauxoo/lasec/-/merge_requests/197

Check the following discussion https://odoo-community.org/groups/contributors-15/contributors-186006?mode=thread&date_begin=&date_end=

OpenBLAS creates a number of threads equal to the number of core threads available: 56 in my case (production server),
so it quickly reached limit_memory_hard
and the process was killed (SIGSEGV)
Forcing OPENBLAS_NUM_THREADS=1 fixed the issue.

Current behavior before PR:
Weird errors related to numpy.

Desired behavior after PR is merged:
No errors related to numpy.

-- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Currently, if numpy is available in the modules even if you are not using it
Odoo try to compile and the system is down only for a type of processor

Currently we know 2 server reproducing the error:

    B&F-production
    Runbot

More info about:

numpy/numpy#17674
numpy/numpy#17759

It is reproducing in the following MR:

https://git.vauxoo.com/vauxoo/lasec/-/merge_requests/197

Check the following discussion https://odoo-community.org/groups/contributors-15/contributors-186006?mode=thread&date_begin=&date_end=

OpenBLAS creates a number of threads equal to the number of core threads available: 56 in my case (production server),
so it quickly reached limit_memory_hard
and the process was killed (SIGSEGV)
Forcing OPENBLAS_NUM_THREADS=1 fixed the issue.
@fernandahf fernandahf force-pushed the master-fix-numpy-weird-errors-fer branch from 4718495 to 6e7d60f Compare April 19, 2021 21:04
@fernandahf
Copy link
Copy Markdown
Contributor Author

@moylop260

Could you review this, please?

Regards.

@moylop260 moylop260 merged commit 5d6171e into Vauxoo:master Apr 19, 2021
@moylop260 moylop260 deleted the master-fix-numpy-weird-errors-fer branch April 19, 2021 21:29
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.

2 participants