-
-
Notifications
You must be signed in to change notification settings - Fork 315
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
Add :solid_pattern
linestyle to allow switching between patterned and solid lines
#4570
base: master
Are you sure you want to change the base?
Conversation
Benchmark ResultsSHA: cbb5096828111f6fa63ed396a3277c7e866e9866 Warning These results are subject to substantial noise because GitHub's CI runs on shared machines that are not ideally suited for benchmarking. |
Would it make sense to special case in the shader when the numbers appear that correspond to |
I'm not sure what you mean. Currently :solid converts to nothing and results in solid line shaders. :dot, :dash, etc converts to an array which results in patterned line shaders. :solid_pattern follows the latter conversion path. There is nothing else that plays into that decision. We could set up some logic to convert :solid to an array when patterned line shaders are chosen, but that only works if you start with :dot etc then. If you start with :solid or nothing, we'd have to predict the future |
I mean turn |
Hmm, that could work too, but it would require padding position of solid lines to Point4f. (Because with linestyles the projections happen on the CPU and clipping won't work if you do |
This seems rather convoluted to set up right now, compared to the plot update branch. I'll probably try it there instead |
Description
Fixes #3693
Fixes #803
This linestyle generates a solid line pattern so it renders with shaders used for linestyles but never actually stops drawing. This enables switching between a solid pattern and other patterns (e.g. :dot, :dash, etc).
We could also adjust
:solid
to do that, though that would makelinestyle = :solid
slower thannothing
(and potentially different if there are any bugs/peculiarities in the linestyle shaders).Type of change
Checklist