Error when calling to_pydict on a message having a repeated timestamp field
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
Piste de recherche
Reproduisez le problème avec la définition de Msg et la valeur datetime fournies, puis suivez le chemin de conversion de to_pydict qui traite les repeated fields et les valeurs google.protobuf.Timestamp. Le travail est terminé lorsque to_pydict s’exécute sans AttributeError et renvoie la représentation attendue de repeated timestamp, conforme au comportement fonctionnel de to_dict.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
When calling the to_pydict method on a message which has a repeated timestamp field, an AttributeError is raised.
Reproduction Steps
Compile the following message :
message Msg {
repeated google.protobuf.Timestamp ts = 1;
}
Then, execute the following code:
from datetime import datetime
msg = Msg(ts=[datetime(2015, 3, 5)])
print(msg.to_dict())
print(msg.to_pydict())
Expected Results
In this situation, it is expected for the two calls to work without error.
Actual Results
The call to to_dict works as expected: it prints {'ts': ['2015-03-05T00:00:00Z']}. However, the call to to_pydict fails with the error: AttributeError: 'datetime.datetime' object has no attribute 'to_pydict'.
It is possible to simply fix this bug (and the related bugs) by patching the to_pydict function, but it might be easier with a bit of refactoring first. I will probably be able to work on this once my other PRs are approved.
System Information
libprotoc 3.12.4
Python 3.12.5
Name: betterproto
Version: 2.0.0b7
Summary: A better Protobuf / gRPC generator & library
Home-page: https://github.com/danielgtaylor/python-betterproto
Author: Daniel G. Taylor
Author-email: [email protected]
License: MIT
Location: XXX/.venv/lib/python3.12/site-packages
Requires: grpclib, python-dateutil, typing-extensions
Required-by: pygrpc-poc
Checklist
- I have searched the issues for duplicates.
- I have shown the entire traceback, if possible.
- I have verified this issue occurs on the latest prelease of betterproto which can be installed using
pip install -U --pre betterproto, if possible.
- Langage dominant
- Python
- Étoiles
- 1.8k
- Forks
- 235
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de danielgtaylor/python-betterproto
-
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
-
bug investigation needed
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
-
bug investigation needed
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
-
bug investigation needed
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
danielgtaylor/python-betterproto#676 · 3 commentaires ·
Toutes les issues de danielgtaylor/python-betterproto
Issues similaires
-
[Resource]: LoreOuverteresource-submission validation-passed
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
hesreallyhim/awesome-claude-code#3234 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
openminion/openminion#376 ·
Les mainteneurs répondent en général sous 1 jour
-
Type: ☹︎ Bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
posit-dev/great-tables#1028 ·
Les mainteneurs répondent en général sous 1 jour
-
integration: apple_tv
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
home-assistant/core#188282 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
A "since" range is worded as a career: "the past two seasons" says "regular season career (2025 on)"OuverteP4: low
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
jeffknupp/association#538 ·