-
Notifications
You must be signed in to change notification settings - Fork 70
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
Make points
lazy
#531
Comments
Supporting mapped domains is a complication here. I think for that we need to resolve |
Also, maybe |
I think we just have struct MappedGrid{MAP, GRID, T} <: AbstractVector{T}
op::MAP
grid::GRID
end also, grids should have domains. Then one can do @inline default_broadcast(f, A, Bs...) = Base.Broadcast.broadcast_c(f, containertype(A, Bs...), A, Bs...)
function broadcast(f::Fun{S}, grid::Grid{D}) where S<:Space{D} where D
if domain(f) == domain(grid) && length(grid) ≥ ncoefficients(f) # otherwise would need clever aliasing
values(pad(f, length(grid)))
else
default_broadcast(f, grid)
end
end |
Probably better to support |
This was referenced Sep 12, 2017
This would also allow for "value-spaces" as |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
That is, return an
AbstractVector
.This has the benefit that we can do fast transforms like:
While we are at it, each point should be stored in angle variables?
Then
f(x::CosPoint)
would no longer be numerically unstable.The text was updated successfully, but these errors were encountered: