Clarify which parameters are accepted by Traversable.open
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- Python
- Estrelas
- 77.2k
- Forks
- 35.9k
- Métricas de merge de PRs
- Métricas de PR pendentes
Descrição
Documentation
The docs for the open method of the importlib.resources.abc.Traversable protocol state:
When opening as text, accepts encoding parameters such as those accepted by io.TextIOWrapper.
It would be helpful to clarify exactly which parameters are accepted here. In particular, is the newline parameter covered by this? That parameter is accepted by io.TextIOWrapper, but is arguably not an "encoding parameter", so it's unclear whether it's supported.
An aside: typeshed does not seem to think that newline is supported here, but I'm wondering whether that might be a defect in typeshed. If the consensus here is that newline is supported, I'll open a typeshed issue.
The precise case I ran into was reading CSV data from a package resource. For portable CSV file reading, using newline="" is recommended, so I'm doing:
with csv_data.open(newline="") as f:
reader = csv.DictReader(f)
for row in reader:
... # do something with row
where csv_data implements importlib.resources.abc.Traversable. The open call is flagged as invalid by mypy and pyright, but the code appears to work correctly (including in the case where the resource is zipfile-based).
If newline is not intended to be accepted here, then I guess I instead have to do:
with importlib.resources.as_file(csv_data) as csv_file:
with csv_file.open(newline="") as f:
reader = csv.DictReader(f)
for row in reader:
... # do something with row
That's fine, but it would be cleaner to be able to stream data straight from the resource without having to materialise the resource as a file first.
Linked PRs
- gh-137076
- gh-157294
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece pela documentação de Traversable.open na página vinculada de importlib.resources.abc e, em seguida, revise os parâmetros documentados e reais do método open, especialmente encoding e newline. Verifique a discussão e os PRs vinculados gh-137076 e gh-157294 antes de decidir se a documentação deve descrever explicitamente newline; a tarefa estará concluída quando os parâmetros aceitos e o uso pretendido de CSV forem inequívocos.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- python
- Domínio
- documentation
- Tipo de issue
- Documentação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 25/100