Skip to content

Conversation

@giladgd
Copy link
Member

@giladgd giladgd commented Aug 24, 2025

Description of change

  • fix(Vulkan): context creation edge cases
  • fix: don't share loaded shared libraries between backends
  • fix(inspect gpu command): validate the loaded GPU type before printing info
  • fix: more CUDA compilation issues
  • fix: remove unused dependency
  • fix: typos
  • docs: change CUDA 12.2 to CUDA 12.4

Pull-Request Checklist

  • Code is up-to-date with the master branch
  • npm run format to apply eslint formatting
  • npm run test passes with this change
  • This pull request links relevant issues as Fixes #0000
  • There are new or updated unit tests validating the change
  • Documentation has been updated to reflect this change
  • The new commits and pull request title follow conventions explained in pull request guidelines (PRs that do not follow this convention will not be merged)

@giladgd giladgd requested a review from ido-pluto August 24, 2025 16:56
@giladgd giladgd self-assigned this Aug 24, 2025
@giladgd giladgd enabled auto-merge (squash) August 24, 2025 18:51
Copy link
Contributor

@ido-pluto ido-pluto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@giladgd giladgd merged commit 12749c0 into master Aug 25, 2025
19 checks passed
@giladgd giladgd deleted the gilad/vulkanFixes branch August 25, 2025 07:29
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