geovisio 2.10.0__py3-none-any.whl → 2.12.0__py3-none-any.whl

geovisio/__init__.py

@@ -1,14 +1,16 @@
 """GeoVisio API - Main"""
 
-__version__ = "2.10.0"
+__version__ = "2.12.0"
 
 import os
 from flask import Flask, jsonify, stream_template, send_from_directory, redirect, request, url_for
-from flask.cli import with_appcontext
+from flask.cli import with_appcontext, FlaskGroup
 from flask_cors import CORS
 from flask_compress import Compress
 from flask_babel import Babel
 from flasgger import Swagger
+import click
+import atexit
 import logging
 from logging.config import dictConfig
 
@@ -31,6 +33,7 @@ from geovisio.web import (
     prepare,
     pages,
     annotations,
+    queryables,
 )
 from geovisio.workers import runner_pictures
 
@@ -65,6 +68,21 @@ LOGGING_CONFIG = {
 dictConfig(LOGGING_CONFIG)
 
 
+def cleanup_app(app):
+    """Cleanup app before closing it"""
+    if not app.pool.closed:
+        app.pool.close()
+    if not app.long_queries_pool.closed:
+        app.long_queries_pool.close()
+    app.background_processor.stop()
+
+
+def on_app_exit(app):
+    """Explicitly cleanup before exiting, to make sure all the database connections are closed, and the background processes stopped
+    This is necessary because python 3.13 changed the way the threads bahave at interpreter shutdown"""
+    cleanup_app(app)
+
+
 # Init i18n
 def get_locale():
     try:
@@ -136,6 +154,7 @@ def create_app(test_config=None, app=None):
     app.register_blueprint(prepare.bp)
     app.register_blueprint(pages.bp)
     app.register_blueprint(annotations.bp)
+    app.register_blueprint(queryables.bp)
 
     # Register CLI comands
     app.register_blueprint(admin_cli.bp, cli_group=None)
@@ -159,6 +178,8 @@ def create_app(test_config=None, app=None):
 
     template_vars = {"API_VERSION_MAJOR_MINOR": ".".join(__version__.split(".")[0:2])}
 
+    atexit.register(on_app_exit, app=app)
+
     # Main page
     @app.route("/")
     def index():
@@ -251,3 +272,12 @@ def createDirNoFailure(directory):
         os.makedirs(directory)
     except OSError:
         pass
+
+
+@click.group(cls=FlaskGroup, create_app=create_app)
+def cli():
+    """Panoramax backend management commands"""
+    # Note: This makes it possible to do:
+    # `panoramax_backend db upgrade` instead of `flask --app geovisio db upgrade` (but the later still works)
+    # since this `cli` function is used as a project.scripts
+    pass

geovisio/admin_cli/db.py

@@ -20,7 +20,7 @@ def upgrade():
     is_flag=True,
     default=False,
     show_default=True,
-    help="rollbacks all migrations instead, meaning everything created by Geovisio in database is deleted",
+    help="rollbacks all migrations instead, meaning everything created by Panoramax in database is deleted",
 )
 @with_appcontext
 def rollback(all):

geovisio/admin_cli/user.py

@@ -6,6 +6,7 @@ from flask.cli import with_appcontext
 from geovisio.utils import db
 from geovisio.utils.auth import AccountRole
 from psycopg.rows import dict_row
+from geovisio.utils.users import delete_user_data
 
 bp = Blueprint("user", __name__)
 
@@ -18,10 +19,11 @@ class Account:
 
 @bp.cli.command("user")
 @click.argument("account_id_or_name")
-@click.option("--set-role", required=True, help="Role you want to give to the account. Must be one of: admin or user")
+@click.option("--set-role", help="Role you want to give to the account. Must be one of: admin or user")
 @click.option("--create", is_flag=True, show_default=True, default=False, help="If provided, create the account if it does not exist")
+@click.option("--delete-data", is_flag=True, show_default=True, default=False, help="If provided, delete all the user's pictures")
 @with_appcontext
-def update_user(account_id_or_name, set_role=None, create=False):
+def update_user(account_id_or_name, set_role=None, create=False, delete_data=False):
     """
     Update some information about a user.
     To identify the account, either the account_id or the account_name must be provided.
@@ -33,6 +35,9 @@ def update_user(account_id_or_name, set_role=None, create=False):
         if set_role is not None:
             update_role(cursor, account, set_role)
 
+        if delete_data:
+            delete_user_data(conn, account)
+
 
 def get_account(cursor, account_id_or_name, create):
     account_id = None

geovisio/config_app.py

@@ -9,7 +9,7 @@ from pydantic import BaseModel, EmailStr
 from pydantic.color import Color
 from pydantic.networks import HttpUrl
 import json
-from flask import Flask, current_app
+from flask import Flask
 
 from geovisio.utils import website
 from geovisio.utils.model_query import get_db_params_and_values
@@ -53,7 +53,8 @@ class DefaultConfig:
     API_WEBSITE_URL = (
         website.WEBSITE_UNDER_SAME_HOST
     )  # by default we consider that there is a panoramax website on the same host as the API
-    API_REGISTRATION_IS_OPEN = False  # tells that anyone can create an account. Only used for reference in the federation for the moment
+    API_REGISTRATION_IS_OPEN = False  # tells that anyone can create an account. Used for reference in the federation and to know if we can have a `logged-only` visibility.
+    API_DEFAULT_PICTURE_VISIBILITY = "anyone"
 
 
 def read_config(app, test_config):
@@ -96,6 +97,7 @@ def read_config(app, test_config):
         "API_ENFORCE_TOS_ACCEPTANCE",
         "API_WEBSITE_URL",
         "API_REGISTRATION_IS_OPEN",
+        "API_DEFAULT_PICTURE_VISIBILITY",
         # Picture process
         "PICTURE_PROCESS_DERIVATES_STRATEGY",
         "PICTURE_PROCESS_THREADS_LIMIT",
@@ -229,6 +231,10 @@ def read_config(app, test_config):
     if not croniter.croniter.is_valid(cron_val):
         raise Exception(f"PICTURE_PROCESS_REFRESH_CRON should be a valid cron syntax, got '{cron_val}'")
 
+    default_visibility = app.config["API_DEFAULT_PICTURE_VISIBILITY"]
+    if default_visibility not in ["anyone", "owner-only", "logged-only"]:
+        raise Exception(f"API_DEFAULT_PICTURE_VISIBILITY should be 'anyone', 'owner-only' or 'logged-only', got '{default_visibility}'")
+
     app.config["PICTURE_PROCESS_KEEP_UNBLURRED_PARTS"] = _read_bool(app.config, "PICTURE_PROCESS_KEEP_UNBLURRED_PARTS")
 
     app.config["API_ACCEPT_DUPLICATE"] = _read_bool(app.config, "API_ACCEPT_DUPLICATE")
@@ -302,6 +308,7 @@ class DBConfiguration(BaseModel):
     Not all configurations are meant to be persisted in the database"""
 
     collaborative_metadata: Optional[bool] = None
+    default_visibility: Optional[str] = None
 
 
 def persist_config(app: Flask):
@@ -312,13 +319,13 @@ def persist_config(app: Flask):
     """
     from geovisio.utils import db
     from psycopg.rows import class_row
-    from psycopg.sql import SQL, Literal
-    from psycopg.errors import UndefinedTable
+    from psycopg.sql import SQL
+    from psycopg.errors import UndefinedTable, UndefinedColumn
 
-    with db.conn(app) as conn, conn.transaction() as tr, conn.cursor(row_factory=class_row(DBConfiguration)) as cur:
+    with db.conn(app) as conn, conn.transaction(), conn.cursor(row_factory=class_row(DBConfiguration)) as cur:
         try:
-            db_config = cur.execute("SELECT * FROM configurations LIMIT 1").fetchone()
-        except UndefinedTable:
+            db_config = cur.execute("SELECT collaborative_metadata, default_visibility FROM configurations LIMIT 1").fetchone()
+        except (UndefinedTable, UndefinedColumn):
             logging.warning("Database schema has not been updated yet, configuration will not be persisted")
             return
         if not db_config:
@@ -333,6 +340,13 @@ def persist_config(app: Flask):
             logging.warning(
                 "The environment variable `API_DEFAULT_COLLABORATIVE_METADATA_EDITING` has a different value than its value in the database, it will be ignored. Update the `collaborative_metadata` field in the database if you want to change it."
             )
+        default_visibility = app.config["API_DEFAULT_PICTURE_VISIBILITY"]
+        if db_config.default_visibility is None:
+            config_to_persist.default_visibility = default_visibility
+        elif db_config.default_visibility != default_visibility and default_visibility is not None:
+            logging.warning(
+                "The environment variable `API_DEFAULT_PICTURE_VISIBILITY` has a different value than its value in the database, it will be ignored. Update the `default_visibility` field in the database if you want to change it."
+            )
 
         params_as_dict = get_db_params_and_values(config_to_persist)
         fields = params_as_dict.fields_for_set()

geovisio/db_migrations.py

@@ -67,7 +67,7 @@ ALTER DATABASE {dbName.as_string()} SET TIMEZONE TO 'UTC';"""
                 # * nor for the tests
                 if "db" not in sys.argv and "pytest" not in sys.modules:
                     raise Exception(
-                        f"""Geovisio database schema needs an update !
+                        f"""Panoramax database schema needs an update !
 
 The following database migrations are required:
   {migrationNames}
@@ -105,4 +105,4 @@ def get_yoyo_backend(dbUrl):
 
 
 def get_migrations():
-    return yoyo.read_migrations(os.path.join(os.path.dirname(__file__), "../migrations"))
+    return yoyo.read_migrations(os.path.join(os.path.dirname(__file__), "migrations"))

geovisio/migrations/20221201_01_wpCGc-initial-schema.rollback.sql

@@ -0,0 +1,9 @@
+-- initial schema
+-- depends: 
+
+DROP TABLE IF EXISTS next_sequences CASCADE;
+DROP TABLE IF EXISTS sequences_pictures CASCADE;
+DROP TABLE IF EXISTS sequences CASCADE;
+DROP TYPE IF EXISTS sequence_status;
+DROP TABLE IF EXISTS pictures CASCADE;
+DROP TYPE IF EXISTS picture_status;

geovisio/migrations/20221201_01_wpCGc-initial-schema.sql

@@ -0,0 +1,71 @@
+-- initial schema
+-- depends: 
+
+
+--
+-- Script for database setup
+--
+
+CREATE EXTENSION IF NOT EXISTS postgis;
+CREATE EXTENSION IF NOT EXISTS "uuid-ossp";
+CREATE EXTENSION IF NOT EXISTS pg_trgm;
+
+
+-- Pictures
+CREATE TYPE picture_status AS ENUM (
+	'preparing',         -- Default state
+	'broken',           -- State when an error occured during import/blurring
+	'ready',            -- State when picture is ready to serve
+	'hidden'            -- State when admin disabled picture
+);
+
+CREATE TABLE pictures(
+	id UUID PRIMARY KEY DEFAULT uuid_generate_v4(),
+	file_path VARCHAR NOT NULL,    -- Relative to instance storage path
+	status picture_status NOT NULL DEFAULT 'preparing',
+	ts TIMESTAMPTZ NOT NULL DEFAULT CURRENT_TIMESTAMP,
+	heading INT,
+	metadata JSONB,
+	geom GEOMETRY(Point, 4326) NOT NULL
+);
+
+CREATE INDEX pictures_geom_idx ON pictures USING GIST(geom);
+CREATE INDEX pictures_status_idx ON pictures(status);
+
+-- Sequences
+CREATE TYPE sequence_status AS ENUM (
+	'preparing',         -- Default state
+	'broken',           -- State when an error occured during import
+	'ready',            -- State when sequence is ready to serve
+	'hidden'            -- State when admin disabled sequence
+);
+
+CREATE TABLE sequences(
+	id UUID PRIMARY KEY DEFAULT uuid_generate_v4(),
+	folder_path VARCHAR NOT NULL,
+	status sequence_status NOT NULL DEFAULT 'preparing',
+	metadata JSONB,
+	geom GEOMETRY(LineString, 4326)
+);
+
+CREATE INDEX sequences_geom_idx ON sequences USING GIST(geom);
+CREATE INDEX sequences_status_idx ON sequences(status);
+CREATE INDEX sequences_folder_path_idx ON sequences(folder_path);
+
+-- Link between pictures and sequences
+CREATE TABLE sequences_pictures(
+	seq_id UUID NOT NULL REFERENCES sequences(id),
+	rank BIGINT NOT NULL,
+	pic_id UUID NOT NULL REFERENCES pictures(id),
+	PRIMARY KEY (seq_id, rank)
+);
+
+CREATE INDEX sequences_pictures_pic_id_idx ON sequences_pictures(pic_id);
+
+-- Link between sequences
+CREATE TABLE next_sequences(
+	seq_id UUID NOT NULL REFERENCES sequences(id),
+	rank INT NOT NULL,
+	next_seq_id UUID NOT NULL REFERENCES sequences(id),
+	PRIMARY KEY (seq_id, rank)
+);

geovisio/migrations/20221201_02_ZG8AR-camera-information.rollback.sql

@@ -0,0 +1,5 @@
+-- add-account
+-- depends: 20221201_01_wpCGc-initial-schema
+
+-- the camera info has been deprecated since versions > 2.7.1__. Newer versions uses data from Geo-picture tag reader directly.
+DROP TABLE IF EXISTS cameras;

geovisio/migrations/20221201_02_ZG8AR-camera-information.sql

@@ -0,0 +1,10 @@
+-- add-account
+-- depends: 20221201_01_wpCGc-initial-schema
+
+-- the camera info has been deprecated since versions > 2.7.1__. Newer versions uses data from Geo-picture tag reader directly.
+CREATE TABLE cameras(
+	model VARCHAR PRIMARY KEY,
+	sensor_width FLOAT NOT NULL
+);
+
+CREATE INDEX cameras_model_idx ON cameras USING GIST(model gist_trgm_ops);

geovisio/migrations/20221222_01_fsB6f-add-account.rollback.sql

@@ -0,0 +1,7 @@
+-- add-account
+-- depends: 20221201_02_ZG8AR-camera-information
+
+ALTER TABLE pictures DROP CONSTRAINT account_fk_id;
+ALTER TABLE pictures DROP COLUMN account_id;
+
+DROP TABLE IF EXISTS accounts CASCADE;

geovisio/migrations/20221222_01_fsB6f-add-account.sql

@@ -0,0 +1,33 @@
+-- add-account
+-- depends: 20221201_02_ZG8AR-camera-information
+
+CREATE TABLE accounts(
+	id UUID PRIMARY KEY DEFAULT uuid_generate_v4(),
+	name VARCHAR NOT NULL,
+	is_default BOOLEAN NOT NULL DEFAULT false,
+
+	CONSTRAINT name_unique UNIQUE (name) 
+);
+
+-- At most one account can be the default one
+CREATE UNIQUE INDEX at_most_one_default 
+ON accounts (is_default) WHERE (is_default);
+
+-- Add a link betwen account and picture
+ALTER TABLE pictures ADD COLUMN IF NOT EXISTS account_id UUID;
+ALTER TABLE pictures ADD CONSTRAINT account_fk_id FOREIGN KEY (account_id) REFERENCES accounts (id);
+
+-- Create default account and associate existing pictures to default account
+WITH default_account AS (
+	INSERT INTO accounts(name, is_default) 
+		VALUES('Default account', true)
+		ON CONFLICT DO NOTHING
+		RETURNING id
+)
+UPDATE pictures
+	SET account_id = default_account.id
+	FROM default_account
+	WHERE account_id IS NULL;
+
+-- After initial init, set NOT NULL constraint on account_id
+ALTER TABLE pictures ALTER COLUMN account_id SET NOT NULL;

geovisio/migrations/20230113_01_0co97-rm-metadata-duplicates.rollback.sql

@@ -0,0 +1,5 @@
+-- rm-metadata-duplicates
+-- depends: 20221222_01_fsB6f-add-account
+
+UPDATE pictures
+SET metadata = metadata || jsonb_build_object('lat', ST_Y(geom), 'lon', ST_X(geom), 'ts', EXTRACT(epoch FROM ts), 'heading', heading);

geovisio/migrations/20230113_01_0co97-rm-metadata-duplicates.sql

@@ -0,0 +1,5 @@
+-- rm-metadata-duplicates
+-- depends: 20221222_01_fsB6f-add-account
+
+UPDATE pictures
+SET metadata = metadata - '{lat,lon,ts,heading}'::text[];

geovisio/migrations/20230116_01_9PkjZ-add-oauth-provider.rollback.sql

@@ -0,0 +1,12 @@
+-- add-provider-oidc
+-- depends: 20230113_01_0co97-rm-metadata-duplicates
+
+DROP INDEX oauth_id_idx;
+
+ALTER TABLE accounts 
+	DROP COLUMN oauth_provider,
+	DROP COLUMN oauth_id,
+	DROP COLUMN created_at;
+
+ALTER TABLE accounts ADD CONSTRAINT name_unique UNIQUE (name);
+

geovisio/migrations/20230116_01_9PkjZ-add-oauth-provider.sql

@@ -0,0 +1,10 @@
+-- add-provider-oidc
+-- depends: 20230113_01_0co97-rm-metadata-duplicates
+
+
+ALTER TABLE accounts ADD COLUMN IF NOT EXISTS oauth_provider VARCHAR;
+ALTER TABLE accounts ADD COLUMN IF NOT EXISTS oauth_id VARCHAR;
+ALTER TABLE accounts ADD COLUMN IF NOT EXISTS created_at TIMESTAMPTZ NOT NULL DEFAULT CURRENT_TIMESTAMP;
+ALTER TABLE accounts DROP CONSTRAINT name_unique;
+
+CREATE UNIQUE INDEX oauth_id_idx ON accounts (oauth_provider, oauth_id);

geovisio/migrations/20230117_01_K71Pd-pictures-ts-index.rollback.sql

@@ -0,0 +1,4 @@
+-- pictures-ts-index
+-- depends: 20230113_01_0co97-rm-metadata-duplicates
+
+DROP INDEX IF EXISTS pictures_ts_idx;

geovisio/migrations/20230117_01_K71Pd-pictures-ts-index.sql

@@ -0,0 +1,4 @@
+-- pictures-ts-index
+-- depends: 20230113_01_0co97-rm-metadata-duplicates
+
+CREATE INDEX pictures_ts_idx ON pictures(ts);

geovisio/migrations/20230130_01_VRIv2-sequences-account.rollback.sql

@@ -0,0 +1,5 @@
+-- sequences-account
+-- depends: 20230117_01_K71Pd-pictures-ts-index
+
+-- Drop account column on sequences
+ALTER TABLE sequences DROP COLUMN account_id;

geovisio/migrations/20230130_01_VRIv2-sequences-account.sql

@@ -0,0 +1,25 @@
+-- sequences-account
+-- depends: 20230117_01_K71Pd-pictures-ts-index
+
+-- Add account column on sequences
+ALTER TABLE sequences ADD COLUMN IF NOT EXISTS account_id UUID;
+ALTER TABLE sequences ADD CONSTRAINT account_fk_id FOREIGN KEY (account_id) REFERENCES accounts (id);
+
+-- Try to fill it with sequence's first picture account
+UPDATE sequences
+SET account_id = p.account_id
+FROM (
+	SELECT DISTINCT ON (sp.seq_id) sp.seq_id, p.account_id, COUNT(*) as nb
+	FROM pictures p
+	JOIN sequences_pictures sp on sp.pic_id = p.id
+	GROUP BY sp.seq_id, p.account_id
+	ORDER BY sp.seq_id, nb DESC
+) p;
+
+-- Otherwise, use default account
+UPDATE sequences
+SET account_id = (SELECT id FROM accounts WHERE is_default)
+WHERE account_id IS NULL;
+
+-- After initial init, set NOT NULL constraint on account_id
+ALTER TABLE sequences ALTER COLUMN account_id SET NOT NULL;

geovisio/migrations/20230324_01_ba9WA-status.rollback.sql

@@ -0,0 +1,35 @@
+-- status
+-- depends: 20230116_01_9PkjZ-add-oauth-provider  20230130_01_VRIv2-sequences-account
+
+UPDATE pictures SET status = 'preparing' WHERE status::text like 'preparing-' OR status = 'waiting-for-process';
+-- Note: PG does not support removing enum values, so we need to rename and replace with another one
+
+CREATE TYPE picture_status_new AS ENUM (
+	'preparing',
+	'broken',
+	'ready',
+	'hidden'
+);
+
+
+alter table pictures ALTER COLUMN status DROP DEFAULT;
+alter table pictures ALTER COLUMN status TYPE picture_status_new USING status::text::picture_status_new;
+alter table pictures ALTER COLUMN status SET DEFAULT 'preparing';
+DROP TYPE picture_status;
+ALTER TYPE picture_status_new RENAME TO picture_status;
+
+
+CREATE TYPE sequence_status_new AS ENUM (
+	'preparing',
+	'broken',
+	'ready',
+	'hidden'
+);
+
+
+alter table sequences ALTER COLUMN status DROP DEFAULT;
+alter table sequences ALTER COLUMN status TYPE sequence_status_new USING status::text::sequence_status_new;
+alter table sequences ALTER COLUMN status SET DEFAULT 'preparing';
+
+DROP TYPE sequence_status;
+ALTER TYPE sequence_status_new RENAME TO sequence_status;

geovisio/migrations/20230324_01_ba9WA-status.sql

@@ -0,0 +1,11 @@
+-- status
+-- depends: 20230116_01_9PkjZ-add-oauth-provider  20230130_01_VRIv2-sequences-account
+
+-- type cannot be altered in a transaction, so we ask yolo not to create one
+-- transactional: false
+
+ALTER TYPE picture_status ADD VALUE 'waiting-for-process';
+ALTER TYPE picture_status ADD VALUE 'preparing-derivates';
+ALTER TYPE picture_status ADD VALUE 'preparing-blur';
+
+ALTER TYPE sequence_status ADD VALUE 'waiting-for-process';

geovisio/migrations/20230324_02_efgI6-picture-process.rollback.sql

@@ -0,0 +1,20 @@
+-- status
+-- depends: 20230324_01_ba9WA-status
+
+ALTER TABLE pictures 
+	DROP COLUMN IF EXISTS inserted_at,
+	DROP COLUMN IF EXISTS processed_at,
+	DROP COLUMN IF EXISTS nb_errors,
+	DROP COLUMN IF EXISTS process_error
+;
+
+DROP TABLE pictures_to_process;
+
+-- update default from 'waiting-for-process' to 'preparing'
+UPDATE pictures SET status = 'preparing' WHERE status = 'waiting-for-process';
+ALTER TABLE pictures ALTER COLUMN status SET DEFAULT 'preparing'; 
+
+UPDATE sequences SET status = 'preparing' WHERE status = 'waiting-for-process';
+ALTER TABLE sequences ALTER COLUMN status SET DEFAULT 'preparing'; 
+
+DROP FUNCTION picture_insertion CASCADE;

geovisio/migrations/20230324_02_efgI6-picture-process.sql

@@ -0,0 +1,44 @@
+-- status
+-- depends: 20230324_01_ba9WA-status
+
+ALTER TABLE pictures 
+	ADD COLUMN IF NOT EXISTS inserted_at TIMESTAMPTZ NOT NULL DEFAULT CURRENT_TIMESTAMP,
+	ADD COLUMN IF NOT EXISTS processed_at TIMESTAMPTZ,
+	ADD COLUMN IF NOT EXISTS nb_errors INT NOT NULL DEFAULT 0,
+	ADD COLUMN IF NOT EXISTS process_error VARCHAR
+;
+
+CREATE TABLE pictures_to_process(
+	picture_id UUID PRIMARY KEY,
+	ts TIMESTAMPTZ NOT NULL DEFAULT CURRENT_TIMESTAMP
+);
+ALTER TABLE pictures_to_process ADD CONSTRAINT picture_id_fk FOREIGN KEY (picture_id) REFERENCES pictures (id);
+
+CREATE INDEX pictures_to_process_ts_idx ON pictures_to_process(ts);
+
+-- update default from 'preparing' to 'waiting-for-process'
+UPDATE pictures SET status = 'waiting-for-process' WHERE status = 'preparing';
+ALTER TABLE pictures ALTER COLUMN status SET DEFAULT 'waiting-for-process'; 
+
+CREATE INDEX pictures_processed_at_idx ON pictures(inserted_at);
+
+UPDATE sequences SET status = 'waiting-for-process' WHERE status = 'preparing';
+ALTER TABLE sequences ALTER COLUMN status SET DEFAULT 'waiting-for-process'; 
+
+-- Create Trigger to insert each new picture into pictures_to_process
+CREATE OR REPLACE FUNCTION picture_insertion() RETURNS TRIGGER AS
+$BODY$
+BEGIN
+    INSERT INTO
+        pictures_to_process(picture_id)
+	VALUES 
+		(new.id);
+	RETURN new;
+END;
+$BODY$
+language plpgsql;
+
+CREATE TRIGGER trigger_process_picture_insertion
+     AFTER INSERT ON pictures
+     FOR EACH ROW
+     EXECUTE PROCEDURE picture_insertion();

geovisio/migrations/20230407_01_wofh1-computed-headings.rollback.sql

@@ -0,0 +1,6 @@
+-- computed_headings
+-- depends: 20230324_02_efgI6-picture-process
+
+-- We need to be able to tell if the heading have been computed or given as input
+
+ALTER TABLE pictures DROP COLUMN IF EXISTS heading_computed;

geovisio/migrations/20230407_01_wofh1-computed-headings.sql

@@ -0,0 +1,6 @@
+-- computed_headings
+-- depends: 20230324_02_efgI6-picture-process
+
+-- We need to be able to tell if the heading have been computed or given as input
+
+ALTER TABLE pictures ADD COLUMN IF NOT EXISTS heading_computed BOOLEAN NOT NULL DEFAULT false;

geovisio/migrations/20230417_01_ZgLMY-add-exif-metadata-column-for-pictures.rollback.sql

@@ -0,0 +1,4 @@
+-- Add exif metadata column for pictures
+-- depends: 20230407_01_wofh1-computed-headings
+
+ALTER TABLE pictures DROP COLUMN exif;

geovisio/migrations/20230417_01_ZgLMY-add-exif-metadata-column-for-pictures.sql

@@ -0,0 +1,4 @@
+-- Add exif metadata column for pictures
+-- depends: 20230407_01_wofh1-computed-headings
+
+ALTER TABLE pictures ADD COLUMN exif JSONB;

geovisio/migrations/20230420_01_elaN3-remove-picture-and-sequence-file-paths.rollback.sql

@@ -0,0 +1,5 @@
+-- Remove picture and sequence file paths
+-- depends: 20230417_01_ZgLMY-add-exif-metadata-column-for-pictures
+
+ALTER TABLE pictures ADD COLUMN file_path VARCHAR;
+ALTER TABLE sequences ADD COLUMN folder_path VARCHAR;

geovisio/migrations/20230420_01_elaN3-remove-picture-and-sequence-file-paths.sql

@@ -0,0 +1,5 @@
+-- Remove picture and sequence file paths
+-- depends: 20230417_01_ZgLMY-add-exif-metadata-column-for-pictures
+
+ALTER TABLE pictures DROP COLUMN IF EXISTS file_path;
+ALTER TABLE sequences DROP COLUMN IF EXISTS folder_path;

geovisio/migrations/20230425_01_gYP77-pictures-edits-triggers.rollback.sql

@@ -0,0 +1,5 @@
+-- pictures_edits_triggers
+-- depends: 20230420_01_elaN3-remove-picture-and-sequence-file-paths
+
+DROP FUNCTION IF EXISTS pictures_update_sequence CASCADE;
+DROP FUNCTION IF EXISTS sequences_pictures_delete CASCADE;