From 64f31e653d29929c89f2905e90c1c780c20b37b3 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Mon, 1 Jan 2024 17:29:47 +0000 Subject: [PATCH] [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --- ipympl/backend_nbagg.py | 4 +-- src/mpl_widget.ts | 56 +++++++++++++++++------------------ src/plugin.ts | 2 +- src/toolbar_widget.ts | 10 +++---- ui-tests/tests/ipympl.test.ts | 8 ++--- 5 files changed, 39 insertions(+), 41 deletions(-) diff --git a/ipympl/backend_nbagg.py b/ipympl/backend_nbagg.py index cd098190..36b1565a 100644 --- a/ipympl/backend_nbagg.py +++ b/ipympl/backend_nbagg.py @@ -24,7 +24,7 @@ from collections.abc import Iterable except ImportError: # Python 2.7 - from collections import Iterable + from collections.abc import Iterable import matplotlib import numpy as np @@ -91,7 +91,6 @@ def connection_info(): class Toolbar(DOMWidget, NavigationToolbar2WebAgg): - _model_module = Unicode('jupyter-matplotlib').tag(sync=True) _model_module_version = Unicode(js_semver).tag(sync=True) _model_name = Unicode('ToolbarModel').tag(sync=True) @@ -178,7 +177,6 @@ def _on_orientation_collapsed_changed(self, change): class Canvas(DOMWidget, FigureCanvasWebAggCore): - _model_module = Unicode('jupyter-matplotlib').tag(sync=True) _model_module_version = Unicode(js_semver).tag(sync=True) _model_name = Unicode('MPLCanvasModel').tag(sync=True) diff --git a/src/mpl_widget.ts b/src/mpl_widget.ts index 9fc268d7..3e9592da 100644 --- a/src/mpl_widget.ts +++ b/src/mpl_widget.ts @@ -238,7 +238,7 @@ export class MPLCanvasModel extends DOMWidgetModel { const url_creator = window.URL || window.webkitURL; const buffer = new Uint8Array( - ArrayBuffer.isView(buffers[0]) ? buffers[0].buffer : buffers[0] + ArrayBuffer.isView(buffers[0]) ? buffers[0].buffer : buffers[0], ); const blob = new Blob([buffer], { type: 'image/png' }); const image_url = url_creator.createObjectURL(blob); @@ -277,7 +277,7 @@ export class MPLCanvasModel extends DOMWidgetModel { } catch (e) { console.log( "No handler for the '" + msg_type + "' message type: ", - msg + msg, ); return; } @@ -330,7 +330,7 @@ export class MPLCanvasModel extends DOMWidgetModel { 0, 0, this.offscreen_canvas.width, - this.offscreen_canvas.height + this.offscreen_canvas.height, ); } @@ -412,23 +412,23 @@ export class MPLCanvasView extends DOMWidgetView { model_events() { this.model.on( 'change:header_visible', - this._update_header_visible.bind(this) + this._update_header_visible.bind(this), ); this.model.on( 'change:footer_visible', - this._update_footer_visible.bind(this) + this._update_footer_visible.bind(this), ); this.model.on( 'change:toolbar_visible', - this._update_toolbar_visible.bind(this) + this._update_toolbar_visible.bind(this), ); this.model.on( 'change:toolbar_position', - this._update_toolbar_position.bind(this) + this._update_toolbar_position.bind(this), ); this.model.on( 'change:_figure_label', - this._update_figure_label.bind(this) + this._update_figure_label.bind(this), ); this.model.on('change:_message', this._update_message.bind(this)); this.model.on('change:_cursor', this._update_cursor.bind(this)); @@ -461,7 +461,7 @@ export class MPLCanvasView extends DOMWidgetView { this.header.classList.add( 'jupyter-widgets', 'widget-label', - 'jupyter-matplotlib-header' + 'jupyter-matplotlib-header', ); this._update_header_visible(); this._update_figure_label(); @@ -476,7 +476,7 @@ export class MPLCanvasView extends DOMWidgetView { const canvas_container = document.createElement('div'); canvas_container.classList.add( 'jupyter-widgets', - 'jupyter-matplotlib-canvas-container' + 'jupyter-matplotlib-canvas-container', ); this.figure.appendChild(canvas_container); @@ -485,7 +485,7 @@ export class MPLCanvasView extends DOMWidgetView { canvas_div.style.clear = 'both'; canvas_div.classList.add( 'jupyter-widgets', - 'jupyter-matplotlib-canvas-div' + 'jupyter-matplotlib-canvas-div', ); canvas_div.addEventListener('keydown', this.key_event('key_press')); @@ -514,35 +514,35 @@ export class MPLCanvasView extends DOMWidgetView { top_canvas.addEventListener('dblclick', this.mouse_event('dblclick')); top_canvas.addEventListener( 'mousedown', - this.mouse_event('button_press') + this.mouse_event('button_press'), ); top_canvas.addEventListener( 'mouseup', - this.mouse_event('button_release') + this.mouse_event('button_release'), ); top_canvas.addEventListener( 'mousemove', throttle( this.mouse_event('motion_notify'), - this.model.get('pan_zoom_throttle') - ) + this.model.get('pan_zoom_throttle'), + ), ); top_canvas.addEventListener( 'mouseenter', - this.mouse_event('figure_enter') + this.mouse_event('figure_enter'), ); top_canvas.addEventListener( 'mouseleave', - this.mouse_event('figure_leave') + this.mouse_event('figure_leave'), ); top_canvas.addEventListener( 'wheel', throttle( this.mouse_event('scroll'), - this.model.get('pan_zoom_throttle') - ) + this.model.get('pan_zoom_throttle'), + ), ); top_canvas.addEventListener('wheel', (event: any) => { if (this.model.get('capture_scroll')) { @@ -568,7 +568,7 @@ export class MPLCanvasView extends DOMWidgetView { async _init_toolbar() { this.toolbar_view = (await this.create_child_view( - this.model.get('toolbar') + this.model.get('toolbar'), )) as ToolbarView; this.figure.appendChild(this.toolbar_view.el); @@ -595,7 +595,7 @@ export class MPLCanvasView extends DOMWidgetView { 0, 0, this.canvas.width, - this.canvas.height + this.canvas.height, ); } else { this.context.drawImage(this.model.offscreen_canvas, 0, 0); @@ -605,7 +605,7 @@ export class MPLCanvasView extends DOMWidgetView { 0, 0, this.top_canvas.width, - this.top_canvas.height + this.top_canvas.height, ); // Draw rubberband @@ -624,7 +624,7 @@ export class MPLCanvasView extends DOMWidgetView { this.model.get('_rubberband_x'), this.model.get('_rubberband_y'), this.model.get('_rubberband_width'), - this.model.get('_rubberband_height') + this.model.get('_rubberband_height'), ); } @@ -636,7 +636,7 @@ export class MPLCanvasView extends DOMWidgetView { this.top_canvas.height - this.resize_handle_size, // Stop this.top_canvas.width, - this.top_canvas.height + this.top_canvas.height, ); gradient.addColorStop(0, 'white'); gradient.addColorStop(1, 'black'); @@ -648,15 +648,15 @@ export class MPLCanvasView extends DOMWidgetView { this.top_context.beginPath(); this.top_context.moveTo( this.top_canvas.width, - this.top_canvas.height + this.top_canvas.height, ); this.top_context.lineTo( this.top_canvas.width, - this.top_canvas.height - this.resize_handle_size + this.top_canvas.height - this.resize_handle_size, ); this.top_context.lineTo( this.top_canvas.width - this.resize_handle_size, - this.top_canvas.height + this.top_canvas.height, ); this.top_context.closePath(); this.top_context.fill(); @@ -675,7 +675,7 @@ export class MPLCanvasView extends DOMWidgetView { this.footer.classList.add( 'jupyter-widgets', 'widget-label', - 'jupyter-matplotlib-footer' + 'jupyter-matplotlib-footer', ); this._update_footer_visible(); this._update_message(); diff --git a/src/plugin.ts b/src/plugin.ts index 1305d7c1..8fe80481 100644 --- a/src/plugin.ts +++ b/src/plugin.ts @@ -27,7 +27,7 @@ export default jupyterMatplotlibPlugin; */ function activateWidgetExtension( app: Application, - registry: IJupyterWidgetRegistry + registry: IJupyterWidgetRegistry, ): void { registry.registerWidget({ name: MODULE_NAME, diff --git a/src/toolbar_widget.ts b/src/toolbar_widget.ts index b13637fb..28e2a3eb 100644 --- a/src/toolbar_widget.ts +++ b/src/toolbar_widget.ts @@ -38,7 +38,7 @@ export class ToolbarView extends DOMWidgetView { 'jupyter-widgets', 'jupyter-matplotlib-toolbar', 'widget-container', - 'widget-box' + 'widget-box', ); // Fade-in/fade-out mode by default, the figure will decide @@ -69,14 +69,14 @@ export class ToolbarView extends DOMWidgetView { button.classList.add( 'jupyter-matplotlib-button', 'jupyter-widgets', - 'jupyter-button' + 'jupyter-button', ); button.setAttribute('href', '#'); button.setAttribute('title', tooltip); button.style.outline = 'none'; button.addEventListener( 'click', - this.toolbar_button_onclick(method_name) + this.toolbar_button_onclick(method_name), ); const icon = document.createElement('i'); @@ -162,7 +162,7 @@ export class ToolbarView extends DOMWidgetView { } set_visibility( - value: 'visible' | 'hidden' | 'fade-in-fade-out' | boolean + value: 'visible' | 'hidden' | 'fade-in-fade-out' | boolean, ): void { // For backward compatibility with the old API if (typeof value === 'boolean') { @@ -247,7 +247,7 @@ export class ToolbarView extends DOMWidgetView { this.model.on_some_change( ['button_style', '_current_action'], this.set_buttons_style, - this + this, ); } } diff --git a/ui-tests/tests/ipympl.test.ts b/ui-tests/tests/ipympl.test.ts index 8e26187d..b4c55c9b 100644 --- a/ui-tests/tests/ipympl.test.ts +++ b/ui-tests/tests/ipympl.test.ts @@ -17,7 +17,7 @@ const filterUpdateNotebooks = (item) => { const testCellOutputs = async ( page: IJupyterLabPageFixture, - tmpPath: string + tmpPath: string, ) => { const paths = klaw(path.resolve(__dirname, './notebooks'), { filter: (item) => !filterUpdateNotebooks(item), @@ -53,7 +53,7 @@ const testCellOutputs = async ( for (let c = 0; c < numCellImages; ++c) { expect(results[c]).toMatchSnapshot( getCaptureImageName(notebook, c), - { threshold: 0.4 } + { threshold: 0.4 }, ); } @@ -96,7 +96,7 @@ const testUpdates = async (page: IJupyterLabPageFixture, tmpPath: string) => { for (let i = 0; i < cellCount; i++) { expect(results[i]).toMatchSnapshot( getCaptureImageName(notebook, i), - { threshold: 0.4 } + { threshold: 0.4 }, ); } @@ -108,7 +108,7 @@ test.describe('ipympl Visual Regression', () => { test.beforeEach(async ({ page, tmpPath }) => { await page.contents.uploadDirectory( path.resolve(__dirname, './notebooks'), - tmpPath + tmpPath, ); await page.filebrowser.openDirectory(tmpPath); });