Skip to content

Commit ee878be

Browse files
matthiasdienerinducer
authored andcommitted
KeyBuilder: Fix time hashing
1 parent a5efddf commit ee878be

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

pytools/persistent_dict.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -460,7 +460,7 @@ def update_for_time(self, key_hash: Hash, key: Any) -> None:
460460
import datetime
461461

462462
# Convert to datetime object
463-
self.rec(key_hash, datetime.datetime.combine(datetime.date.today(), key))
463+
self.rec(key_hash, datetime.datetime.combine(datetime.date.min, key))
464464
self.rec(key_hash, "<time>")
465465

466466
def update_for_datetime(self, key_hash: Hash, key: Any) -> None:

pytools/test/test_persistent_dict.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -621,7 +621,7 @@ def test_datetime_hashing() -> None:
621621
== keyb(datetime.time(12, 0))
622622
== keyb(datetime.time(12, 0, 0))
623623
== keyb(datetime.time(12, 0, 0, 0))
624-
== "bf73f48b2f2666b5c42f6993e628fdc15e0b6c3127186c3ab44ce08ed83d0472")
624+
== "288ec82f6a00ac15968d4d257d4aca1089b863c61ef2ee200e64351238397705")
625625
assert keyb(datetime.time(12, 0)) != keyb(datetime.time(12, 1))
626626

627627
# Aware time
@@ -634,7 +634,7 @@ def test_datetime_hashing() -> None:
634634
assert t1 == t2
635635
assert (keyb(t1)
636636
== keyb(t2)
637-
== "c0947587c92ab6e2df90475dd497aff1d83df55fbd5af6c55b2a0a221b2437a4")
637+
== "3587427ca9d581779d532b397df206ddeadfcf4e38b1ee69c19174e8e1268cc4")
638638

639639
assert t1 != t3
640640
assert keyb(t1) != keyb(t3)

0 commit comments

Comments
 (0)