Rs fix parsing improvements

[jdeniau]

Pull request :twisted_rightwards_arrows: created by @RomainMapado on 2016-08-29 13:59 Last updated on 2016-08-31 14:24 Original Bitbucket pull request id: 28

Participants:

• @RomainMapado
• @jdeniau (reviewer) :heavy_check_mark:
• @jnieuviarts (reviewer) :heavy_check_mark:
• @zwit (reviewer) :heavy_check_mark:
• @dallegoet (reviewer) :heavy_check_mark:

Source: https://github.com/mapado/datection/commit/cea847c8e71e on branch rs-fix-parsing-improvements Destination: https://github.com/mapado/datection/commit/e70d722bbf4a on branch master Merge commit: https://github.com/mapado/datection/commit/4fe7bb8cf19c

State: MERGED

• add keyword: jusqu'au, mercr
• parsing + rendering des exclusions multiples
• parsing + rendering des alterations (tous les jours à 15h, sauf jeudi à 13h)
• parsing des listes de datetimes (le 14/10 et le 27/10/)
• transfert du timing de l'exclusion si nécessaire (tous les jours sauf mardi à 19h)
• rendering : regroupement des timings par jours (le jeudi, de 13h à 14h et de 16h à 19h)

[jdeniau]

@RomainMapado commented on 2016-08-29 14:01

Location: line 490 of datection/tokenize.py

L'idée c'est de regrouper les exclusions tant qu'on a le même type de token (du 12/01 au 13/05, sauf le mardi et le mercredi)

[jdeniau]

@RomainMapado commented on 2016-08-29 14:02

Location: datection/timepoint.py

Ajout d'une méthode de calcul de durée pour les objets pour lesquels c'est pertinent

[jdeniau]

@RomainMapado commented on 2016-08-29 14:03

Location: line 90 of datection/test/test_tokenize.py

Changement du mock pour refléter les changements internes du tokenizer

[jdeniau]

@RomainMapado commented on 2016-08-29 14:05

Location: datection/schedule.py

Support des exclusions multiples et des alterations (exclusion avec une durée/timing différent) au niveau schedule

[jdeniau]

@RomainMapado commented on 2016-08-29 14:06

Location: line 149 of datection/render.py

Sert à regrouper "le jeudi de 13h à 14h" et "le jeudi de 15h à 15h30" en "le jeudi de 13h à 14h et de 15h à 15h30"

[jdeniau]

@RomainMapado commented on 2016-08-29 14:07

Location: line 1013 of datection/render.py

Rendering des alterations (tous les jours à 15h, sauf mercredi à 16h)

[jdeniau]

@RomainMapado commented on 2016-08-29 14:08

Location: datection/parse.py

Parsing des exclusions multiples

[jdeniau]

@RomainMapado commented on 2016-08-29 14:10

Location: line 131 of datection/models.py

Petit hack pour contourner un bug de la lib dateutil.rrule pour la génération des datetimes

[jdeniau]

@RomainMapado commented on 2016-08-29 14:10

Location: line 104 of datection/models.py

Les durations sont utilisées par les altérations (tous les jours de 18h à 20h, sauf lundi de 18h à 19h)

[jdeniau]

@RomainMapado commented on 2016-08-29 14:11

Location: datection/exclude.py

Transfert du timing de l'exclusion vers le match (tous les jours sauf mardi à 18h)

[jdeniau]

@jdeniau commented on 2016-08-30 08:06

Outdated location: line 249 of datection/test/test_parse.py

Le mardi 9 décembre ne devrait pas être présent dans la liste (puisque "sauf le mardi")

[jdeniau]

@jdeniau commented on 2016-08-30 08:09

Location: datection/models.py

Petit hack pour contourner un bug de la lib dateutil.rrule pour la génération des datetimes

pas fixé dans la lib sur une mise à jour ? n'hésite pas à faire une PR sur la lib si c'est un vrai bug et le référencer ici pour suppression après fix dans la lib

[jdeniau]

@jdeniau commented on 2016-08-30 08:10

Ok après fix du mardi 9 dec

[jdeniau]

@jdeniau approved :heavy_check_mark: the pull request on 2016-08-30 08:11

[jdeniau]

@dallegoet approved :heavy_check_mark: the pull request on 2016-08-30 08:13

[jdeniau]

@jnieuviarts approved :heavy_check_mark: the pull request on 2016-08-30 09:23

[jdeniau]

@RomainMapado commented on 2016-08-30 13:20

Ok après fix du mardi 9 dec

Done ;)

[jdeniau]

@zwit approved :heavy_check_mark: the pull request on 2016-08-31 08:23