Improve metadata handler by operations.

Consecutive calls using the same arguments are cached and served from
cache. This applies for the backend setup, get_metadata and get_keys, as
these are rather expensive calls.

This may be problematic if metadata is written to the image in-between
calls.

Author: jcjgraf

vimiv/plugins/metadata_piexif.py

@@ -14,6 +14,7 @@
 """
 
 import contextlib
+import functools
 from pathlib import Path
 from typing import Any, Sequence, Iterable, Optional, Dict
 
@@ -25,6 +26,7 @@
 _logger = log.module_logger(__name__)
 
 
+@functools.lru_cache(1)
 def prepare_backend(path: Path) -> Optional[Dict[str, Any]]:
     """Extract all metadata from the image."""
     try:
@@ -78,6 +80,7 @@ def get_date_time(path: Path) -> str:
 
 
 @exif.register(exif.Operations.get_metadata)
+@functools.lru_cache(1)
 def get_metadata(path: Path, desired_keys: Sequence[str]) -> exif.MetadataDictT:
     """Get value of all desired keys."""
     metadata = prepare_backend(path)
@@ -126,6 +129,7 @@ def get_metadata(path: Path, desired_keys: Sequence[str]) -> exif.MetadataDictT:
 
 
 @exif.register(exif.Operations.get_keys)
+@functools.lru_cache(1)
 def get_keys(path: Path) -> Iterable[str]:
     """Retrieve the key of all metadata values available in the current image."""
     metadata = prepare_backend(path)

vimiv/plugins/metadata_py3exiv2.py

@@ -13,6 +13,7 @@
 """
 
 import contextlib
+import functools
 import itertools
 from pathlib import Path
 from typing import Any, Sequence, Iterable, Optional
@@ -25,6 +26,7 @@
 _logger = log.module_logger(__name__)
 
 
+@functools.lru_cache(1)
 def prepare_backend(path: Path) -> Optional[pyexiv2.ImageMetadata]:
     """Load metadata from image."""
     try:
@@ -81,6 +83,7 @@ def get_date_time(path: Path) -> str:
 
 
 @exif.register(exif.Operations.get_metadata)
+@functools.lru_cache(1)
 def get_metadata(path: Path, desired_keys: Sequence[str]) -> exif.MetadataDictT:
     """Get value of all desired keys."""
     metadata = prepare_backend(path)
@@ -120,6 +123,7 @@ def get_metadata(path: Path, desired_keys: Sequence[str]) -> exif.MetadataDictT:
 
 
 @exif.register(exif.Operations.get_keys)
+@functools.lru_cache(1)
 def get_keys(path: Path) -> Iterable[str]:
     """Retrieve the key of all metadata values available in the current image."""
     metadata = prepare_backend(path)