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

SuiteSparse v7.2.0.beta1 #380

Merged
merged 241 commits into from
Sep 8, 2023
Merged

SuiteSparse v7.2.0.beta1 #380

merged 241 commits into from
Sep 8, 2023

Conversation

DrTimothyAldenDavis
Copy link
Owner

@DrTimothyAldenDavis DrTimothyAldenDavis commented Sep 5, 2023 •
edited
Loading

Sept 8, 2023: version 7.2.0 (BETA1)

* build system: modern cmake structure, by Markus Muetzel.
    Most packages updated to vX.2.0 where X is unchanged (except SPQR
    and Example package).
* SPQR v4.2.0: Major SO update.  Support for int32 indices by Raye Kimmerer

rayegun and others added 30 commits May 11, 2023 00:04
@rayegun
messy init, needs tests
52f2bb4
@rayegun
possibly enable gpu, saving to test on linux
55c062d
@rayegun
add explicit instantiations
1ac1bde
@rayegun
fix silly templating
8fd4f74
@rayegun
oops!
4241e88
@rayegun
fix gpu specializations/templates
ca2e41e
@rayegun
better use of templates in GPUQREngine
69b51a1
@rayegun
instantiate less
6eac89c
@rayegun
fix accidentally building the GPU options
c2a6142
@rayegun
attempt to fix tcov (unsuccessful), fix GPUQR
b76e154
@rayegun
change GPUQREngine to explicit instantiation, fix newlines
004e350
@rayegun
extern template
0f03c68
@mmuetzel
SuiteSparse_config: Generate import target files.
4b69fc1
@mmuetzel
Use autogenerated import targets instead of FindSuiteSparse_config.cmake
b7fe0d9
@mmuetzel
Adjust example to use imported target.
133b1db 
Also only link to static library if it was actually built.
@mmuetzel
Mongoose: Export targets instead of FindMongoose.cmake
50e111d
@mmuetzel
AMD: Generate import target files.
add6338
@mmuetzel
Use autogenerated import targets instead of FindAMD.cmake
c093df1
@mmuetzel
BTF: Generate import target files.
778a9ce
@mmuetzel
Use autogenerated import targets instead of FindBTF.cmake
2ba6277
@mmuetzel
CAMD: Generate import target files.
7185be7
@mmuetzel
Use autogenerated import targets instead of FindCAMD.cmake
86d9eaf
@mmuetzel
CCOLAMD: Generate import target files.
b739b0b
@mmuetzel
Use autogenerated import targets instead of FindCCOLAMD.cmake
1ad9331
@mmuetzel
COLAMD: Generate import target files.
e7c7004
@mmuetzel
Use autogenerated import targets instead of FindCOLAMD.cmake
6102dd7
@mmuetzel
CHOLMOD: Generate import target files.
b5c91e5
@mmuetzel
Use autogenerated import targets instead of FindCHOLMOD.cmake
14a98f4
@mmuetzel
CXSparse: Export targets instead of FindCXSparse.cmake
29b88b6
@mmuetzel
GPUQREngine: Export targets instead of FindGPUQREngine.cmake
cc80916
@DrTimothyAldenDavis
test coverage for int32 version of SPQR
947b4c9
@DrTimothyAldenDavis
typo in user guide
c31e8e5
@DrTimothyAldenDavis
Merge pull request #378 from mmuetzel/ci
d51c90f 
CI: Add runners that build with CUDA on Ubuntu
@DrTimothyAldenDavis
CHOLMOD METIS interface: use GKRAND with threadprivate
f3ac3ea
@DrTimothyAldenDavis
remove debug printf
c0d1a62
@DrTimothyAldenDavis
README
b7a9d53
@DrTimothyAldenDavis
GraphBLAS readme
a27f33b
@DrTimothyAldenDavis
added SPQR Tcov tests for 32-bit indices
2cda544
@mmuetzel
CI: Use ccache also for CUDA compiler.
183e3db
@DrTimothyAldenDavis
Merge pull request #379 from mmuetzel/ci
db74df5 
CI: Use ccache also for CUDA compiler.
@DrTimothyAldenDavis
full test coverage for GraphBLAS v8.2.0
f524455
@DrTimothyAldenDavis
date tagged: v7.2.0.beta, Sept 5, 2023
b3d9fb2
@DrTimothyAldenDavis
Mongoose user guide
f134f1a
@DrTimothyAldenDavis
demo output
2714a7f
@DrTimothyAldenDavis
GraphBLAS v8.2.0 beta, Sept 5, 2023
0098782
@mmuetzel
CHOLMOD: Don't include external headers in extern "C" block.
495621d
@mmuetzel
Don't generate .pc files for MSVC compatible tool chains.
1332e63
@DrTimothyAldenDavis
Merge pull request #382 from mmuetzel/pkg-config
0dd6e8f 
Don't generate .pc files for MSVC compatible tool chains.
@DrTimothyAldenDavis
Merge pull request #381 from mmuetzel/cholmod
cb3528a 
CHOLMOD: Don't include external headers in `extern "C"` block.
@mmuetzel
SPQR: Overhaul CMake build rules.
4e9d1e4 
Use targets more consistently instead of variables.
Set properties on the target instead of globally where possible.
No need for a CUDA compiler for the sources of SPQR.
There are no OpenMP accelerated expressions in the sources. So, don't depend on OpenMP.
Don't link transient targets of CHOLMOD (AMD and COLAMD) explicitly.
Link static libraries to static libraries if possible.
@DrTimothyAldenDavis
add more test cases to SPQR
9ca178d
@mmuetzel
SPQR: Add option for OpenMP again.
717841a 
SPQR might use OpenMP in the future. By default, OpenMP is disabled for
now to avoid over-linking.
@DrTimothyAldenDavis
Merge pull request #383 from mmuetzel/spqr
05f9fa9 
SPQR: Overhaul CMake build rules.
@DrTimothyAldenDavis
target date: Sept 8, 2023
c2c07ae
@DrTimothyAldenDavis
demo output for Sept 8 beta release
c73edbd
@DrTimothyAldenDavis
Mongoose user guide
0a472ba
@DrTimothyAldenDavis
acknowledgements
eaa4684
@DrTimothyAldenDavis DrTimothyAldenDavis merged commit 0ddf46e into dev Sep 8, 2023
15 checks passed
@mmuetzel
Copy link
Contributor

mmuetzel commented Sep 8, 2023

There seems to be a build issue when CHOLMOD is configured with NOPENMP.
I guess that moving SuiteSparse__thread.cmake to the cmake_modules folder inside SuiteSparse_config might fix it. But I haven't tested that yet.

Is there a reason why it's not in the same folder as the other .cmake files?

@DrTimothyAldenDavis
Copy link
Owner Author

oops! That's an oversight. It should be inside the SuiteSparse_config/cmake_modules subfolder. I'll move it there now and post a 7.2.0.beta2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@DrTimothyAldenDavis @mmuetzel @rayegun @whuaegeanse