Skip to content

Commit

Permalink
Merge branch 'main' into tsvd_debug
Browse files Browse the repository at this point in the history
  • Loading branch information
terasakisatoshi authored Nov 7, 2024
2 parents 0a528bd + ecd1926 commit ad98e33
Show file tree
Hide file tree
Showing 3 changed files with 43 additions and 2 deletions.
41 changes: 41 additions & 0 deletions .github/workflows/CI_cmake.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
# This starter workflow is for a CMake project running on a single platform. There is a different starter workflow if you need cross-platform coverage.
# See: https://github.com/actions/starter-workflows/blob/main/ci/cmake-multi-platform.yml
name: CMake on a single platform

on:
push:
branches: [ "main" ]
pull_request:
branches: [ "main" ]

env:
# Customize the CMake build type here (Release, Debug, RelWithDebInfo, etc.)
BUILD_TYPE: Release

jobs:
build:
# The CMake configure and build commands are platform agnostic and should work equally well on Windows or Mac.
# You can convert this to a matrix build if you need cross-platform coverage.
# See: https://docs.github.com/en/free-pro-team@latest/actions/learn-github-actions/managing-complex-workflows#using-a-build-matrix
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v4
- name: Install Eigen
run: sudo apt install libeigen3-dev

- name: Configure CMake
# Configure CMake in a 'build' subdirectory. `CMAKE_BUILD_TYPE` is only required if you are using a single-configuration generator such as make.
# See https://cmake.org/cmake/help/latest/variable/CMAKE_BUILD_TYPE.html?highlight=cmake_build_type
run: cmake -B ${{github.workspace}}/build -DSPARSEIR_BUILD_TESTING=ON -DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}}

- name: Build
# Build your program with the given configuration
run: cmake --build ${{github.workspace}}/build --config ${{env.BUILD_TYPE}}

- name: Test
working-directory: ${{github.workspace}}/build
# Execute tests defined by the CMake configuration.
# See https://cmake.org/cmake/help/latest/manual/ctest.1.html for more detail
run: ctest -C ${{env.BUILD_TYPE}}

2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
# libsparseir

[![CMake on a single platform](https://github.com/SpM-lab/libsparseir/actions/workflows/CI_cmake.yml/badge.svg)](https://github.com/SpM-lab/libsparseir/actions/workflows/CI_cmake.yml)

> [!WARNING]
> This C++ project is still under construction. Please use other repositories:
> - https://github.com/SpM-lab/sparse-ir
Expand Down
2 changes: 0 additions & 2 deletions test/_linalg.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,6 @@ TEST_CASE("TSVD", "[linalg]") {
int k = s.size();

auto S_diag = s.asDiagonal();

auto Areconst = U * S_diag * V.transpose();
auto diff = (A - Areconst).norm() / A.norm();
// std::cout << "diff " << diff << std::endl;
Expand All @@ -301,7 +300,6 @@ TEST_CASE("TSVD", "[linalg]") {
// std::cout << "norm diff" << Aorig.norm() - Areconst.norm() << std::endl;

REQUIRE(Areconst.isApprox(Aorig, tol * Aorig.norm()));

REQUIRE((U.transpose() * U).isIdentity());
REQUIRE((V.transpose() * V).isIdentity());
REQUIRE(std::is_sorted(s.data(), s.data() + s.size(), std::greater<DDouble>()));
Expand Down

0 comments on commit ad98e33

Please sign in to comment.