Skip to content

Conversation

@robik75
Copy link
Member

@robik75 robik75 commented May 27, 2025

What ❔

This PR contains changes needed to make it possible to use multiple GPUs for proving from the same process.

Is this a breaking change?

  • [x] Yes - implicit host allocator creation was removed from ProverContext constructor and needs to be called explicitly
  • No

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Code has been formatted.

@robik75 robik75 self-assigned this May 27, 2025
@robik75 robik75 requested review from EmilLuta and mm-zk May 27, 2025 11:09
@robik75 robik75 force-pushed the rr/multiple_gpus branch from 1de6ca0 to 93b1f19 Compare May 28, 2025 14:51
@robik75 robik75 force-pushed the rr/multiple_gpus branch from 7d106b7 to 5670443 Compare May 29, 2025 07:48
@robik75 robik75 enabled auto-merge (squash) May 29, 2025 09:03
@robik75 robik75 merged commit cf5d00f into main May 29, 2025
17 checks passed
@robik75 robik75 deleted the rr/multiple_gpus branch May 29, 2025 09:05
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.

2 participants