Regression on using time on latest master

[jaakkor2]

PPTX v0.10.4

julia> @time using PPTX
  0.342742 seconds (438.89 k allocations: 33.840 MiB, 3.94% gc time, 6.92% compilation time) # Julia 1.10.10
  0.596990 seconds (671.29 k allocations: 45.770 MiB, 8.36% gc time, 1.06% compilation time) # Julia 1.11.6

PPTX v0.10.5 (not registered, repo from 2025-9-3)

julia> @time using PPTX
  2.090685 seconds (2.42 M allocations: 167.098 MiB, 5.18% gc time, 6.32% compilation time: 20% of which was recompilation) # Julia 1.10.10
  2.590967 seconds (2.83 M allocations: 180.736 MiB, 6.94% gc time, 5.21% compilation time: 26% of which was recompilation) # Julia 1.11.6

#75 introduced a dependency on Images.jl. This was back in 2023 seen as a reason for slow compilation and using time #25 . As far as I can see, Images.jl is not necessary, as tests pass on master without Images.jl.

While at it, registration of v0.10.5 JuliaRegistries/General#137783 complains about missing compats for Images and VideoIO.

[asrono]

Hey thanks Jaakko for your quick analysis.
I added a function to create a thumbnail, which used the Images package to read the first frame from an mp4 file:
https://github.com/ASML-Labs/PPTX.jl/blob/main/src/Video.jl#L151
It's tested here: https://github.com/ASML-Labs/PPTX.jl/blob/main/test/testVideo.jl#L63

1. I am not sure why the tests pass without images. Let me get back to you.
2. Perhaps I can find a way to create a thumbnail without the Images dependency.
3. I wasn't aware about the compat issue. Here is PR to update that: Update compat for Images and VideoIO; remove Images dependency #77

[jaakkor2]

This reads and creates an image from mp4 without Images.jl just fine.

using VideoIO, ImageIO, FileIO
video = openvideo(joinpath(PPTX.ASSETS_DIR, "sample_video.mp4"))
frame = read(video)
save("foo.png", frame)

[asrono]

@jaakkor2
I remember trying to use ImageIO to create a thumbnail, but failed. Somehow, I overlooked this simple commands.

1. I think the reason the tests pass is because Images and ImageIO both define read and save
2. This is update now; see Update compat for Images and VideoIO; remove Images dependency #77
3. See point 2.

Thanks for the pointers! :)

[asrono]

Do you think it's useful to have a test that tests compilation time? In general, shouldn't there be specs, or at least guidlines what numbers are acceptable and not? :)

[matthijscox-asml]

Do you think it's useful to have a test that tests compilation time? In general, shouldn't there be specs, or at least guidlines what numbers are acceptable and not? :)

That would be great to have of course. I know Makie.jl has some custom scripts for testing compilation time.
Though those tests will increase our build time, so that's the downside. I suppose most of our compilation time comes from dependencies, which means we only need to check compilation times if dependencies are added to the Project.toml?

Also note that compilation times change with Julia version, so deciding on absolute specs might be tricky. But at least given the same Julia version we'd like compilation times to not increase without our approval. Right now we were lucky that @jaakkor2 checked the compilation times.

[jaakkor2]

PPTX v0.10.5 is fine.