Skip to content
This repository has been archived by the owner on Sep 28, 2021. It is now read-only.

Bump CMake to 3.12 #568

Merged
merged 6 commits into from
Oct 11, 2020
Merged

Bump CMake to 3.12 #568

merged 6 commits into from
Oct 11, 2020

Conversation

junghans
Copy link
Member

Only for master, stable will still be on CMake-3.10.

@codecov
Copy link

codecov bot commented Oct 10, 2020

Codecov Report

Merging #568 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #568   +/-   ##
======================================
  Coverage    57.2%   57.2%           
======================================
  Files         312     312           
  Lines       32434   32434           
======================================
  Hits        18565   18565           
  Misses      13869   13869           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5e9d254...5d3242e. Read the comment docs.

@junghans
Copy link
Member Author

@JensWehner:

157/225 Test #157: unit_test_bse ......................................***Failed    0.03 sec
Running 1 test case...
CUDA Runtime Error: forward compatibility was attempted on non supported HW
CUDA Runtime Error: forward compatibility was attempted on non supported HW
CUDA Runtime Error: forward compatibility was attempted on non supported HW
unknown location(0): fatal error: in "bse_test/bse_hamiltonian": std::runtime_error: Error copy arrays to device
/home/gitlab-runner/builds/enfdGEYs/0/votca/xtp/votca/xtp/src/tests/test_bse.cc(36): last checkpoint: "bse_hamiltonian" test entry

*** 1 failure is detected in the test module "bse_test"

all GPU tests fail.

@junghans
Copy link
Member Author

Or should we just disable gitlab CI for now?

@JensWehner
Copy link
Member

Yes pls, @baumeier has to update his hardware.

@junghans junghans marked this pull request as ready for review October 10, 2020 18:31
@baumeier
Copy link
Member

@JensWehner So, you and @felipeZ has not done anything on that machine regarding github action instead of Gitlab CI? I can upgrade the machine then.

@JensWehner
Copy link
Member

@baumeier Yes, we have not done anything. We (okay mainly @felipeZ ) know what do. So go ahead.

@JensWehner JensWehner merged commit 2ce17ea into master Oct 11, 2020
@JensWehner JensWehner deleted the newer_cmake branch October 11, 2020 10:53
votca-bot added a commit to votca/votca that referenced this pull request Oct 11, 2020
@baumeier
Copy link
Member

@JensWehner OS updated, do whatever you must do

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants