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.

Sign up for GitHub

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

Jump to bottom

sci-libs/vtk: reduce required memory for cuda compilation #31487

Closed
wants to merge 2 commits into from
Closed

sci-libs/vtk: reduce required memory for cuda compilation #31487

wants to merge 2 commits into from

Conversation

negril
Copy link
Contributor

@negril negril commented Jun 16, 2023

Prior logic assumes infinite parallel nvcc calls, while real-life testing shows a max of 4.
This adds crude logic to require no more memory then needed for 4 parallel calls.

Bug: https://bugs.gentoo.org/901241

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @negril
Areas affected: ebuilds
Packages affected: sci-libs/vtk

sci-libs/vtk: @gentoo/sci, @waebbl, @gentoo/proxy-maint

Linked bugs

Bugs linked: 901241

In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Jun 16, 2023
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-06-16 16:55 UTC
Newest commit scanned: b814257
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/c201c4f712/output.html

@negril
sci-libs/vtk: reduce required memory for cuda compilation
11c8639 
Prior logic assumes infinite parallel nvcc calls, while real-life
testing shows a max of 4.
This adds crude logic to require no more memory then needed for 4
parallel calls.

Bug: https://bugs.gentoo.org/901241

Signed-off-by: Paul Zander <negril.nx+gentoo@gmail.com>
@negril
sci-libs/vtk: add 9.2.6
3b3f4c1 
Signed-off-by: Paul Zander <negril.nx+gentoo@gmail.com>
@negril
Copy link
Contributor Author

negril commented Jun 22, 2023

Is there anything else I need to provide to get this merged?

@negril
Copy link
Contributor Author

negril commented Jun 26, 2023

@waebbl can give feedback on this?

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-06-28 10:20 UTC
Newest commit scanned: bf07be4
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/71271ea5f2/output.html

@negril negril closed this Jun 28, 2023
@negril negril deleted the vtk-memlimit branch June 28, 2023 10:49
@negril negril reopened this Jun 28, 2023
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-06-28 12:00 UTC
Newest commit scanned: 3b3f4c1
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/74a0cc4ff0/output.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR.
Projects
None yet
Milestone
No milestone
3 participants
@negril @gentoo-bot @gentoo-repo-qa-bot