Conversation
|
Thanks @bclehmann, this is a great start! Pulling some convo from Discord (#1966)
These are all good extensions for this plot type! I'll review and merge this PR in now, and refinements can be made in future PRs 👍 |
|
Note that I'm renaming |
|
Also (let me know if I'm messing up here) I'm going to rename |
|
Also, dang, I'm sorry I just noticed this branch was marked "draft" this whole time 😩 I've been pretty aggressively merging PRs as I see them these days 😅 I hope I didn't mess you up too much by modifying it and merging it this morning... I'll still merge this now (because my edits expanded the scope a bit beyond vector fields) so that future PRs that refine this behavior can be small and easier to work with. Thanks again for bringing this new plot type to ScottPlot5! 🚀 |
|
|
I think the main thing that's missing is in the plottable factory, I don't think it automatically assigns a colour to the vector field (so it will be black by default). I don't think we should release a new version without that, but fortunately that's a 2 minute fix. I also wonder if the legend should have a little arrow (right now it's just a line, which isn't great). As for all the colourmap stuff, that would be cool and probably pretty easy, but maybe not essential. |
|
Good points! Thanks for your feedback.
Now that you mention it, I can see multiple sides to this. Since the
|
This example is lifted from the 4.1 cookbook: