Skip to content

Update TA#63

Merged
ryanmrichard merged 1 commit into
masterfrom
ryanmrichard-patch-1
Apr 5, 2022
Merged

Update TA#63
ryanmrichard merged 1 commit into
masterfrom
ryanmrichard-patch-1

Conversation

@ryanmrichard
Copy link
Copy Markdown
Member

Fixes #62

r2g

@ryanmrichard ryanmrichard self-assigned this Apr 5, 2022
@ryanmrichard ryanmrichard enabled auto-merge (squash) April 5, 2022 13:50
@keceli
Copy link
Copy Markdown
Contributor

keceli commented Apr 5, 2022

This fixes #62 but leads to another problem:

FAILED: CMakeFiles/test_tensorwrapper.dir/tests/tensor/scalar_tensor_wrapper.cpp.o 
/soft/libraries/mpi/openmpi/4.1.1/bin/mpicxx -DBPHASH_USE_TYPEID -DBTAS_HAS_BLAS_LAPACK=1 -DBTAS_HAS_BOOST_SERIALIZATION=1 -DBTAS_HAS_INTEL_MKL=1 -DLAPACK_COMPLEX_CPP=1 -DMADNESS_HAS_CEREAL -DMADNESS_MPI_HEADER=\"mpi.h\" -I../src -I_deps/catch2-src/single_include -I../include -I_deps/utilities-src -I_deps/parallelzone-src/include -I_deps/madworld-src/src -I_deps/madworld-build/src -I_deps/madworld-build/src/madness/world -I_deps/madworld-src/src/madness/world -I_deps/cereal-src/include -I_deps/cereal-build -I_deps/cereal-src -I_deps/bphash-src -I_deps/tiledarray-src/src -I_deps/tiledarray-build/src -I/soft/restricted/CNDA/sdk/2022.01.30.001/oneapi/mkl/20220311/mkl/include -I_deps/eigen-src -I_deps/btas-src -I_deps/blaspp-build/include -I_deps/blaspp-src/include -I_deps/lapackpp-build/include -I_deps/lapackpp-src/include -I_deps/umpire-src/src -I_deps/umpire-src/src/umpire/tpl/camp/include -I_deps/umpire-build/include -isystem /home/keceli/gpfs_keceli/nwx/boost_1_78_0 -fvisibility-inlines-hidden -std=c++17 -fmessage-length=0 -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-strong -fno-plt -O2 -ffunction-sections -pipe -isystem /home/keceli/miniconda3/include -O3 -DNDEBUG -fPIE   -Wfatal-errors -Wfatal-errors -fvisibility-inlines-hidden -std=c++17 -fmessage-length=0 -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-strong -fno-plt -O2 -ffunction-sections -pipe -isystem /home/keceli/miniconda3/include -fopenmp -std=c++17 -MD -MT CMakeFiles/test_tensorwrapper.dir/tests/tensor/scalar_tensor_wrapper.cpp.o -MF CMakeFiles/test_tensorwrapper.dir/tests/tensor/scalar_tensor_wrapper.cpp.o.d -o CMakeFiles/test_tensorwrapper.dir/tests/tensor/scalar_tensor_wrapper.cpp.o -c ../tests/tensor/scalar_tensor_wrapper.cpp
../tests/tensor/scalar_tensor_wrapper.cpp: In instantiation of ‘void C_A_T_C_H_T_E_M_P_L_A_T_E_T_E_S_T_F_U_N_C_0() [with TestType = TiledArray::DistArray<TiledArray::Tensor<double>, TiledArray::SparsePolicy>]’:
../tests/tensor/scalar_tensor_wrapper.cpp:16:1:   required from ‘void {anonymous}::ns_C_A_T_C_H_T_E_M_P_L_A_T_E_T_E_S_T_1::C_A_T_C_H_T_E_M_P_L_A_T_E_T_E_S_T_1<Types>::reg_tests() [with Types = {TiledArray::DistArray<TiledArray::Tensor<double, Eigen::aligned_allocator<double> >, TiledArray::SparsePolicy>}]’
../tests/tensor/scalar_tensor_wrapper.cpp:16:1:   required from here
../tests/tensor/scalar_tensor_wrapper.cpp:287:25: error: use of deleted function ‘tensorwrapper::tensor::detail_::LabeledTensorWrapper<tensorwrapper::tensor::TensorWrapper<tensorwrapper::tensor::field::Scalar> >& tensorwrapper::tensor::detail_::LabeledTensorWrapper<tensorwrapper::tensor::TensorWrapper<tensorwrapper::tensor::field::Scalar> >::operator=(tensorwrapper::tensor::detail_::LabeledTensorWrapper<tensorwrapper::tensor::TensorWrapper<tensorwrapper::tensor::field::Scalar> >&&)’
  287 |         A("i,j")        = mat("i,j");
      |         ~~~~~~~~~~~~~~~~^~~~~~~~~~~~

@ryanmrichard
Copy link
Copy Markdown
Member Author

@keceli I can't replicate your error and CI passes. Can anyone else replicate @keceli's error?

@keceli
Copy link
Copy Markdown
Contributor

keceli commented Apr 5, 2022

It was a problem on my side. Build is ok and tests passes on my build as well.

@ryanmrichard ryanmrichard merged commit 4102081 into master Apr 5, 2022
@ryanmrichard ryanmrichard deleted the ryanmrichard-patch-1 branch April 5, 2022 14:48
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.

Problem Building TA through TensorWrapper

2 participants