Skip to content

[mpi] update to msmpi10.1 and openmpi4.0.3#11321

Closed
cenit wants to merge 7 commits intomicrosoft:masterfrom
cenit:dev/cenit/msmpi_vcpkg
Closed

[mpi] update to msmpi10.1 and openmpi4.0.3#11321
cenit wants to merge 7 commits intomicrosoft:masterfrom
cenit:dev/cenit/msmpi_vcpkg

Conversation

@cenit
Copy link
Contributor

@cenit cenit commented May 12, 2020

As per title

@cenit
Copy link
Contributor Author

cenit commented May 12, 2020

@BillyONeal this update requires an update also on the CI machines. Any chance for this?
edit: otherwise we can just let it fail on CI, but since now it passes it would be nice to continue to do so :)

@cenit cenit changed the title [msmpi] update to 10.1 [mpi] update to msmpi10.1 and openmpi4.0.3 May 12, 2020
@cenit cenit marked this pull request as draft May 12, 2020 16:08
@BillyONeal
Copy link
Member

@cenit Include edits to the scripts that spit out the infrastructure and we might be able to deploy that :)

@cenit
Copy link
Contributor Author

cenit commented May 12, 2020

I tried to update image provisioning scripts. Please let me know. Thanks for the quick reply!

@cenit cenit marked this pull request as ready for review May 12, 2020 17:46
@MVoz
Copy link
Contributor

MVoz commented May 12, 2020

@cenit
Copy link
Contributor Author

cenit commented May 12, 2020

I think using intelmpi from here might be more easy:
https://anaconda.org/intel/impi_rt/files

can you confirm @voskrese that it is what you need?

@MVoz
Copy link
Contributor

MVoz commented May 12, 2020

yes + lib https://anaconda.org/intel/impi-devel

hmm... NuGet impi.lib + impicxx.lib , anaconda only impi.lib

2020-05-13_033826

@cenit
Copy link
Contributor Author

cenit commented May 12, 2020

let's discuss it anyway in another issue and for another PR, which will propose a new port (intelmpi). Ok? @voskrese

@MVoz
Copy link
Contributor

MVoz commented May 12, 2020

Ok

@BillyONeal BillyONeal added the depends:vm-update PR contains changes to the VM provisioning scripts label May 12, 2020
@LilyWangL LilyWangL requested a review from strega-nil May 13, 2020 06:17
@JackBoosY
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

BillyONeal pushed a commit to BillyONeal/vcpkg that referenced this pull request Jun 11, 2020
# Conflicts:
#	scripts/azure-pipelines/windows/provision-image.ps1
@BillyONeal
Copy link
Member

@cenit Thanks for your contribution! The changes from this PR were entirely merged by #11365 (which included a deploy of the VMs)

@BillyONeal BillyONeal closed this Jun 11, 2020
@BillyONeal BillyONeal removed the depends:vm-update PR contains changes to the VM provisioning scripts label Jun 11, 2020
@cenit
Copy link
Contributor Author

cenit commented Jun 11, 2020

Thank you!

@cenit cenit deleted the dev/cenit/msmpi_vcpkg branch June 15, 2020 08:36
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.

6 participants