psi icon indicating copy to clipboard operation
psi copied to clipboard

Extending CartesianChartVisualizationObject with more properties

Open danbohus opened this issue 3 years ago • 0 comments

(this issue was created based on issue #120)

The CartesianChartVisualizationObject from Microsoft.Psi.LiveCharts.Visualization leverages LiveCharts to visualize chart data. The initial implementation of this visualization object we have provided does not yet expose all the properties that the underlying charting technology has. As one example, the LiveCharts documentation available here indicates that one should be able to set the axis min and max values. There are a number of other unexposed properties as well of the underlying Axis, for instance FontFamily, FontSize, FontWeight, LabelsRotation, etc., on the Chart itself, on the underlying Axis.Separator object, etc. More information on the various available properties can be found in the livecharts documentation and also in the LiveCharts GitHub repository.

We would like to extend the CartesianChartVisualizationObject to expose these properties and thus make them user-editable when using this visualizer from Platform for Situated Intelligence studio. An example of the pattern used to expose these properties is given by the currently-exposed AxisXTitle, AxisYTitle and DisableAnimations properties. A corresponding property needs to be created in the CartesianChartVisualizationObject.cs and the necessary bindings need to be constructed in the CartesianChartVisualizationObjectView.xaml -- see for instance the bindings to the AxisXTitle, AxisYTitle and DisableAnimations.

This issue represents a good community contribution opportunity, and is labeled 'help-wanted'. If you decide you would like to develop this feature and contribute it to the codebase, we would be happy to consider/review a pull request. If you have further questions, please post them below. To avoid duplication of work, if you start working on this with the goal of creating a pull request, please mention this by replying to this issue.

danbohus avatar Apr 06 '21 15:04 danbohus