Skip to content
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 Chart2Music keyboard and sound accessibility features #6680

Open
wants to merge 81 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
81 commits
Select commit Hold shift + click to select a range
6fb9c65
initial commit, single scatter plot
Jul 21, 2023
87b4c5c
single scatter plot
Jul 21, 2023
64ae505
options and info
Jul 21, 2023
d37d24f
data and trace labels
Jul 21, 2023
d0467bb
false default
Jul 21, 2023
3a73776
remove debugging prints
Jul 21, 2023
b4df959
use index if no x provided
Jul 21, 2023
97b608f
Lib requirement
Jul 22, 2023
6205d62
layout destructuring
Jul 22, 2023
7e40cde
layout defaults
Jul 22, 2023
6d9185b
destructure trace
Jul 22, 2023
935cf54
test sourcetype module
Jul 22, 2023
44deae2
remove sourcetype
Jul 22, 2023
c38ab67
into es5
Jul 28, 2023
c6a27be
include mjs files
Jul 28, 2023
cac174c
transpile accessibility
Jul 30, 2023
159ca1b
accessibility in config
Jul 30, 2023
7467a47
Merge branch 'master' into chart2music
ayjayt Jan 12, 2024
08b820b
Remove special rules from webpack.config:
ayjayt Jan 12, 2024
41b0fb9
Reorder declarations to make a bit more readable
ayjayt Jan 12, 2024
94e25c3
Lint
ayjayt Jan 13, 2024
61ea8ce
Change import/export scheme to promise
ayjayt Jan 17, 2024
c9a0b91
Remove a console.log
ayjayt Jan 17, 2024
cc92639
Limit chunks to 1
ayjayt Jan 17, 2024
7baa0d3
Upgrade chart2music to support commonjs
ayjayt Jan 17, 2024
3127dee
Rewrite accessibility in es5
ayjayt Jan 17, 2024
2429e55
Merge pull request #1 from geopozo/pikul-music-import-promise
aliwelchoo Jan 17, 2024
00572e8
Fix some undef errors in accessibility
ayjayt Jan 17, 2024
30f4e11
update plot-schema diff
ayjayt Jan 17, 2024
247b301
Merge branch 'master' into pikul-music-import-promise
ayjayt Jan 25, 2024
d08adfb
Add config so dev can place closed captions:
ayjayt Jan 25, 2024
dbce5da
Merge branch 'chart2music' of https://github.com/aliwelchoo/plotly.js…
ayjayt Jan 25, 2024
e1f9563
Add more default assurance in accessibility
ayjayt Jan 25, 2024
3f4ab54
Lint
ayjayt Jan 25, 2024
92d97b7
Skip empty trace data in accessibility.enable:
ayjayt Jan 25, 2024
6b2dcc9
update plot-schema diff
ayjayt Jan 25, 2024
b5bb517
Modify how accessibility treats config vars:
ayjayt Jan 26, 2024
25ca44a
Remove jasmine ftest focus
ayjayt Jan 26, 2024
7328324
Run schema plot diff
ayjayt Jan 26, 2024
00b2750
Test if previous commits deprecate hack-fix
ayjayt Jan 26, 2024
b04bb17
Set c2music default to not make its caption div:
ayjayt Jan 26, 2024
d4bd71b
update plot-scheme diff
ayjayt Jan 26, 2024
af2ff80
Add closed captions to dev dashboard
ayjayt Jan 26, 2024
80b4cf3
Merge pull request #2 from geopozo/pikul-music-import-promise
aliwelchoo Jan 28, 2024
ee99f35
Merge branch 'master' into chart2music
ayjayt Jan 29, 2024
3841640
Merge branch 'master' into chart2music
ayjayt Feb 9, 2024
d84d108
Merge branch 'master' into chart2music
ayjayt Feb 28, 2024
0c43407
Refactor accessibility/c2m:
ayjayt Feb 28, 2024
4c75035
Lint
ayjayt Feb 28, 2024
3c49ed6
Fix import path
ayjayt Feb 28, 2024
f7be666
Merge remote-tracking branch 'origin/build-with-esbuild' into chart2m…
archmoj Feb 28, 2024
4a260af
Merge branch 'master' into chart2music
ayjayt Mar 10, 2024
f67c513
Merge branch 'master' into chart2music
ayjayt Mar 20, 2024
41baab9
Merge branch 'master' into chart2music
ayjayt Mar 27, 2024
26754b7
Merge with master
marthacryan Sep 16, 2024
ed18907
Change name of config to sonification, clean up directory
marthacryan Sep 23, 2024
1e4bcef
Spelling
marthacryan Sep 23, 2024
2c1abf4
fix directory name
marthacryan Sep 23, 2024
614fb8b
Reset package-lock
marthacryan Sep 23, 2024
5f5e599
Update package-lock.json
marthacryan Sep 23, 2024
fe9736e
Reset tests
marthacryan Sep 23, 2024
59ad5c6
Update auto-generated schema
marthacryan Sep 23, 2024
d18c584
Rename more stuff to sonification and clean up contributing
marthacryan Sep 24, 2024
043d408
Update filepath for new location of enable_sonification.js
marthacryan Sep 24, 2024
129eb3b
Code readability improvements
marthacryan Sep 24, 2024
acc8517
Add baseline for new mock
marthacryan Sep 24, 2024
c5ad240
Remove changes in existing mock
marthacryan Sep 24, 2024
0113d90
Update baseline
marthacryan Sep 24, 2024
771029e
Remove function that was introduced through incorrect merge
marthacryan Sep 24, 2024
bfbbbf1
Undo changes to jasmine tests
marthacryan Sep 25, 2024
cadfd7f
Revert "Code readability improvements"
marthacryan Sep 25, 2024
56e582d
Fix syntax
marthacryan Sep 25, 2024
8e55f98
remove comment
marthacryan Sep 25, 2024
86d75d8
Remove todo
marthacryan Sep 25, 2024
15671b0
Schema update
marthacryan Oct 1, 2024
057e067
Merge branch 'master' of github.com:plotly/plotly.js into chart2music
marthacryan Oct 1, 2024
44f7ad3
Remove accessibility wrapper directory
marthacryan Oct 1, 2024
8e3b62b
Fix filepaths
marthacryan Oct 1, 2024
aed1e02
Code cleanup
marthacryan Oct 1, 2024
1777fba
Code cleanup
marthacryan Oct 1, 2024
3a0384f
Code cleanup
marthacryan Oct 1, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -276,9 +276,9 @@ npm run baseline mock_*
```

**IMPORTANT:** the `baseline`, `test-image` and `test-export` scripts do **not** bundle the source files before
running the image tests. We recommend running `npm run watch` or `npm start` in
running the image tests. We recommend running `npm start` in
a separate tab to ensure that the most up-to-date code is used.
Also if you are adding a new mock, you may need to re-run `npm start` or `npm run watch`
Also if you are adding a new mock, you may need to re-run `npm start`
to be able to find the new mock in the browser.
To help ensure valid attributes are used in your new mock(s), please run `npm run test-mock`
or `npm run test-mock mock_name(s)` after adding new mocks or implementing any new attributes.
Expand Down
108 changes: 107 additions & 1 deletion package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,7 @@
"@turf/centroid": "^7.1.0",
"base64-arraybuffer": "^1.0.2",
"canvas-fit": "^1.5.0",
"chart2music": "^1.13.0",
"color-alpha": "1.0.4",
"color-normalize": "1.5.0",
"color-parse": "2.0.0",
Expand Down
13 changes: 7 additions & 6 deletions src/plot_api/plot_api.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ var manageArrays = require('./manage_arrays');
var helpers = require('./helpers');
var subroutines = require('./subroutines');
var editTypes = require('./edit_types');
var sonification = require('../sonification/enable_sonification');

var AX_NAME_PATTERN = require('../plots/cartesian/constants').AX_NAME_PATTERN;

Expand Down Expand Up @@ -381,10 +382,14 @@ function _doPlot(gd, data, layout, config) {
// happens outside of marginPushers where all the other automargins are
// calculated. Would be much better to separate margin calculations from
// component drawing - see https://github.com/plotly/plotly.js/issues/2704
Plots.doAutoMargin,
Plots.previousPromises
Plots.doAutoMargin
);

if(gd._context.sonification.enabled) seq.push(sonification.enable_sonification);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IS sonification key always present?


seq.push(Plots.previousPromises);


// even if everything we did was synchronous, return a promise
// so that the caller doesn't care which route we took
var plotDone = Lib.syncOrAsync(seq, gd);
Expand Down Expand Up @@ -2667,7 +2672,6 @@ function react(gd, data, layout, config) {
setPlotContext(gd, config);
configChanged = diffConfig(oldConfig, gd._context);
}

gd.data = data || [];
helpers.cleanData(gd.data);
gd.layout = layout || {};
Expand Down Expand Up @@ -2738,7 +2742,6 @@ function react(gd, data, layout, config) {

Plots.doCalcdata(gd);
subroutines.doAutoRangeAndConstraints(gd);

seq.push(function() {
return Plots.transitionFromReact(gd, restyleFlags, relayoutFlags, oldFullLayout);
});
Expand Down Expand Up @@ -3654,7 +3657,6 @@ function deleteFrames(gd, frameList) {
*/
function purge(gd) {
gd = Lib.getGraphDiv(gd);

var fullLayout = gd._fullLayout || {};
var fullData = gd._fullData || [];

Expand All @@ -3672,7 +3674,6 @@ function purge(gd) {

// in contrast to _doPlots.purge which does NOT clear _context!
delete gd._context;

return gd;
}

Expand Down
15 changes: 14 additions & 1 deletion src/plot_api/plot_config.js
Original file line number Diff line number Diff line change
Expand Up @@ -471,7 +471,20 @@ var configAttributes = {
'instead of MM/DD/YYYY). Currently `grouping` and `currency` are ignored',
'for our automatic number formatting, but can be used in custom formats.'
].join(' ')
}
},

sonification: {
valType: 'any',
dflt: {
enabled: false,
options: {},
info: {},
closedCaptions: {generate: false, elId: 'c2m-plotly-cc', elClassname: 'c2m-plotly-closed_captions'}
marthacryan marked this conversation as resolved.
Show resolved Hide resolved
},
description: ['Sonification options: whether to enable, options to pass to the library, info to pass to the library, closedCaptions to control how plotly renders the closed-captions element.',
'chart2music is supported and options here include Options and Info from https://www.chart2music.com/docs/API/Config. '
].join(' ')
},
};

var dfltConfig = {};
Expand Down
29 changes: 29 additions & 0 deletions src/sonification/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
# `plotly.js` wrapper for `chart2music`, sonified charts

This wrapper attaches a context object to `gd._context._c2m` with the following properties:

* `.options`: the options object as required by c2m
* `.info`: the info object as required by c2m
* `.ccOptions`: information about where to place the closed caption div
* `.c2mHandler`: the object returned by calling the c2m library's initializer

The first three have most of their values set by the defaults in *src/plot_api/plot_config.js*.


### index.js

**index.js** exposes the following api:

* `initC2M(gd, defaultConfig)` full resets the `c2mChart` object.
* `defaultConfig` is equal to `gd._context.sonification`, as defined in *src/plot_api/plot_config.js*.

### all_codecs.js
**all_codecs.js** agregates all the individual **_codec.js* files, all are expect to export exactly two functions: `test` and `process`.

I chose to aggregate all codecs in a separate file because it will ultimately be a 1-1 conversion if `plotly.js` adopts ES modules.

## Writing a Codec

This section is TODO

chart2music supports N types of graphs. We should figure out how how to convert as many plotly types to chart2music types. I will write descriptions for each type in the folder, if not the actual code, along with what `test` must return and what `process` must return.
5 changes: 5 additions & 0 deletions src/sonification/all_codecs.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
'use strict';

var scatterXY = require('./xy_scatter_codec');

exports.codecs = [ scatterXY ];
10 changes: 10 additions & 0 deletions src/sonification/enable_sonification.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
'use strict';

var c2mPlotly = require('.');

function enable_sonification(gd) {
// Collecting defaults
var defaultConfig = gd._context.sonification;
c2mPlotly.initC2M(gd, defaultConfig);
}
exports.enable_sonification = enable_sonification;
103 changes: 103 additions & 0 deletions src/sonification/index.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,103 @@
'use strict';

var c2m = require('chart2music');
var Fx = require('../components/fx');
var Lib = require('../lib');

var codecs = require('./all_codecs').codecs;

/* initClosedCaptionDiv: Initialize the closed caption div with the given configuration.
* This function works by either creating a new div or returning the existing div
*/
function initClosedCaptionDiv(gd, config) {
if(config.generate) {
var closedCaptions = document.createElement('div');
closedCaptions.id = config.elId;
closedCaptions.className = config.elClassname;
gd.parentNode.insertBefore(closedCaptions, gd.nextSibling); // this really might not work
return closedCaptions;
} else {
return document.getElementById(config.elId);
}
Comment on lines +19 to +21
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could simply return and drop else.

Suggested change
} else {
return document.getElementById(config.elId);
}
}
return document.getElementById(config.elId);

}

/* initC2M: Initialize the chart2music library with the given configuration.
* This function works by resetting the c2m context of the given graph div
*/
function initC2M(gd, defaultConfig) {
var c2mContext = gd._context._c2m = {};
c2mContext.options = Lib.extendDeepAll({}, defaultConfig.options);
c2mContext.info = Lib.extendDeepAll({}, defaultConfig.info);
c2mContext.ccOptions = Lib.extendDeepAll({}, defaultConfig.closedCaptions);

var labels = [];
// Set the onFocusCallback to highlight the hovered point
c2mContext.options.onFocusCallback = function(dataInfo) {
Fx.hover(gd, [{
curveNumber: labels.indexOf(dataInfo.slice),
pointNumber: dataInfo.index
}]);
};

var ccElement = initClosedCaptionDiv(gd, c2mContext.ccOptions);

// Get the chart, x, and y axis titles from the layout.
// This will be used for the closed captions.
var titleText = 'Chart';
if((gd._fullLayout.title !== undefined) && (gd._fullLayout.title.text !== undefined)) {
titleText = gd._fullLayout.title.text;
}
var xAxisText = 'X Axis';
if((gd._fullLayout.xaxis !== undefined) &&
(gd._fullLayout.xaxis.title !== undefined) &&
(gd._fullLayout.xaxis.title.text !== undefined)) {
xAxisText = gd._fullLayout.xaxis.title.text;
}
var yAxisText = 'Y Axis';
if((gd._fullLayout.yaxis !== undefined) &&
(gd._fullLayout.yaxis.title !== undefined) &&
(gd._fullLayout.yaxis.title.text !== undefined)) {
yAxisText = gd._fullLayout.yaxis.title.text;
}

// Convert the data to the format that c2m expects
var c2mData = {};
var types = [];
var fullData = gd._fullData;

// Iterate through the traces and find the codec that matches the trace
for (var trace of fullData) {
for (var codec of codecs) {
var test = codec.test(trace);
if (!test) continue;

// Generate a unique label for the trace
var label = test.name;
var labelCount = 0;
var originalLabel = label;
for (label in c2mData) {
labelCount++;
label = originalLabel + labelCount.toString();
}

labels.push(label);
types.push(test.type);
c2mData[label] = codec.process(trace);
}
}

c2mContext.c2mHandler = c2m.c2mChart({
title: titleText,
type: types,
axes: {
x: { label: xAxisText },
y: { label: yAxisText },
},
element: gd,
cc: ccElement,
data: c2mData,
options: c2mContext.options,
info: c2mContext.info
});
}
exports.initC2M = initC2M;
Loading