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.

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

Release 1.3.0 for master #627

Merged
merged 134 commits into from Aug 27, 2020
Merged

Release 1.3.0 for master #627

merged 134 commits into from Aug 27, 2020

Conversation

tcojean
Copy link
Member

@tcojean tcojean commented Aug 26, 2020

This applies the release to the master branch.

To get a proper view of what really changed, use in the ginkgo repository:

git checkout develop && git pull
git checkout release/1.3.0 && git pull --rebase
git diff develop

This should correctly report that only a few references change from develop to master.

tcojean and others added 30 commits August 26, 2020 20:33
change the name and the commit number of CudaArchitectureSelector
We only need num_cols columns, not stride
* Copy libraries with correct version + symlinks
* Add RPATH linker flag
* Add missing pthread flag
* formatting
* fix duplicate output
* add example description

Co-authored-by: Yuhsiang M. Tsai <yhmtsai@gmail.com>
Co-authored-by: Fritz Goebel <goebel.fritz@googlemail.com>
1. use components prefix_sum in parilut
2. prefix_sum uses correct shared_memory value_type
Co-authored-by: Tobias Ribizel <ribizel@kit.edu>
Co-authored-by: Thomas Grützmacher <thomas.gruetzmacher@kit.edu>
Co-authored-by: Pratik Nayak <pratikvn@protonmail.com>
On some systems, this can cause problems if the two compilers are tied
to different glib versions. At linking time (esp. for examples or
tests), it is then possible that some symbols are not found.
Co-authored-by: Thomas Grützmacher <thomas.gruetzmacher@kit.edu>
Co-authored-by: Terry Cojean <terry.cojean@kit.edu>
Co-authored-by: tcojean <terry.cojean@kit.edu>
* move away from CMake 2.8
* add project(...) to external projects
* set a few CMake policies
+ Replace our own code with calls to the standard implementation
+ Do not directly call `xstd` in our code for base C++14 features.
tcojean and others added 13 commits August 26, 2020 20:35
+ Apply the filter only to specific file patterns. Replacing all `$`
  characters by `\f$` creates a big issue in bash and other languages
  where `$` has another signification. This convenience feature is needed
  only for the hpp/cpp/cu* files.
+ Remove the `FILE_VERSION_FILTER` from the Doxyfile configuration: this
  command is used to extract the version of every file and is always ran.
  We already publish the version of Ginkgo so it's unneeded. In addition,
  when building out of source, the example files are not part of a git
  repository and that creates issues.
+ Remove the `doxygen_image_path` as it's unused and throws a warning
+ Use `Ginkgo_SOURCE_DIR` to locate the md files instead of `../..`.
+ s/CygWin/Cygwin/
+ Add a reference to the JOSS paper in `CITING.md`

Co-authored-by: Yuhsiang M. Tsai <yhmtsai@gmail.com>
Co-authored-by: Pratik Nayak <pratikvn@protonmail.com>
@tcojean tcojean added 1:ST:ready-for-review This PR is ready for review 1:ST:high-importance This issue/PR is of high importance and must be addressed as soon as possible. labels Aug 26, 2020
@tcojean tcojean self-assigned this Aug 26, 2020
@sonarcloud
Copy link

sonarcloud bot commented Aug 27, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

36.1% 36.1% Coverage
6.8% 6.8% Duplication

warning The version of Java (1.8.0_121) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@codecov
Copy link

codecov bot commented Aug 27, 2020

Codecov Report

Merging #627 into master will increase coverage by 8.66%.
The diff coverage is 89.34%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #627      +/-   ##
==========================================
+ Coverage   84.22%   92.88%   +8.66%     
==========================================
  Files         296      303       +7     
  Lines       20655    21331     +676     
==========================================
+ Hits        17397    19814    +2417     
+ Misses       3258     1517    -1741     
Impacted Files Coverage Δ
core/base/extended_float.hpp 91.26% <ø> (+43.68%) ⬆️
core/device_hooks/common_kernels.inc.cpp 0.00% <0.00%> (ø)
core/factorization/par_ict.cpp 100.00% <ø> (+10.63%) ⬆️
core/test/base/polymorphic_object.cpp 97.61% <ø> (ø)
core/test/utils/assertions.hpp 70.98% <ø> (+27.67%) ⬆️
include/ginkgo/core/base/array.hpp 92.52% <ø> (+34.57%) ⬆️
include/ginkgo/core/base/exception_helpers.hpp 90.90% <ø> (+9.09%) ⬆️
include/ginkgo/core/base/name_demangling.hpp 85.71% <ø> (ø)
include/ginkgo/core/base/range.hpp 95.48% <ø> (+22.58%) ⬆️
include/ginkgo/core/base/types.hpp 92.59% <ø> (+25.92%) ⬆️
... and 168 more

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 b4be2be...f482bdc. Read the comment docs.

@tcojean tcojean merged commit 4678668 into master Aug 27, 2020
@tcojean tcojean deleted the release/1.3.0 branch August 27, 2020 08:05
@upsj upsj added 1:ST:ready-to-merge This PR is ready to merge. and removed 1:ST:ready-for-review This PR is ready for review labels Nov 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1:ST:high-importance This issue/PR is of high importance and must be addressed as soon as possible. 1:ST:ready-to-merge This PR is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants