KomaMRI.jl icon indicating copy to clipboard operation
KomaMRI.jl copied to clipboard

Simplify ArbitraryMotion struct

Open pvillacorta opened this issue 2 months ago • 2 comments

Change

mutable struct ArbitraryMotion{T<:Real,V<:AbstractVector{T}} <: MotionModel{T}
    period_durations::AbstractVector{T}
    dx::AbstractArray{T,2}
    dy::AbstractArray{T,2}
    dz::AbstractArray{T,2}
    ux::Vector{LinearInterpolator{T,V}}
    uy::Vector{LinearInterpolator{T,V}}
    uz::Vector{LinearInterpolator{T,V}}
end 

to:

 mutable struct ArbitraryMotion{T<:Real} <: MotionModel{T}
     period_durations::AbstractVector{T}
     dx::AbstractArray{T, 2}
     dy::AbstractArray{T, 2}
     dz::AbstractArray{T, 2}
 end

Originally posted by @cncastillo in https://github.com/JuliaHealth/KomaMRI.jl/pull/184#discussion_r1561943304

pvillacorta avatar Apr 16 '24 15:04 pvillacorta