Skip to content

Convert decimal module to use PyLongWriter API - #5

Closed
skirpichev wants to merge 2 commits into
vstinner:long_exportfrom
skirpichev:long_export-decimal
Closed

Convert decimal module to use PyLongWriter API#5
skirpichev wants to merge 2 commits into
vstinner:long_exportfrom
skirpichev:long_export-decimal

Conversation

@skirpichev

@skirpichev skirpichev commented Jul 6, 2024 •
edited
Loading

Copy link
Copy Markdown

Second commit (patch v2) has a quick path for int64_t-sized integers.

Benchmark master patch patch2
1<<7 651 ns 658 ns: 1.01x slower 531 ns: 1.22x faster
1<<38 747 ns 700 ns: 1.07x faster 533 ns: 1.40x faster
1<<300 2.35 us 2.32 us: 1.01x faster not significant
Geometric mean (ref) 1.02x faster 1.15x faster

Benchmark hidden because not significant (1): 1<<3000

Details 
import pyperf
from decimal import Decimal as D

runner = pyperf.Runner()
runner.bench_func('1<<7', int, D(1 << 7))
runner.bench_func('1<<38', int, D(1 << 38))
runner.bench_func('1<<300', int, D(1 << 300))
runner.bench_func('1<<3000', int, D(1 << 3000))

@skirpichev
skirpichev force-pushed the long_export-decimal branch from f2d8efd to 6c77cf2 Compare July 6, 2024 07:15
@skirpichev skirpichev mentioned this pull request Jul 6, 2024
Merged
@skirpichev
skirpichev force-pushed the long_export-decimal branch from 6c77cf2 to 196d878 Compare July 11, 2024 03:02
@skirpichev skirpichev mentioned this pull request Jul 15, 2024
Closed
@skirpichev
skirpichev force-pushed the long_export-decimal branch from 196d878 to b7744dc Compare August 20, 2024 08:40
@skirpichev skirpichev closed this Dec 13, 2024
@skirpichev skirpichev mentioned this pull request Dec 13, 2024
Merged
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

See More

Successfully merging this pull request may close these issues.

1 participant

@skirpichev