Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

P1 (all in 1) #314

Open
wants to merge 58 commits into
base: master
Choose a base branch
from
Open

P1 (all in 1) #314

wants to merge 58 commits into from

Conversation

why-not-try-calmer
Copy link
Contributor

No description provided.

@spch-GL
Copy link
Contributor

spch-GL commented Dec 21, 2023

OK, mais quelle version je prends ? la master ?

@spch-GL
Copy link
Contributor

spch-GL commented Dec 21, 2023

J'ai pris la branche p1 :-)

@spch-GL
Copy link
Contributor

spch-GL commented Jan 23, 2024

Avec la 2.1.3_uat_P1 les rapports annuels ne sortent pas.
Pour info, il ne demande pas quels rapports générer...

Ci-joint le log, mais rien n'est écrit dans le dossier choisi
LOG_uat_P1.txt

@spch-GL
Copy link
Contributor

spch-GL commented Jan 25, 2024

Précision: aucun message d'erreur, mais aucun fichier produit (à moins qu'il n'ait été écrit ailleurs ?)

@spch-GL
Copy link
Contributor

spch-GL commented Jan 25, 2024

Non, il n'y avait pas de fichier produit à cause du filtre sur la date.
avec ce filtre

def _prepare_yearly_report(
    file_path: str, year: int, template_path: str, section_id: str, callback_progress
):
    # Get first count to be used as example
    count_qs = models.Count.objects.filter(
        id_installation__lane__id_section=section_id,
        start_process_date__year__lte=year,
        end_process_date__year__gte=year
    )
    if not count_qs.exists():
        return

ça passe plus loin, mais cette fois une erreur survient:

2024-01-25T17:26:45     WARNING    Traceback (most recent call last):
              File "C:\apps\OSGeo4W\apps\Python39\lib\site-packages\pandas\core\indexes\base.py", line 3791, in get_loc
              return self._engine.get_loc(casted_key)
              File "index.pyx", line 152, in pandas._libs.index.IndexEngine.get_loc
              File "index.pyx", line 181, in pandas._libs.index.IndexEngine.get_loc
              File "pandas\_libs\hashtable_class_helper.pxi", line 7080, in pandas._libs.hashtable.PyObjectHashTable.get_item
              File "pandas\_libs\hashtable_class_helper.pxi", line 7088, in pandas._libs.hashtable.PyObjectHashTable.get_item
             KeyError: 'date'
             
             The above exception was the direct cause of the following exception:
             
             Traceback (most recent call last):
              File "C:\Users/xxx/AppData/Roaming/QGIS/QGIS3\profiles\cptg_dev/python/plugins\comptages\core\report_task.py", line 44, in run
              raise e
              File "C:\Users/xxx/AppData/Roaming/QGIS/QGIS3\profiles\cptg_dev/python/plugins\comptages\core\report_task.py", line 32, in run
              report.prepare_reports(
              File "C:\Users/xxx/AppData/Roaming/QGIS/QGIS3\profiles\cptg_dev/python/plugins\comptages\core\report.py", line 39, in prepare_reports
              _prepare_yearly_report(
              File "C:\Users/xxx/AppData/Roaming/QGIS/QGIS3\profiles\cptg_dev/python/plugins\comptages\core\report.py", line 113, in _prepare_yearly_report
              _data_day_yearly(count, section, year, workbook)
              File "C:\Users/xxx/AppData/Roaming/QGIS/QGIS3\profiles\cptg_dev/python/plugins\comptages\core\report.py", line 410, in _data_day_yearly
              df = statistics.get_time_data_yearly(year, section, direction=2)
              File "C:\Users/xxx/AppData/Roaming/QGIS/QGIS3\profiles\cptg_dev/python/plugins\comptages\core\statistics.py", line 95, in get_time_data_yearly
              df = df.groupby([df["date"].dt.dayofweek, "hour"]).thm.sum()
              File "C:\apps\OSGeo4W\apps\Python39\lib\site-packages\pandas\core\frame.py", line 3893, in __getitem__
              indexer = self.columns.get_loc(key)
              File "C:\apps\OSGeo4W\apps\Python39\lib\site-packages\pandas\core\indexes\base.py", line 3798, in get_loc
              raise KeyError(key) from err
             KeyError: 'date'

Remarque:

  • Lorsqu'un test est effectué, ne pas quitter la procédure sans avertir l'utilisateur de ce qu'il se passe

@spch-GL
Copy link
Contributor

spch-GL commented Jan 25, 2024

Précision

  • mon comptage est défini ainsi:
    image

Si ce n'est pas une bonne idée, merci de le préciser dans la doc

@spch-GL
Copy link
Contributor

spch-GL commented Jan 29, 2024

En fait, il manque les tests sur la quantité de données car l'erreur survient sur une section sans données pour une direction

Merci de faire les corrections nécessaires rapidement

@spch-GL
Copy link
Contributor

spch-GL commented Jan 29, 2024

Avec ces quelques corrections du code, les rapports semblent générés correctement, mais j'attends la validation de Jonathan.
Corrections.zip

@why-not-try-calmer
Copy link
Contributor Author

En fait, il manque les tests sur la quantité de données car l'erreur survient sur une section sans données pour une direction

Merci de faire les corrections nécessaires rapidement

Toutes les modifications demandées les 3-4 derniers jours seront faites aujourdhui.

@spch-GL
Copy link
Contributor

spch-GL commented Jan 30, 2024

En fait, il manque les tests sur la quantité de données car l'erreur survient sur une section sans données pour une direction
Merci de faire les corrections nécessaires rapidement

Toutes les modifications demandées les 3-4 derniers jours seront faites aujourdhui.

Je ne les trouve pas...
Et merci de fusionner P1, P2 dans P3 afin que je puisse tester le tout

@why-not-try-calmer
Copy link
Contributor Author

En fait, il manque les tests sur la quantité de données car l'erreur survient sur une section sans données pour une direction
Merci de faire les corrections nécessaires rapidement

Toutes les modifications demandées les 3-4 derniers jours seront faites aujourdhui.

Je ne les trouve pas... Et merci de fusionner P1, P2 dans P3 afin que je puisse tester le tout

Merge p1-3 + release faits!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants