icesat2py/icepyx

add page size checks for sync/async request mode

Aberta

#243 aberto em 9 de nov. de 2021

Ver no GitHub
 (2 comentários) (0 reação) (0 responsável)Python (119 forks)auto 404
IS2HW_2022good first issue

Métricas do repositório

Stars
 (256 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

There's still a reference to using a smaller page size in examples/examples/ICESat-2_DAAC_DataAccess_Example.ipynb because it refers to synchronous requests. Note that this notebook indicates that synchronous requests are done by default, but I think this must be outdated. It looks like the request mode is async by default (https://github.com/icesat2py/icepyx/blob/development/icepyx/core/APIformatting.py#L446).

Given that the page size limit is only 100 when the request mode is synchronous, maybe we need another check somewhere that decreases the page size when a user overrides that default and uses synchronous mode?

Originally posted by @trey-stafford in https://github.com/icesat2py/icepyx/issues/239#issuecomment-963612931

Guia do colaborador

Direção de pesquisa
Investigue o arquivo APIformatting.py por volta da linha 446 para entender como o modo de requisição é definido. Em seguida, implemente uma verificação que reduza o tamanho da página para 100 quando o modo síncrono for usado e garanta que a mudança seja refletida nos exemplos.
Pilha de tecnologia
python
Domain
backendapi
Tipo Issue
Funcionalidade
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.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Pythonsync/async programming
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65