disdrodb 0.0.20__py3-none-any.whl → 0.1.0__py3-none-any.whl

disdrodb/data_transfer/upload_data.py

@@ -18,17 +18,20 @@
 # -----------------------------------------------------------------------------.
 """Routines to upload data to the DISDRODB Decentralized Data Archive."""
 
-from typing import List, Optional
+from typing import Optional
 
 import click
 
 from disdrodb.api.path import define_metadata_filepath
+from disdrodb.configs import get_data_archive_dir, get_metadata_archive_dir
 from disdrodb.data_transfer.zenodo import upload_station_to_zenodo
-from disdrodb.metadata import get_list_metadata
+from disdrodb.metadata.search import get_list_metadata
+from disdrodb.utils.compression import archive_station_data
 from disdrodb.utils.yaml import read_yaml
 
 
 def click_upload_options(function: object):
+    """Click command arguments for DISDRODB data upload."""
     function = click.option(
         "--platform",
         type=click.Choice(["zenodo", "sandbox.zenodo"], case_sensitive=False),
@@ -95,7 +98,7 @@ def _check_if_upload(metadata_filepath: str, force: bool):
             raise ValueError(f"'force' is False and {metadata_filepath} has already a 'disdrodb_data_url' specified.")
 
 
-def _filter_already_uploaded(metadata_filepaths: List[str], force: bool) -> List[str]:
+def _filter_already_uploaded(metadata_filepaths: list[str], force: bool) -> list[str]:
     """Filter metadata files that already have a remote url specified."""
     filtered = []
     for metadata_filepath in metadata_filepaths:
@@ -124,7 +127,8 @@ def upload_station(
     station_name: str,
     platform: Optional[str] = "sandbox.zenodo",
     force: bool = False,
-    base_dir: Optional[str] = None,
+    data_archive_dir: Optional[str] = None,
+    metadata_archive_dir: Optional[str] = None,
 ) -> None:
     """
     Upload data from a single DISDRODB station on a remote repository.
@@ -141,45 +145,57 @@ def upload_station(
         The name of the campaign. Must be provided in UPPER CASE.
     station_name : str
         The name of the station.
-    base_dir : str, optional
-        The base directory of DISDRODB, expected in the format ``<...>/DISDRODB``.
-        If not specified, the path specified in the DISDRODB active configuration will be used.
+    data_archive_dir : str (optional)
+        The directory path where the DISDRODB Data Archive is located.
+        The directory path must end with ``<...>/DISDRODB``.
+        If ``None``, it uses the ``data_archive_dir`` path specified
+        in the DISDRODB active configuration.
     platform: str, optional
-        Name of the remote platform.
-        The default platform is "sandbox.zenodo"
+        Name of the remote data storage platform.
+        The default platform is ``"sandbox.zenodo"`` (for testing purposes).
+        Switch to ``"zenodo"`` for final data dissemination.
     force: bool, optional
-        If True, upload the data and overwrite the disdrodb_data_url.
-        The default is force=False.
+        If ``True``, upload the data and overwrite the ``disdrodb_data_url``.
+        The default value is ``force=False``.
 
     """
+    # Retrieve the DISDRODB Metadata and Data Archive Directories
+    data_archive_dir = get_data_archive_dir(data_archive_dir)
+    metadata_archive_dir = get_metadata_archive_dir(metadata_archive_dir)
+
+    # Check valid platform
     _check_valid_platform(platform)
 
     # Define metadata_filepath
     metadata_filepath = define_metadata_filepath(
+        metadata_archive_dir=metadata_archive_dir,
         data_source=data_source,
         campaign_name=campaign_name,
         station_name=station_name,
-        base_dir=base_dir,
-        product="RAW",
         check_exists=True,
     )
     # Check if data must be uploaded
     _check_if_upload(metadata_filepath, force=force)
 
-    print(f"Start uploading of {data_source} {campaign_name} {station_name}")
+    # Zip station data
+    print(f" - Zipping station data  of {data_source} {campaign_name} {station_name}")
+    station_zip_filepath = archive_station_data(metadata_filepath, data_archive_dir=data_archive_dir)
+
+    print(f" - Start uploading of {data_source} {campaign_name} {station_name}")
     # Upload the data
     if platform == "zenodo":
-        upload_station_to_zenodo(metadata_filepath, sandbox=False)
+        upload_station_to_zenodo(metadata_filepath, station_zip_filepath=station_zip_filepath, sandbox=False)
 
     else:  # platform == "sandbox.zenodo":  # Only for testing purposes, not available through CLI
-        upload_station_to_zenodo(metadata_filepath, sandbox=True)
+        upload_station_to_zenodo(metadata_filepath, station_zip_filepath=station_zip_filepath, sandbox=True)
 
 
 def upload_archive(
     platform: Optional[str] = None,
     force: bool = False,
-    base_dir: Optional[str] = None,
-    **kwargs,
+    data_archive_dir: Optional[str] = None,
+    metadata_archive_dir: Optional[str] = None,
+    **fields_kwargs,
 ) -> None:
     """Find all stations containing local data and upload them to a remote repository.
 
@@ -187,39 +203,49 @@ def upload_archive(
     ----------
     platform: str, optional
         Name of the remote platform.
-        If not provided (None), the default platform is Zenodo.
-        The default is platform=None.
+        The default platform is ``"sandbox.zenodo"`` (for testing purposes).
+        Switch to ``"zenodo"`` for final data dissemination.
     force: bool, optional
-        If True, upload even if data already exists on another remote location.
-        The default is force=False.
-    base_dir : str (optional)
-        Base directory of DISDRODB. Format: <...>/DISDRODB
-        If None (the default), the disdrodb config variable 'dir' is used.
+        If ``True``, upload even if data already exists on another remote location.
+        The default value is ``force=False``.
+    data_archive_dir : str (optional)
+        The directory path where the DISDRODB Data Archive is located.
+        The directory path must end with ``<...>/DISDRODB``.
+        If ``None``, it uses the ``data_archive_dir`` path specified
+        in the DISDRODB active configuration.
 
     Other Parameters
     ----------------
-
     data_sources: str or list of str, optional
         Data source name (eg: EPFL).
-        If not provided (None), all data sources will be uploaded.
-        The default is data_source=None.
+        If not provided (``None``), all data sources will be uploaded.
+        The default value is ``data_source=None``.
     campaign_names: str or list of str, optional
         Campaign name (eg:  EPFL_ROOF_2012).
-        If not provided (None), all campaigns will be uploaded.
-        The default is campaign_name=None.
+        If not provided (``None``), all campaigns will be uploaded.
+        The default value is ``campaign_name=None``.
     station_names: str or list of str, optional
         Station name.
-        If not provided (None), all stations will be uploaded.
-        The default is station_name=None.
+        If not provided (``None``), all stations will be uploaded.
+        The default value is ``station_name=None``.
     """
     _check_valid_platform(platform)
 
-    # Get list metadata
+    # Retrieve the DISDRODB Metadata and Data Archive Directories
+    data_archive_dir = get_data_archive_dir(data_archive_dir)
+    metadata_archive_dir = get_metadata_archive_dir(metadata_archive_dir)
+
+    # Retrieve only metadata_filepaths of stations with RAW data in the local DISDRODB Data Archive
     metadata_filepaths = get_list_metadata(
-        **kwargs,
-        base_dir=base_dir,
-        with_stations_data=True,
+        metadata_archive_dir=metadata_archive_dir,
+        data_archive_dir=data_archive_dir,
+        product="RAW",  # --> Search in local DISDRODB Data Archive
+        available_data=True,  # --> Select only stations with raw data
+        raise_error_if_empty=False,  # Do not raise error if no matching metadata file found
+        invalid_fields_policy="raise",  # Raise error if invalid filtering criteria are specified
+        **fields_kwargs,  # data_sources, campaign_names, station_names
     )
+
     # If force=False, keep only metadata without disdrodb_data_url
     if not force:
         metadata_filepaths = _filter_already_uploaded(metadata_filepaths, force=force)
@@ -237,7 +263,8 @@ def upload_archive(
         station_name = metadata["station_name"]
         try:
             upload_station(
-                base_dir=base_dir,
+                data_archive_dir=data_archive_dir,
+                metadata_archive_dir=metadata_archive_dir,
                 data_source=data_source,
                 campaign_name=campaign_name,
                 station_name=station_name,

disdrodb/data_transfer/zenodo.py

@@ -20,12 +20,10 @@
 
 import json
 import os
-from typing import Tuple
 
 import requests
 
 from disdrodb.configs import get_zenodo_token
-from disdrodb.utils.compression import archive_station_data
 from disdrodb.utils.yaml import read_yaml, write_yaml
 
 
@@ -35,7 +33,6 @@ def _check_http_response(
     task_description: str,
 ) -> None:
     """Check the Zenodo HTTP request response status code and raise an error if not the expected one."""
-
     if response.status_code == expected_status_code:
         return
 
@@ -52,7 +49,7 @@ def _check_http_response(
     raise ValueError(error_message)
 
 
-def _create_zenodo_deposition(sandbox) -> Tuple[int, str]:
+def _create_zenodo_deposition(sandbox) -> tuple[int, str]:
     """Create a new Zenodo deposition and get the deposit information.
 
     At every function call, the deposit_id and bucket url will change !
@@ -60,8 +57,8 @@ def _create_zenodo_deposition(sandbox) -> Tuple[int, str]:
     Parameters
     ----------
     sandbox : bool
-        If True, create the deposit on Zenodo Sandbox for testing purposes.
-        If False, create the deposit on Zenodo.
+        If ``True``, create the deposit on Zenodo Sandbox for testing purposes.
+        If ``False``, create the deposit on Zenodo.
 
     Returns
     -------
@@ -96,7 +93,6 @@ def _define_disdrodb_data_url(zenodo_host, deposit_id, filename):
 
 def _upload_file_to_zenodo(filepath: str, metadata_filepath: str, sandbox: bool) -> None:
     """Upload a file to a Zenodo bucket."""
-
     # Read metadata
     metadata = read_yaml(metadata_filepath)
     data_source = metadata["data_source"]
@@ -186,7 +182,7 @@ def _define_zenodo_metadata(metadata):
     description = f"Disdrometer measurements of the {name} station. "
     description += "This dataset is part of the DISDRODB project. "
     description += "Station metadata are available at "
-    description += f"https://github.com/ltelab/disdrodb-data/blob/main/DISDRODB/Raw/{data_source}/{campaign_name}/metadata/{station_name}.yml . "  # noqa
+    description += f"https://github.com/ltelab/DISDRODB-METADATA/blob/main/DISDRODB/METADATA/{data_source}/{campaign_name}/metadata/{station_name}.yml . "  # noqa: E501
     description += "The software to easily process and standardize the raw data into netCDF files is available at "
     description += "https://github.com/ltelab/disdrodb ."
 
@@ -196,7 +192,7 @@ def _define_zenodo_metadata(metadata):
             "upload_type": "dataset",
             "description": description,
             "creators": _define_creators_list(metadata),
-        }
+        },
     }
     return zenodo_metadata
 
@@ -207,7 +203,7 @@ def _update_metadata_with_zenodo_url(metadata_filepath: str, disdrodb_data_url:
     Parameters
     ----------
     metadata_filepath: str
-        Metadata file path.
+        Path to the station metadata file.
     disdrodb_data_url: str
         Remote URL where the station data are stored.
     """
@@ -216,26 +212,27 @@ def _update_metadata_with_zenodo_url(metadata_filepath: str, disdrodb_data_url:
     write_yaml(metadata_dict, metadata_filepath)
 
 
-def upload_station_to_zenodo(metadata_filepath: str, sandbox: bool = True) -> str:
+def upload_station_to_zenodo(metadata_filepath: str, station_zip_filepath: str, sandbox: bool = True) -> str:
     """Zip station data, upload data to Zenodo and update the metadata disdrodb_data_url.
 
     Parameters
     ----------
     metadata_filepath: str
-        Metadata file path.
+        Path to the station metadata file.
+    station_zip_filepath: str
+        Path to the zip file containing the station data.
     sandbox: bool
-        If True, upload to Zenodo Sandbox for testing purposes.
+        If ``True``, upload to Zenodo Sandbox (for testing purposes).
+        If ``False``, upload to Zenodo.
     """
-    # Zip station data
-    print(" - Zipping station data")
-    station_zip_filepath = archive_station_data(metadata_filepath)
-
     # Upload the station data zip file on Zenodo
     # - After upload, it removes the zip file !
     print(" - Uploading station data")
     try:
         disdrodb_data_url = _upload_file_to_zenodo(
-            filepath=station_zip_filepath, metadata_filepath=metadata_filepath, sandbox=sandbox
+            filepath=station_zip_filepath,
+            metadata_filepath=metadata_filepath,
+            sandbox=sandbox,
         )
         os.remove(station_zip_filepath)
     except Exception as e:

disdrodb/docs.py

@@ -25,7 +25,7 @@ from disdrodb.api.checks import check_sensor_name
 
 
 def open_sensor_documentation(sensor_name):
-    """Open the sensor documentation pdf in the browser."""
+    """Open the sensor documentation PDF in the browser."""
     from disdrodb import __root_path__
 
     check_sensor_name(sensor_name)

disdrodb/issue/__init__.py

@@ -1,9 +1,22 @@
 #!/usr/bin/env python3
-"""
-Created on Sat Nov 11 01:12:40 2023
 
-@author: ghiggi
-"""
+# -----------------------------------------------------------------------------.
+# Copyright (c) 2021-2023 DISDRODB developers
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+# -----------------------------------------------------------------------------.
+"""DISDRODB issue files module."""
 
 from disdrodb.issue.reader import read_station_issue
 

disdrodb/issue/checks.py

@@ -22,26 +22,22 @@ import logging
 import numpy as np
 import pandas as pd
 
-from disdrodb.utils.logger import log_error
-
 logger = logging.getLogger(__name__)
 
 
 def _is_numpy_array_string(arr):
-    """Check if the numpy array contains strings
+    """Check if the numpy array contains strings.
 
     Parameters
     ----------
     arr : numpy array
         Numpy array to check.
     """
-
-    dtype = arr.dtype.type
-    return dtype == np.str_ or dtype == np.unicode_
+    return np.issubdtype(arr.dtype, np.str_)
 
 
 def _is_numpy_array_datetime(arr):
-    """Check if the numpy array contains datetime64
+    """Check if the numpy array contains datetime64.
 
     Parameters
     ----------
@@ -53,7 +49,7 @@ def _is_numpy_array_datetime(arr):
     numpy array
         Numpy array checked.
     """
-    return arr.dtype.type == np.datetime64
+    return np.issubdtype(arr.dtype, np.datetime64)
 
 
 def _check_timestep_datetime_accuracy(timesteps, unit="s"):
@@ -64,7 +60,7 @@ def _check_timestep_datetime_accuracy(timesteps, unit="s"):
     timesteps : numpy array
         Numpy array to check.
     unit : str, optional
-        Unit, by default "s"
+        Time unit. The default value is "s".
 
     Returns
     -------
@@ -75,14 +71,13 @@ def _check_timestep_datetime_accuracy(timesteps, unit="s"):
     ------
     ValueError
     """
-    if not timesteps.dtype == f"<M8[{unit}]":
+    if timesteps.dtype != f"<M8[{unit}]":
         msg = f"The timesteps does not have datetime64 {unit} accuracy."
-        log_error(logger, msg=msg, verbose=False)
         raise ValueError(msg)
     return timesteps
 
 
-def _check_timestep_string_second_accuracy(timesteps, n=19):
+def _check_timestep_string_second_accuracy(timesteps):
     """Check the timesteps string are provided with second accuracy.
 
     Note: it assumes the YYYY-mm-dd HH:MM:SS format
@@ -94,7 +89,6 @@ def _check_timestep_string_second_accuracy(timesteps, n=19):
             f"The following timesteps are mispecified: {mispecified_timesteps}. Expecting the YYYY-mm-dd HH:MM:SS"
             " format."
         )
-        log_error(logger, msg=msg, verbose=False)
         raise ValueError(msg)
     return timesteps
 
@@ -116,7 +110,6 @@ def _check_timesteps_string(timesteps):
             f"The following timesteps are mispecified: {mispecified_timesteps}. Expecting the YYYY-mm-dd HH:MM:SS"
             " format."
         )
-        log_error(logger, msg=msg, verbose=False)
         raise ValueError(msg)
     # Convert to numpy
     new_timesteps = new_timesteps.to_numpy()
@@ -148,18 +141,14 @@ def check_timesteps(timesteps):
 
 def _check_time_period_nested_list_format(time_periods):
     """Check that the time_periods is a list of list of length 2."""
-
     if not isinstance(time_periods, list):
         msg = "'time_periods' must be a list'"
-        log_error(logger, msg=msg, verbose=False)
         raise TypeError(msg)
 
     for time_period in time_periods:
         if not isinstance(time_period, (list, np.ndarray)) or len(time_period) != 2:
             msg = "Every time period of time_periods must be a list of length 2."
-            log_error(logger, msg=msg, verbose=False)
             raise ValueError(msg)
-    return None
 
 
 def check_time_periods(time_periods):
@@ -178,7 +167,6 @@ def check_time_periods(time_periods):
     for time_period in new_time_periods:
         if time_period[0] > time_period[1]:
             msg = f"The {time_period} time_period is invalid. Start time occurs after end time."
-            log_error(logger, msg=msg, verbose=False)
             raise ValueError(msg)
     return new_time_periods
 
@@ -201,7 +189,7 @@ def _get_issue_time_periods(issue_dict):
 
 
 def check_issue_dict(issue_dict):
-    """Check validity of the issue dictionary"""
+    """Check validity of the issue dictionary."""
     # Check is empty
     if len(issue_dict) == 0:
         return issue_dict
@@ -211,7 +199,6 @@ def check_issue_dict(issue_dict):
     invalid_keys = [k for k in keys if k not in valid_keys]
     if len(invalid_keys) > 0:
         msg = f"Invalid {invalid_keys} keys. The issue YAML file accept only {valid_keys}"
-        log_error(logger, msg=msg, verbose=False)
         raise ValueError(msg)
 
     # Check timesteps
@@ -225,12 +212,12 @@ def check_issue_dict(issue_dict):
     return issue_dict
 
 
-def check_issue_compliance(data_source, campaign_name, station_name, base_dir=None):
+def check_issue_compliance(data_source, campaign_name, station_name, metadata_archive_dir=None):
     """Check DISDRODB issue compliance."""
     from disdrodb.issue.reader import read_station_issue
 
     issue_dict = read_station_issue(
-        base_dir=base_dir,
+        metadata_archive_dir=metadata_archive_dir,
         data_source=data_source,
         campaign_name=campaign_name,
         station_name=station_name,

disdrodb/issue/reader.py

@@ -28,16 +28,16 @@ logger = logging.getLogger(__name__)
 
 
 class NoDatesSafeLoader(yaml.SafeLoader):
+    """A YAML loader that does not parse dates."""
+
     @classmethod
     def remove_implicit_resolver(cls, tag_to_remove):
         """
-        Remove implicit resolvers for a particular tag
+        Remove implicit resolvers for a particular tag.
 
         Takes care not to modify resolvers in super classes.
 
-        We want to load datetimes as strings, not dates, because we
-        go on to serialise as json which doesn't have the advanced types
-        of yaml, and leads to incompatibilities down the track.
+        We want to load datetimes as strings, not dates.
         """
         if "yaml_implicit_resolvers" not in cls.__dict__:
             cls.yaml_implicit_resolvers = cls.yaml_implicit_resolvers.copy()
@@ -49,7 +49,7 @@ class NoDatesSafeLoader(yaml.SafeLoader):
 
 
 def _load_yaml_without_date_parsing(filepath):
-    "Read a YAML file without converting automatically date string to datetime."
+    """Read a YAML file without converting automatically date string to datetime."""
     NoDatesSafeLoader.remove_implicit_resolver("tag:yaml.org,2002:timestamp")
     with open(filepath) as f:
         dictionary = yaml.load(f, Loader=NoDatesSafeLoader)
@@ -77,7 +77,7 @@ def read_issue(filepath: str) -> dict:
     return issue_dict
 
 
-def read_station_issue(data_source, campaign_name, station_name, base_dir=None):
+def read_station_issue(data_source, campaign_name, station_name, metadata_archive_dir=None):
     """Open the station issue YAML file into a dictionary.
 
     Parameters
@@ -90,12 +90,9 @@ def read_station_issue(data_source, campaign_name, station_name, base_dir=None):
         The name of the campaign. Must be provided in UPPER CASE.
     station_name : str
         The name of the station.
-    base_dir : str, optional
+    data_archive_dir : str, optional
         The base directory of DISDRODB, expected in the format ``<...>/DISDRODB``.
-        If not specified, the path specified in the DISDRODB active configuration will be used.
-    product : str, optional
-        The DISDRODB product in which to search for the metadata file.
-        The default is "RAW".
+        If not specified, the ``data_archive_dir`` path specified in the DISDRODB active configuration will be used.
 
     Returns
     -------
@@ -105,7 +102,7 @@ def read_station_issue(data_source, campaign_name, station_name, base_dir=None):
     """
     # Retrieve metadata filepath
     issue_filepath = define_issue_filepath(
-        base_dir=base_dir,
+        metadata_archive_dir=metadata_archive_dir,
         data_source=data_source,
         campaign_name=campaign_name,
         station_name=station_name,

disdrodb/issue/writer.py

@@ -20,6 +20,7 @@
 
 import logging
 import os
+from typing import Optional
 
 import yaml
 
@@ -29,8 +30,9 @@ logger = logging.getLogger(__name__)
 
 
 def _write_issue_docs(f):
-    """Provide template for issue.yml"""
-    f.write("""# This file is used to store timesteps/time periods with wrong/corrupted observation.
+    """Provide template for issue.yml."""
+    f.write(
+        """# This file is used to store timesteps/time periods with wrong/corrupted observation.
 # The specified timesteps are dropped during the L0 processing.
 # The time format used is the isoformat : YYYY-mm-dd HH:MM:SS.
 # The 'timesteps' key enable to specify the list of timesteps to be discarded.
@@ -46,11 +48,11 @@ def _write_issue_docs(f):
 # - ['2018-08-01 12:00:00', '2018-08-01 14:00:00']
 # - ['2018-08-01 15:44:30', '2018-08-01 15:59:31']
 # - ['2018-08-02 12:44:30', '2018-08-02 12:59:31'] \n
-""")
-    return None
+""",
+    )
 
 
-def write_issue(filepath: str, timesteps: list = None, time_periods: list = None) -> None:
+def write_issue(filepath: str, timesteps: Optional[list] = None, time_periods: Optional[list] = None) -> None:
     """Write the issue YAML file.
 
     Parameters
@@ -59,20 +61,17 @@ def write_issue(filepath: str, timesteps: list = None, time_periods: list = None
         Filepath of the issue YAML to write.
     timesteps : list, optional
         List of timesteps (to be dropped in L0 processing).
-        The default is None.
+        The default value is ``None``..
     time_periods : list, optional
         A list of time periods (to be dropped in L0 processing).
-        The default is None.
+        The default value is ``None``..
     """
     # Preprocess timesteps and time_periods (to plain list of strings)
     if timesteps is not None:
         timesteps = timesteps.astype(str).tolist()
 
     if time_periods is not None:
-        new_periods = []
-        for time_period in time_periods:
-            new_periods.append(time_period.astype(str).tolist())
-        time_periods = new_periods
+        time_periods = [time_period.astype(str).tolist() for time_period in time_periods]
 
     # Write the issue YAML file
     logger.info(f"Creating issue YAML file at {filepath}")
@@ -89,10 +88,9 @@ def write_issue(filepath: str, timesteps: list = None, time_periods: list = None
         if time_periods is not None:
             time_periods_dict = {"time_periods": time_periods}
             yaml.dump(time_periods_dict, f, default_flow_style=None)
-    return None
 
 
-def create_station_issue(data_source, campaign_name, station_name, base_dir=None):
+def create_station_issue(data_source, campaign_name, station_name, metadata_archive_dir=None):
     """Write an empty YAML issue YAML file for a DISDRODB station.
 
     An error is raised if the file already exists !
@@ -107,9 +105,9 @@ def create_station_issue(data_source, campaign_name, station_name, base_dir=None
         The name of the campaign. Must be provided in UPPER CASE.
     station_name : str
         The name of the station.
-    base_dir : str, optional
+    data_archive_dir : str, optional
         The base directory of DISDRODB, expected in the format ``<...>/DISDRODB``.
-        If not specified, the path specified in the DISDRODB active configuration will be used.
+        If not specified, the ``data_archive_dir`` path specified in the DISDRODB active configuration will be used.
 
     """
     # Define issue filepath
@@ -117,7 +115,7 @@ def create_station_issue(data_source, campaign_name, station_name, base_dir=None
         data_source=data_source,
         campaign_name=campaign_name,
         station_name=station_name,
-        base_dir=base_dir,
+        metadata_archive_dir=metadata_archive_dir,
         check_exists=False,
     )
     if os.path.exists(issue_filepath):
@@ -128,4 +126,3 @@ def create_station_issue(data_source, campaign_name, station_name, base_dir=None
     # Write issue file
     write_issue(filepath=issue_filepath)
     print(f"An empty issue YAML file for station {station_name} has been created .")
-    return None