google/benchmark

[FR] Allow ThreadRange Multiplier to be different from Arg Ranges one

Open

#1.531 aberto em 16 de jan. de 2023

Ver no GitHub
 (0 comments) (1 reaction) (0 assignees)C++ (7.968 stars) (1.539 forks)batch import
enhancementhelp wanted

Description

Is your feature request related to a problem? Please describe. Once a RangeMultiplier is set, the threadrange also uses the exact same one, it would be easier if one could either set a ThreadRangeMultiplier or a multiplier argument to ThreadRange(min,max,multiplier)

Describe the solution you'd like A simple solution could be accepting a multiplier argument to the the ThreadRange arguments which can be used in case you'd need a customization on ThreadRange different from RangeMultiplier

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

Guia do colaborador

Pilha de tecnologia
cpp
Domain
performance
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
half day
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
C++Google Benchmark library basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65
Direção de pesquisa
Start by examining the current implementation of ThreadRange and RangeMultiplier in the benchmark library. The relevant code is likely in include/benchmark/benchmark.h and src/benchmark.cc. Implement an additional parameter for multiplier in ThreadRange, following the pattern of the existing RangeMultiplier. Then update the documentation and tests accordingly.