Curtis Vogt
Curtis Vogt
I have some concerns about the lazy loading now. We seem to just be passing the buck on performing the cache initialization. I especially don't like it now that packages...
Thanks for bringing this up. An alternate solution could be to utilize `__init__` functions to populate `Ref` constants. The benefit of this approach is that it lets us control when...
We probably do want to calculate the artifact location at runtime so that overrides are applied. I'll need to think about this more yet.
Any motivation behind this?
We may want to start testing with threads yet but the original reason for this is no longer happening.
> Deprecations are commonly included in minor version bumps for Julia packages. The difference here is that TimeZones.jl doesn't allow the use of legacy/deprecated time zones by default. I could...
Baseline timings for my machine. I'll need to resolve the conflict yet. Julia 1.10.0 Current `master` (ac06a91) ```julia julia> @btime TimeZone("America/Winnipeg"); 41.246 ns (2 allocations: 64 bytes) julia> @btime TimeZone("UTC");...
This PR 511d2cc ```julia julia> @time_imports using TimeZones 0.4 ms Scratch 10.2 ms Preferences 0.4 ms PrecompileTools ┌ 0.0 ms Parsers.__init__() 45.3 ms Parsers 40.31% compilation time 4.2 ms InlineStrings...
The cache code is definitely becoming unwieldy. I've created a `TimeZoneCache` struct which doesn't affect the performance: This PR 67f2c3f ```julia julia> @time_imports using TimeZones 0.4 ms Scratch 9.1 ms...
These changes are also a step in the right direction for reducing the initial cache loading time This PR 67f2c3f ```julia julia> @btime TimeZones._reload_tz_cache(TimeZones._COMPILED_DIR[]); 20.795 ms (313574 allocations: 12.52 MiB)...