-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature Request Polar Grids #9
Comments
Interesting suggestions, though I don't feel like Passing numbers to the keywords to modify the frequencies of the lines is an interesting idea though, because the default values don't always fit well, so I'll see if I can add that. I'm also planning to document how one could create their own plot based on the internals of the package, when ultimate flexibility is needed. Currently, all that is needed is a 'shader', e.g. a plot that shows the sign of the real part could easily be implemented as: using DomainColoring, Colors
myshader(w) = real(w) > 0 ? colorant"red" : colorant"blue"
myplot(f, args...; kwargs...) = DomainColoring.shadedplot(f, myshader, args...; kwargs...) For But that might change a bit when we move to a Makie recipes. |
For instance, this would be a minimal example of your request: using ColorTypes
import DomainColoring as DC
function myshader(w)
gd = abs(sin(16*angle(w))*sin(5π*log(abs(w))))^0.06
mapc(c -> gd*c, DC.labsweep(angle(w)))
end
myplot(f, args...; kwargs...) = DC.shadedplot(f, myshader, args...; kwargs...) |
Since the latest commit, something like |
Most of this request has been included in the latest release, thanks! |
Additional to the Cartesian grid, which is enabled by
grid=true
it should be possible to enable grids for lines ofequal angles and lines of equal absolute values.
For example
polargrid=true
could behave likepolargrid=(24, 0.3)
, meaning all angles which are multiples of(2π / 24)
and all absolute values which are multiples of
0.3
should be highlighted.Additional keyword argument
log=true
could be used to refer the grids tolog(abs(z))
instead ofabs(z)
.The text was updated successfully, but these errors were encountered: