Skip to content

Add AMD MIGraphX recipe for rocm 3.5#18719

Closed
pfultz2 wants to merge 2 commits intospack:developfrom
pfultz2:migraphx
Closed

Add AMD MIGraphX recipe for rocm 3.5#18719
pfultz2 wants to merge 2 commits intospack:developfrom
pfultz2:migraphx

Conversation

@pfultz2
Copy link
Copy Markdown

@pfultz2 pfultz2 commented Sep 17, 2020

No description provided.

@adamjstewart
Copy link
Copy Markdown
Member

adamjstewart commented Sep 17, 2020

@bvanessen @srekolam @arjun-raj-kuppala @haampie can you review this?

@srekolam srekolam added the AMD label Sep 17, 2020
@arjun-raj-kuppala
Copy link
Copy Markdown
Contributor

arjun-raj-kuppala commented Sep 18, 2020

@adamjstewart
Need your inputs here. This errors out when i run 'spack install migraphx'
==> Error: Multiple providers found for 'pkgconfig': ['[email protected]%[email protected]+internal_glib arch=linux-centos7-zen', '[email protected]%[email protected] arch=linux-centos7-zen']

This goes through fine when i run 'spack install migraphx ^pkg-config'

What is the best solution for this kind of errors?

@adamjstewart
Copy link
Copy Markdown
Member

adamjstewart commented Sep 18, 2020

What is the best solution for this kind of errors?

$ spack clean -m

@srekolam
Copy link
Copy Markdown
Contributor

srekolam commented Oct 13, 2020

@adamjstewart , can you please let us know if there is any thing pending on our side ?

@adamjstewart
Copy link
Copy Markdown
Member

adamjstewart commented Oct 13, 2020

Can you resolve the merge conflict with hip?

@srekolam srekolam mentioned this pull request Oct 14, 2020
Merged
@srekolam
Copy link
Copy Markdown
Contributor

srekolam commented Oct 14, 2020

@adamjstewart , i submitted a PR 19300 that resolves this conflict and updates the checksum for rocm3.7.0 and rocm-3.8.0 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adamjstewart adamjstewart adamjstewart approved these changes

Assignees

No one assigned

Labels

AMD dependencies new-package new-version

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@pfultz2 @adamjstewart @arjun-raj-kuppala @srekolam