@jupytergis/base 0.1.4 → 0.1.6

package/lib/commands.js

@@ -3,7 +3,6 @@ import { CommandIDs, icons } from './constants';
 import { CreationFormDialog } from './dialogs/formdialog';
 import { LayerBrowserWidget } from './dialogs/layerBrowserDialog';
 import { SymbologyWidget } from './dialogs/symbologyDialog';
-import { TerrainDialogWidget } from './dialogs/terrainDialog';
 /**
  * Add the commands to the application's command registry.
  */
@@ -538,29 +537,6 @@ export function addCommands(app, tracker, translator, formSchemaRegistry, layerB
             });
         }
     });
-    /**
-     * Terrain commands
-     */
-    commands.addCommand(CommandIDs.newTerrain, Object.assign({ label: trans.__('New Terrain'), isEnabled: () => {
-            return tracker.currentWidget
-                ? tracker.currentWidget.context.model.sharedModel.editable
-                : false;
-        }, execute: Private.createTerrainDialog(tracker) }, icons.get(CommandIDs.newTerrain)));
-    commands.addCommand(CommandIDs.removeTerrain, {
-        label: trans.__('Remove Terrain'),
-        isEnabled: () => {
-            return tracker.currentWidget
-                ? tracker.currentWidget.context.model.sharedModel.editable
-                : false;
-        },
-        execute: () => {
-            var _a;
-            (_a = tracker.currentWidget) === null || _a === void 0 ? void 0 : _a.context.model.setTerrain({
-                source: '',
-                exaggeration: 0
-            });
-        }
-    });
     // Console commands
     commands.addCommand(CommandIDs.toggleConsole, {
         label: trans.__('Toggle console'),
@@ -635,19 +611,6 @@ var Private;
         };
     }
     Private.createLayerBrowser = createLayerBrowser;
-    function createTerrainDialog(tracker) {
-        return async () => {
-            const current = tracker.currentWidget;
-            if (!current) {
-                return;
-            }
-            const dialog = new TerrainDialogWidget({
-                context: current.context
-            });
-            await dialog.launch();
-        };
-    }
-    Private.createTerrainDialog = createTerrainDialog;
     function createSymbologyDialog(tracker, state) {
         return async () => {
             const current = tracker.currentWidget;

package/lib/constants.d.ts

@@ -38,8 +38,6 @@ export declare namespace CommandIDs {
     const moveLayerToNewGroup = "jupytergis:moveLayerToNewGroup";
     const renameSource = "jupytergis:renameSource";
     const removeSource = "jupytergis:removeSource";
-    const newTerrain = "jupytergis:newTerrain";
-    const removeTerrain = "jupytergis:removeTerrain";
     const toggleConsole = "jupytergis:toggleConsole";
     const invokeCompleter = "jupytergis:invokeConsoleCompleter";
     const removeConsole = "jupytergis:removeConsole";

package/lib/constants.js

@@ -46,9 +46,6 @@ export var CommandIDs;
     // Source actions
     CommandIDs.renameSource = 'jupytergis:renameSource';
     CommandIDs.removeSource = 'jupytergis:removeSource';
-    // Terrain stuff
-    CommandIDs.newTerrain = 'jupytergis:newTerrain';
-    CommandIDs.removeTerrain = 'jupytergis:removeTerrain';
     // Console commands
     CommandIDs.toggleConsole = 'jupytergis:toggleConsole';
     CommandIDs.invokeCompleter = 'jupytergis:invokeConsoleCompleter';
@@ -80,7 +77,6 @@ const iconObject = {
     [CommandIDs.newVideoEntry]: { iconClass: 'fa fa-video' },
     [CommandIDs.newShapefileLayer]: { iconClass: 'fa fa-file' },
     [CommandIDs.newGeoTiffEntry]: { iconClass: 'fa fa-image' },
-    [CommandIDs.newTerrain]: { iconClass: 'fa fa-mountain' },
     [CommandIDs.symbology]: { iconClass: 'fa fa-brush' }
 };
 /**

package/lib/dialogs/components/symbology/SingleBandPseudoColor.js

@@ -2,9 +2,9 @@ import { faSpinner } from '@fortawesome/free-solid-svg-icons';
 import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
 import { Button } from '@jupyterlab/ui-components';
 import React, { useEffect, useRef, useState } from 'react';
+import { getGdal } from '../../../gdal';
 import BandRow from './BandRow';
 import StopRow from './StopRow';
-import { getGdal } from '../../../gdal';
 const SingleBandPseudoColor = ({ context, state, okSignalPromise, cancel, layerId }) => {
     const functions = ['discrete', 'linear', 'exact'];
     const stopRowsRef = useRef();
@@ -191,7 +191,7 @@ const SingleBandPseudoColor = ({ context, state, okSignalPromise, cancel, layerI
                     colorExpr.push(stop.output);
                 });
                 // fallback value
-                colorExpr.push([0, 0, 0, 1.0]);
+                colorExpr.push([0, 0, 0, 0.0]);
                 break;
             }
             case 'exact': {
@@ -208,7 +208,7 @@ const SingleBandPseudoColor = ({ context, state, okSignalPromise, cancel, layerI
                     colorExpr.push(stop.output);
                 });
                 // fallback value
-                colorExpr.push([0, 0, 0, 1.0]);
+                colorExpr.push([0, 0, 0, 0.0]);
                 break;
             }
         }
@@ -248,7 +248,9 @@ const SingleBandPseudoColor = ({ context, state, okSignalPromise, cancel, layerI
             (currentBand.stats.maximum - currentBand.stats.minimum));
     };
     return (React.createElement("div", { className: "jp-gis-layer-symbology-container" },
-        React.createElement("div", { className: "jp-gis-band-container" }, bandRows.length === 0 ? (React.createElement(FontAwesomeIcon, { icon: faSpinner })) : (React.createElement(BandRow
+        React.createElement("div", { className: "jp-gis-band-container" }, bandRows.length === 0 ? (React.createElement("div", { className: "jp-gis-band-info-loading-container" },
+            React.createElement("span", null, "Fetching band info..."),
+            React.createElement(FontAwesomeIcon, { icon: faSpinner, className: "jp-gis-loading-spinner" }))) : (React.createElement(BandRow
         // Band numbers are 1 indexed
         , { 
             // Band numbers are 1 indexed

package/lib/mainview/mainView.js

@@ -1,6 +1,6 @@
 import { JupyterGISModel } from '@jupytergis/schema';
 import { UUID } from '@lumino/coreutils';
-import { Map as OlMap, View } from 'ol';
+import { Collection, Map as OlMap, View } from 'ol';
 import { ScaleLine } from 'ol/control';
 import { GeoJSON, MVT } from 'ol/format';
 import DragAndDrop from 'ol/interaction/DragAndDrop';
@@ -37,7 +37,9 @@ export class MainView extends React.Component {
                 style: defaultStyle
             };
             const layerStyle = Object.assign({}, defaultRules);
-            if (layer.filters && layer.filters.appliedFilters.length !== 0) {
+            if (layer.filters &&
+                layer.filters.logicalOp &&
+                layer.filters.appliedFilters.length !== 0) {
                 const filterExpr = [];
                 // 'Any' and 'All' operators require more than one argument
                 // So if there's only one filter, skip that part to avoid error
@@ -303,13 +305,17 @@ export class MainView extends React.Component {
             case 'GeoJSONSource': {
                 const data = ((_a = source.parameters) === null || _a === void 0 ? void 0 : _a.data) ||
                     (await this._model.readGeoJSON((_b = source.parameters) === null || _b === void 0 ? void 0 : _b.path));
-                const format = new GeoJSON();
+                const format = new GeoJSON({
+                    featureProjection: this._Map.getView().getProjection()
+                });
                 // TODO: Don't hardcode projection
+                const featureArray = format.readFeatures(data, {
+                    dataProjection: 'EPSG:4326',
+                    featureProjection: this._Map.getView().getProjection()
+                });
+                const featureCollection = new Collection(featureArray);
                 newSource = new VectorSource({
-                    features: format.readFeatures(data, {
-                        dataProjection: 'EPSG:4326',
-                        featureProjection: this._Map.getView().getProjection()
-                    })
+                    features: featureCollection
                 });
                 break;
             }
@@ -481,9 +487,6 @@ export class MainView extends React.Component {
             }
             case 'VectorTileLayer': {
                 layerParameters = layer.parameters;
-                if (!layerParameters.color) {
-                    return;
-                }
                 newMapLayer = new VectorTileLayer({
                     opacity: layerParameters.opacity,
                     source: this._sources[layerParameters.source]

package/lib/toolbar/widget.d.ts

@@ -1,5 +1,5 @@
 import { IJGISExternalCommand, JupyterGISModel } from '@jupytergis/schema';
-import { Toolbar } from '@jupyterlab/ui-components';
+import { Toolbar, ReactiveToolbar } from '@jupyterlab/ui-components';
 import { CommandRegistry } from '@lumino/commands';
 import { Widget } from '@lumino/widgets';
 export declare const TOOLBAR_SEPARATOR_CLASS = "jGIS-Toolbar-Separator";
@@ -10,7 +10,7 @@ export declare class Separator extends Widget {
      */
     constructor();
 }
-export declare class ToolbarWidget extends Toolbar {
+export declare class ToolbarWidget extends ReactiveToolbar {
     constructor(options: ToolbarWidget.IOptions);
 }
 export declare namespace ToolbarWidget {

package/lib/toolbar/widget.js

@@ -1,5 +1,5 @@
 import { CommandToolbarButton } from '@jupyterlab/apputils';
-import { ReactWidget, Toolbar, ToolbarButton, addIcon, redoIcon, undoIcon, terminalIcon } from '@jupyterlab/ui-components';
+import { ReactWidget, ReactiveToolbar, ToolbarButton, addIcon, redoIcon, undoIcon, terminalIcon } from '@jupyterlab/ui-components';
 import { Menu, Widget } from '@lumino/widgets';
 import * as React from 'react';
 import { CommandIDs } from '../constants';
@@ -15,9 +15,9 @@ export class Separator extends Widget {
         this.addClass(TOOLBAR_SEPARATOR_CLASS);
     }
 }
-export class ToolbarWidget extends Toolbar {
+export class ToolbarWidget extends ReactiveToolbar {
     constructor(options) {
-        super(options);
+        super();
         this.addClass('jGIS-toolbar-widget');
         if (options.commands) {
             this.addItem('undo', new CommandToolbarButton({
@@ -96,7 +96,7 @@ export class ToolbarWidget extends Toolbar {
                 }
             });
             this.addItem('New', NewButton);
-            this.addItem('spacer', Toolbar.createSpacerItem());
+            this.addItem('spacer', ReactiveToolbar.createSpacerItem());
             // Users
             this.addItem('users', ReactWidget.create(React.createElement(UsersItem, { model: options.model })));
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jupytergis/base",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "description": "A JupyterLab extension for 3D modelling.",
   "keywords": [
     "jupyter",
@@ -41,7 +41,7 @@
     "@deathbeds/jupyterlab-rjsf": "^1.1.0",
     "@jupyter/docprovider": "^2.0.0",
     "@jupyter/ydoc": "^1.0.0",
-    "@jupytergis/schema": "^0.1.4",
+    "@jupytergis/schema": "^0.1.6",
     "@jupyterlab/application": "^4.0.0",
     "@jupyterlab/apputils": "^4.0.0",
     "@jupyterlab/completer": "^4.2.4",

package/style/base.css

@@ -6,7 +6,6 @@
 @import url('./dialog.css');
 @import url('./layerBrowser.css');
 @import url('./leftPanel.css');
-@import url('./terrainDialog.css');
 @import url('./filterPanel.css');
 @import url('./symbologyDialog.css');
 @import url('ol/ol.css');

package/style/symbologyDialog.css

@@ -92,3 +92,31 @@
 .jp-gis-symbology-button-container {
   padding-top: 0.5rem;
 }
+
+.jp-gis-band-info-loading-container {
+  display: flex;
+  justify-content: center;
+  align-items: center;
+  gap: 2rem;
+  margin: auto;
+}
+
+.jp-gis-loading-spinner {
+  animation: spin 4s linear infinite;
+  font-size: 1.5rem;
+  padding: 1rem 0;
+  margin: auto;
+}
+
+.jp-gis-band-info-loading-container > span {
+  font-size: 1rem;
+}
+
+@keyframes spin {
+  from {
+    transform: rotate(0deg);
+  }
+  to {
+    transform: rotate(360deg);
+  }
+}

package/lib/dialogs/terrainDialog.d.ts

@@ -1,21 +0,0 @@
-import { IJupyterGISModel } from '@jupytergis/schema';
-import { Dialog } from '@jupyterlab/apputils';
-import { DocumentRegistry } from '@jupyterlab/docregistry';
-import { PromiseDelegate } from '@lumino/coreutils';
-import { Signal } from '@lumino/signaling';
-import React from 'react';
-interface ITerrainDialogProps {
-    context: DocumentRegistry.IContext<IJupyterGISModel>;
-    okSignalPromise: PromiseDelegate<Signal<TerrainDialogWidget, null>>;
-    cancel: () => void;
-}
-declare const TerrainDialog: ({ context, okSignalPromise, cancel }: ITerrainDialogProps) => React.JSX.Element;
-export interface ITerrainDialogOptions {
-    context: DocumentRegistry.IContext<IJupyterGISModel>;
-}
-export declare class TerrainDialogWidget extends Dialog<boolean> {
-    private okSignal;
-    constructor(options: ITerrainDialogOptions);
-    resolve(index?: number): void;
-}
-export default TerrainDialog;

package/lib/dialogs/terrainDialog.js

@@ -1,60 +0,0 @@
-import { Dialog } from '@jupyterlab/apputils';
-import { PromiseDelegate } from '@lumino/coreutils';
-import { Signal } from '@lumino/signaling';
-import React, { useEffect, useRef, useState } from 'react';
-const TerrainDialog = ({ context, okSignalPromise, cancel }) => {
-    const rasterDemSources = context.model.getSourcesByType('RasterDemSource');
-    const [selectedSource, setSelectedSource] = useState(Object.keys(rasterDemSources)[0]);
-    const [exaggerationInput, setExaggerationInput] = useState(1);
-    const selectedSourceRef = useRef(selectedSource);
-    const exaggerationInputRef = useRef(exaggerationInput);
-    useEffect(() => {
-        selectedSourceRef.current = selectedSource;
-        exaggerationInputRef.current = exaggerationInput;
-    }, [selectedSource, exaggerationInput]);
-    // Handler for changing the selected option
-    const handleSourceChange = (event) => {
-        setSelectedSource(event.target.value);
-    };
-    // Handler for changing the number input
-    const handleExaggerationChange = (event) => {
-        setExaggerationInput(Number(event.target.value));
-    };
-    const handleOk = () => {
-        context.model.setTerrain({
-            source: selectedSourceRef.current,
-            exaggeration: exaggerationInputRef.current
-        });
-        cancel();
-    };
-    okSignalPromise.promise.then(okSignal => {
-        okSignal.connect(handleOk);
-    });
-    return (React.createElement("div", { className: "jp-gis-terrain-main" },
-        React.createElement("label", { className: "jp-gis-terrain-label", htmlFor: "source" }, "Source:"),
-        React.createElement("select", { id: "source", className: "jp-mod-styled", value: selectedSource, onChange: handleSourceChange, "aria-label": "Select source" }, Object.entries(rasterDemSources).map(([key, value]) => (React.createElement("option", { key: key, value: key }, value)))),
-        React.createElement("label", { className: "jp-gis-terrain-label", htmlFor: "exaggeration" }, "Exaggeration:"),
-        React.createElement("input", { id: "exaggeration", className: "jp-mod-styled", type: "number", min: 0, step: 0.1, value: exaggerationInput, onChange: handleExaggerationChange, placeholder: "Enter an exaggeration value", "aria-label": "Enter exaggeration value" })));
-};
-export class TerrainDialogWidget extends Dialog {
-    constructor(options) {
-        const cancelCallback = () => {
-            this.resolve(0);
-        };
-        const okSignalPromise = new PromiseDelegate();
-        const body = (React.createElement(TerrainDialog, { context: options.context, okSignalPromise: okSignalPromise, cancel: cancelCallback }));
-        super({ title: 'Add New Terrain', body });
-        this.id = 'jupytergis::terrain';
-        this.okSignal = new Signal(this);
-        okSignalPromise.resolve(this.okSignal);
-    }
-    resolve(index) {
-        if (index === 0) {
-            super.resolve(index);
-        }
-        if (index === 1) {
-            this.okSignal.emit(null);
-        }
-    }
-}
-export default TerrainDialog;

package/style/terrainDialog.css

@@ -1,14 +0,0 @@
-.jp-gis-terrain-main {
-  display: flex;
-  flex-direction: column;
-  gap: 0.5rem;
-  overflow: hidden;
-}
-
-.jp-gis-terrain-label {
-  margin: 0;
-  padding: 0;
-  font-weight: bold;
-  display: block;
-  position: relative;
-}