feedc0de/mp-units
1
0
Fork 0
forked from mpusz/mp-units
Code Pull Requests Activity

Revert "Try upping the basis size further"

Browse Source 
This reverts commit 87073856a7.
It didn't fix the problem, and it caused some new ones.  We need a
different approach.
This commit is contained in:
Chip Hogg
2022-03-19 13:21:47 +00:00
parent 87073856a7
commit 1e8460d401
2 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/core/include/units/magnitude.h
View File

@@ -33,7 +33,7 @@ namespace units {
namespace detail
{
// Higher numbers use fewer trial divisions, at the price of more storage space.
using Factorizer = WheelFactorizer<6>;
using Factorizer = WheelFactorizer<4>;
} // namespace detail
/**

1
test/unit_test/static/prime_test.cpp
View File

@@ -50,7 +50,6 @@ static_assert(WheelFactorizer<2>::coprimes_in_first_wheel.size() == 2);
static_assert(WheelFactorizer<3>::coprimes_in_first_wheel.size() == 8);
static_assert(WheelFactorizer<4>::coprimes_in_first_wheel.size() == 48);
static_assert(WheelFactorizer<5>::coprimes_in_first_wheel.size() == 480);
static_assert(WheelFactorizer<6>::coprimes_in_first_wheel.size() == 5760);
static_assert(WheelFactorizer<3>::coprimes_in_first_wheel[0] == 1);
static_assert(WheelFactorizer<3>::coprimes_in_first_wheel[1] == 7);