You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Mar 14, 2021. It is now read-only.
My idea was to implement some plotting, save, load... methods on that type. However, I think that could be accomplished on the Vector also...
It would be cool to pop up a plot of a trajectory whenever a ResultsStore is returned.
However, using ResultsStore as the primary mechanism for visualization has a few problems:
If someone else wants to use your fancy plotting code, they have to understand and create everything in ResultsStore, like Aircraft, etc.
Visualization typically involves many visualization-specific configuration options, for example velocity_vector_color="blue". It doesn't seem appropriate to store these options in ResultsStore.
In my opinion, a better pattern is to have a visualize_trajectory (or similar) function that takes in a trajectory along with visualization options and returns a displayable object. We can talk about more details if you decide to pursue this pattern.
The text was updated successfully, but these errors were encountered:
(Also, if you do end up having a ResultsStore object, you can implement automatic plotting with something like show(io::IO, mime::MIME, rs::ResultsStore) = show(io, mime, visualize_trajectory(rs)).
From #65:
It would be cool to pop up a plot of a trajectory whenever a
ResultsStore
is returned.However, using
ResultsStore
as the primary mechanism for visualization has a few problems:ResultsStore
, likeAircraft
, etc.velocity_vector_color="blue"
. It doesn't seem appropriate to store these options inResultsStore
.In my opinion, a better pattern is to have a
visualize_trajectory
(or similar) function that takes in a trajectory along with visualization options and returns adisplay
able object. We can talk about more details if you decide to pursue this pattern.The text was updated successfully, but these errors were encountered: