Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate to ArrayInterface v7, move Julia minimum version to 1.6 #41

Merged
merged 6 commits into from
May 10, 2023

Conversation

BioTurboNick
Copy link
Contributor

No description provided.

@BioTurboNick
Copy link
Contributor Author

Seems it may also need to add https://github.com/JuliaArrays/StaticArrayInterface.jl as a dependency?

@timholy
Copy link
Member

timholy commented Mar 6, 2023

If so, I'm fine with that.

@BioTurboNick
Copy link
Contributor Author

Apparently everything directly needed from ArrayInterface moved to StaticArrayInterface, and ArrayInterface 7 is a dependency of StaticArrayInterface.

But StaticArrayInterface is not compatible with Julia 1.2. Can support for 1.2 be dropped?

@codecov
Copy link

codecov bot commented Mar 6, 2023

Codecov Report

Merging #41 (ce24561) into master (e17daf5) will decrease coverage by 0.71%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
- Coverage   98.58%   97.87%   -0.71%     
==========================================
  Files           2        2              
  Lines         141      141              
==========================================
- Hits          139      138       -1     
- Misses          2        3       +1     
Impacted Files Coverage Δ
src/TiledIteration.jl 96.05% <100.00%> (-1.32%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@timholy
Copy link
Member

timholy commented Mar 7, 2023

Yes. Obviously we need to bump to 0.5. Should we just go straight to 1.0? I am not seeing the reasons to keep have 0.x releases for a package which is changing only slowly.

@BioTurboNick BioTurboNick changed the title Add compat for ArrayInterface v7 Migrate to ArrayInterface v7, move Julia minimum version to 1.6 Mar 7, 2023
@BioTurboNick
Copy link
Contributor Author

Good question - there is the one PR that, while old, seems to indicate a need for a better API? Maybe I can take a look at it.

@timholy
Copy link
Member

timholy commented Mar 7, 2023

Thanks! I don't mean this package won't change in the future: there's a chance I'll have an extended period where I'll rework a lot of JuliaImages (ideally as grant-funded work). So if you'd rather hold off on 1.0 because you think there are some major changes needed, that's perfectly fine.

@BioTurboNick
Copy link
Contributor Author

Great! Ready to merge and release then?

@laborg
Copy link

laborg commented Apr 28, 2023

Could you please merge this PR and release a new version? I got some precompliation warnings because the compat bound trickeled down to me through ImageFiltering.jl

@BioTurboNick
Copy link
Contributor Author

@timholy - sorry to bug you, could you please merge and register this quick?

@BioTurboNick
Copy link
Contributor Author

BioTurboNick commented May 9, 2023

Oh I guess StaticArrayInterface needs a simple compat entry for the registration to go through, right? - EDIT: done

@timholy timholy merged commit 64a0a7e into JuliaArrays:master May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants