Relax Julia compat version and solve_residuals! method (#65)

* Relax Julia compat version and `solve_residuals!` method

* Try to circumvent the requirement of Metal

* Add a comment

* Add a comment

* Bump version to 2.4.0

.github/workflows/ci.yml

@@ -15,19 +15,19 @@ jobs:
       fail-fast: false
       matrix:
         version:
-          - '1.9' # Replace this with the minimum Julia version that your package supports. E.g. if your package requires Julia 1.5 or higher, change this to '1.5'.
+          - '1.6' # Replace this with the minimum Julia version that your package supports. E.g. if your package requires Julia 1.5 or higher, change this to '1.5'.
           - '1' # Leave this line unchanged. '1' will automatically expand to the latest stable 1.x release of Julia.
         os:
           - ubuntu-latest
         arch:
           - x64
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - uses: julia-actions/setup-julia@v1
         with:
           version: ${{ matrix.version }}
           arch: ${{ matrix.arch }}
-      - uses: actions/cache@v1
+      - uses: actions/cache@v4
         env:
           cache-name: cache-artifacts
         with:
@@ -40,7 +40,7 @@ jobs:
       - uses: julia-actions/julia-buildpkg@v1
       - uses: julia-actions/julia-runtest@v1
       - uses: julia-actions/julia-processcoverage@v1
-      - uses: codecov/codecov-action@v1
+      - uses: codecov/codecov-action@v3
         with:
           file: lcov.info
 

Project.toml

@@ -1,6 +1,6 @@
 name = "FixedEffects"
 uuid = "c8885935-8500-56a7-9867-7708b20db0eb"
-version = "2.3.1"
+version = "2.4.0"
 
 [deps]
 LinearAlgebra = "37e2e46d-f89d-539d-b4ee-838fcccc9c8e"
@@ -11,7 +11,7 @@ GroupedArrays = "6407cd72-fade-4a84-8a1e-56e431fc1533"
 [compat]
 StatsBase = "0.33, 0.34"
 GroupedArrays = "0.3"
-julia = "1.9"
+julia = "1.6"
 
 [extensions]
 CUDAExt = "CUDA"
@@ -24,10 +24,10 @@ Metal = "dde4c033-4e86-420c-a63e-0dd931031962"
 [extras]
 CategoricalArrays = "324d7699-5711-5eae-9e2f-1d82baa6b597"
 CUDA = "052768ef-5323-5732-b1bb-66c8b64840ba"
-Metal = "dde4c033-4e86-420c-a63e-0dd931031962"
+Pkg = "44cfe95a-1eb2-52ea-b672-e2afdf69b78f"
 PooledArrays = "2dfb63ee-cc39-5dd5-95bd-886bf059d720"
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
 
 [targets]
-test = ["CategoricalArrays", "CUDA", "Metal", "PooledArrays", "Test"]
+test = ["CategoricalArrays", "CUDA", "Pkg", "PooledArrays", "Test"]
 

src/AbstractFixedEffectSolver.jl

@@ -79,7 +79,11 @@ function solve_residuals!(r::AbstractVector{<:Real}, feM::AbstractFixedEffectSol
 	return r, iter, converged
 end
 
-function solve_residuals!(xs::AbstractVector{<: AbstractVector}, feM::AbstractFixedEffectSolver; progress_bar = true, kwargs...)
+# A fallback method for collections of x
+# The container for data columns does not have to be a vector
+# This allows the use of iterators and tuples for xs in downstream packages
+# See https://github.com/FixedEffects/FixedEffects.jl/pull/65
+function solve_residuals!(xs, feM::AbstractFixedEffectSolver; progress_bar = true, kwargs...)
     iterations = Int[]
     convergeds = Bool[]
     bar = MiniProgressBar(header = "Demean Variables:", color = Base.info_color(), percentage = false, max = length(xs))

test/runtests.jl

@@ -1,7 +1,15 @@
 tests = ["types.jl", "solve.jl"]
 println("Running tests:")
 
-using Test, StatsBase, CUDA, Metal, FixedEffects, PooledArrays, CategoricalArrays
+# A work around for tests to run on older versions of Julia
+using Pkg
+if VERSION >= v"1.8"
+	Pkg.add("Metal")
+	using Metal
+end
+
+using Test, StatsBase, CUDA, FixedEffects, PooledArrays, CategoricalArrays
+
 for test in tests
 	try
 		include(test)