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

Add oneAPI based container #353

Merged
merged 32 commits into from
Sep 10, 2021
Merged

Add oneAPI based container #353

merged 32 commits into from
Sep 10, 2021

Conversation

caic99
Copy link
Member

@caic99 caic99 commented Sep 9, 2021

This PR implements #156 , and contains other fixes.

  • Add Dockerfile.intel
  • Move the original Dockerfile to Dockerfile.gnu
  • Add github hosted test pipeline: build abacus on push, but not running it
  • Revised .devcontainer: use docker image link directly; add default extensions

@caic99
Dockerfile: remove boost
7703921 
Remove libboost since we are not using boost anymore.
Using built cmake binary.
@caic99
Add github hosted test pipeline for forks.
2d7a5f6
@caic99
Run github hosted test on self docker image
6208c49
@caic99
Merge branch 'deepmodeling:reconstruction' into reconstruction
fb1a4b2
@caic99
build testing
6990ae7
@caic99
Login to ghcr.io
1623c7c
@caic99
Update hosted_test.yml
9bd7584
@caic99
Separate build and test
d782fec
@caic99
Remove test
aa40fef 
Running testing is extremely slow, keep building only.
@caic99
Use cmake to compile OpenBLAS
72a47ab
@caic99
Update Dockerfile
95e684d
@caic99
Fix Openblas install prefix
72fcf9a
@caic99
Fix OpenBLAS install path
2bf6c67
@caic99
Merge branch 'deepmodeling:reconstruction' into reconstruction
d80e4d0
@caic99
Merge branch 'deepmodeling:reconstruction' into reconstruction
9d23581
@caic99
Add Intel oneAPI Dockerfile support.
ce58745
@caic99
Reverting changes attempting to use cmake.
3c9b6e1
@caic99
Workflow now builds both images.
9917c01
@caic99
Fix dockerfile path in container.yml
f6b856b
@caic99
Remove CONTAINER env. Replacing it with given name using tags.
a28d96f
@caic99
Devcontainer now use pre-built docker image directly.
4454a47
@caic99
Fix: use ghcr.io rather than default docker.io
953e0c8
@caic99
Use former abacus-development-kit as docker image tag prefix.
11cbad6
@caic99
Use recommended method to log in ghcr.io .
a685fa2 
See https://github.com/docker/login-action .
@caic99 caic99 added this to In progress in ABACUS Continuous Integration via automation Sep 9, 2021
@caic99 caic99 moved this from In progress to Review in progress in ABACUS Continuous Integration Sep 9, 2021
@caic99 caic99 marked this pull request as draft September 9, 2021 09:58
@caic99
Copy link
Member Author

caic99 commented Sep 9, 2021

Integration test result using Dockerfile.intel: In case 111_PW_S2_elec_add, totalstressref cal=2327.68737000 ref=2327.68716500 deviation=-0.00020500. This is an acceptable deviation.

@caic99 caic99 marked this pull request as ready for review September 9, 2021 10:46
@caic99 caic99 requested a review from dyzheng September 9, 2021 10:46
@dyzheng dyzheng merged commit 872c262 into deepmodeling:reconstruction Sep 10, 2021
ABACUS Continuous Integration automation moved this from Review in progress to Done Sep 10, 2021
@caic99 caic99 linked an issue Sep 11, 2021 that may be closed by this pull request
Intel OneAPI based Container #156
Closed
@caic99 caic99 self-assigned this Sep 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dyzheng dyzheng Awaiting requested review from dyzheng

Assignees

@caic99 caic99

Labels
None yet
Projects
No open projects
ABACUS Continuous Integration
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Intel OneAPI based Container
2 participants
@caic99 @dyzheng