Fix marker opacity of mostly null data series#1941
Fix marker opacity of mostly null data series#1941aestoltm merged 10 commits intoubccr:xdmod11.0from
Conversation
|
You set this to the 11.0.1 milestone, but to merge into main - which would be for 11.5. Please confirm which is correct and fix either the milestone or the merge branch. |
Updated all my PRs that are set to 11.0.1 milestone to merge into branch |
| $hideMarker = in_array($data_description->display_type, array('line', 'spline', 'area', 'areaspline')) | ||
| && ($values_count >= 32 || (count($yAxisDataObjectsArray) > 1 && $values_count >= 21)); | ||
| && ($values_count >= 32 || (count($yAxisDataObjectsArray) > 1 && $values_count >= 21)) | ||
| && $y_values_count > 1; |
There was a problem hiding this comment.
The pull request comments talk about not null values, but this change does not appear to be related to not null. Please explain?
There was a problem hiding this comment.
Refactored the code we discussed. The logic for the marker visibility threshold is the same as during 10.5 which should have been the implementation from the start. The only difference is we did not track non-null (visible) data points for Aggregate charts in 10.5.
* Fix marker opacity when most of the data is null * Add tracking of non-null values for aggregate plots
…ght I did this in ubccr#1941 but it is not the same logic.
* Use xdmod10.5 implementation of deciding when to show markers. I thought I did this in #1941 but it is not the same logic.
* Use xdmod10.5 implementation of deciding when to show markers. I thought I did this in ubccr#1941 but it is not the same logic.
* Use xdmod10.5 implementation of deciding when to show markers. I thought I did this in ubccr#1941 but it is not the same logic.
Description
@ryanrath noticed that data series with hidden markers (due to length of date range) do not show markers when there is only one non-null value. This fix adds an additional check to show the markers if only one non-null data point exists (regardless of the data range). The additional check only looks for one data point because if there are multiple data points within a mostly null dataset there will be "dotted" lines showing the data.
The aggregate charts needed a counter to address this issue.
Motivation and Context
Charts that have one non null value are not viewable (for long date ranges).
Tests performed
Tested on xdmod-dev
Checklist: