Caching regression with cachier v3.1.2

[dphi]

Hi all, the following code works with cachier 3.0.1.

import datetime
import cachier

cachier.set_default_params(caching_enabled=False, separate_files=True)

class Test:
    def __init__(self, cache_ttl = None):
        self.counter = 0
        if cache_ttl is not None:
            stale_after = datetime.timedelta(seconds=cache_ttl)
            cachier.set_default_params(stale_after=stale_after)
            cachier.enable_caching()

    @cachier.cachier()
    def test(self, param):
        self.counter += 1
        assert self.counter < 2
        return param

if __name__ == "__main__":
    t = Test(cache_ttl=1)
    t.test("a")
    t.test("a")

With version 3.1.2, the caching of the test() function does not work as expected and the assert fails. This probably is not a minimal example yet.

Thanks for looking into it.

Philipp

[Borda]

Just a quick question, why do you expect that the second call will not be incremented since you disabled cashing with cachier.set_default_params(caching_enabled=False)?

[dphi]

My understanding of the enable_caching() method call in the __init__() is that it activates caching during runtime, even though it was previously deactivated.

[Borda]

I see your confusion, then we shall update the docs with better explanation...

[shaypal5]

Hey Jirka,

Did you ever get around to updating the docs about that?
@Borda