search

mapado / datection

Detect and normalize temporal expressions
0 stars 0 forks source link

first set of merge and extend #35

Closed jdeniau closed 4 years ago

jdeniau commented 8 years ago

Pull request :twisted_rightwards_arrows: created by @RomainMapado on 2016-10-25 11:19 Last updated on 2016-10-27 09:06 Original Bitbucket pull request id: 35

Participants:

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

Source: https://github.com/mapado/datection/commit/f085aa28720a on branch rs-feat-rrule-packer Destination: https://github.com/mapado/datection/commit/e3c171f5cd06 on branch master Merge commit: https://github.com/mapado/datection/commit/bedf809b2733

State: MERGED

Premier commit pour le packing de rrules, il y a quelques méthodes qui sont encore en "pass" c'est pas un oubli c'est en cours.

Ca peut aider de jeter un oeil à ce doc pour comprendre:

https://docs.google.com/a/mapado.com/document/d/1h4FQlJdtveBCRmrMkI05mimqQXW_GHhD5k6IgNDR7lc/edit?usp=sharing

jdeniau commented 8 years ago

@dallegoet approved :heavy_check_mark: the pull request on 2016-10-25 13:12

jdeniau commented 8 years ago

@badaz commented on 2016-10-25 13:43

wah ! ça fait peur !

jdeniau commented 8 years ago

@badaz commented on 2016-10-25 13:46

Location: line 165 of datection/pack.py

are_contiguous ?

jdeniau commented 8 years ago

@badaz approved :heavy_check_mark: the pull request on 2016-10-25 13:48

jdeniau commented 8 years ago

@jnieuviarts commented on 2016-10-25 21:57

Location: line 32 of datection/test/test_pack.py

Quid du cas "sans horaire" que l'on trouve parfois. Exemple : ddr1 : du 15 au 20 avril à 18h ddr2 : le 18 avril

jdeniau commented 8 years ago

@jnieuviarts commented on 2016-10-25 22:00

Location: line 53 of datection/test/test_pack.py

Il y a peut être un bug dans le test, le BYOUR devrait être égal à autre chose que 3 tel que je le comprend. D'ailleurs, ça serait peut être mieux d'avoir une date à l'intérieur de l'intervalle pour tester les cas _hour / _min / _dura

jdeniau commented 8 years ago

@jnieuviarts commented on 2016-10-25 22:04

Location: line 267 of datection/pack.py

Warning sur la création de cette méthode : on peut parfois se retrouver avec des listes de dates de plusieurs kilomètres, si tu fais du pairwise, tu vas rapidement avoir un nombre important d'itération

jdeniau commented 8 years ago

@jnieuviarts approved :heavy_check_mark: the pull request on 2016-10-25 22:05

jdeniau commented 8 years ago

@RomainMapado commented on 2016-10-26 07:31

Location: datection/pack.py

Warning sur la création de cette méthode : on peut parfois se retrouver avec des listes de dates de plusieurs kilomètres, si tu fais du pairwise, tu vas rapidement avoir un nombre important d'itération

Je suis en train de la coder, et non je suis pas parti sur du pairwise :)

jdeniau commented 8 years ago

@RomainMapado commented on 2016-10-26 07:31

Location: datection/test/test_pack.py

Quid du cas "sans horaire" que l'on trouve parfois. Exemple : ddr1 : du 15 au 20 avril à 18h ddr2 : le 18 avril

Bonne remarque, je vais creuser

jdeniau commented 8 years ago

@RomainMapado commented on 2016-10-26 07:31

Location: datection/test/test_pack.py

Il y a peut être un bug dans le test, le BYOUR devrait être égal à autre chose que 3 tel que je le comprend. D'ailleurs, ça serait peut être mieux d'avoir une date à l'intérieur de l'intervalle pour tester les cas _hour / _min / _dura

Oui le test est mauvais