fargopy 0.4.0__py3-none-any.whl → 1.0.1__py3-none-any.whl

fargopy/simulation.py

@@ -15,99 +15,169 @@ import time
 import gdown
 import os
 import signal
+import ipywidgets as widgets
+import matplotlib.pyplot as plt
+from IPython.display import display, clear_output
+from pathlib import Path
 
 ###############################################################
 # Constants
 ###############################################################
 KB = 1.380650424e-16  # Boltzmann constant: erg/K, erg = g cm^2 / s^2
-MP = 1.672623099e-24 # Mass of the proton, g
-GCONST = 6.67259e-8 # Gravitational constant, cm^3/g/s^2 
-RGAS = 8.314472e7 # Gas constant, erg/K/mol
-MSUN = 1.9891e33 # g
-AU = 1.49598e13 # cm
-YEAR = 31557600.0 # s
-
-PRECOMPUTED_BASEURL = 'https://docs.google.com/uc?export=download&id='
+MP = 1.672623099e-24  # Mass of the proton, g
+GCONST = 6.67259e-8  # Gravitational constant, cm^3/g/s^2
+RGAS = 8.314472e7  # Gas constant, erg/K/mol
+MSUN = 1.9891e33  # g
+AU = 1.49598e13  # cm
+YEAR = 31557600.0  # s
+
+PRECOMPUTED_BASEURL = "https://docs.google.com/uc?export=download&id="
 PRECOMPUTED_SIMULATIONS = dict(
     # Download link: https://drive.google.com/file/d/1YXLKlf9fCGHgLej2fSOHgStD05uFB2C3/view?usp=drive_link
     fargo=dict(
-        id='1YXLKlf9fCGHgLej2fSOHgStD05uFB2C3',
+        id="1YXLKlf9fCGHgLej2fSOHgStD05uFB2C3",
         description="""Protoplanetary disk with a Jovian planet [2D]""",
-        size=55
+        size=55,
     ),
     # Download link: https://drive.google.com/file/d/1KMp_82ylQn3ne_aNWEF1T9ElX2aWzYX6/view?usp=drive_link
     p3diso=dict(
-        id='1KMp_82ylQn3ne_aNWEF1T9ElX2aWzYX6',
+        id="1KMp_82ylQn3ne_aNWEF1T9ElX2aWzYX6",
         description="""Protoplanetary disk with a Super earth planet [3D]""",
-        size=220
+        size=220,
     ),
     # Download link: https://drive.google.com/file/d/1Xzgk9qatZPNX8mLmB58R9NIi_YQUrHz9/view?usp=sharing
     p3disoj=dict(
-        id='1Xzgk9qatZPNX8mLmB58R9NIi_YQUrHz9',
+        id="1Xzgk9qatZPNX8mLmB58R9NIi_YQUrHz9",
         description="""Protoplanetary disk with a Jovian planet [3D]""",
-        size=84
+        size=84,
     ),
     # Download link: https://drive.google.com/file/d/1KSQyxH_kbAqHQcsE30GQFRVgAPhMAcp7/view?usp=drive_link
     fargo_multifluid=dict(
-        id='1KSQyxH_kbAqHQcsE30GQFRVgAPhMAcp7',
+        id="1KSQyxH_kbAqHQcsE30GQFRVgAPhMAcp7",
         description="""Protoplanetary disk with several fluids (dust) and a Jovian planet in 2D""",
-        size=100
+        size=100,
     ),
     # Download link: https://drive.google.com/file/d/12ZWoQS_9ISe6eDij5KWWbqR-bHyyVs2N/view?usp=drive_link
     binary=dict(
-        id='12ZWoQS_9ISe6eDij5KWWbqR-bHyyVs2N',
+        id="12ZWoQS_9ISe6eDij5KWWbqR-bHyyVs2N",
         description="""Disk around a binary with the properties of Kepler-38 in 2D""",
-        size=140
+        size=140,
+    ),
+    # Download link: https://drive.google.com/file/d/1Xhunx2-eiW770p91OeTg1kvQdGwOOHhZ/view?usp=sharing
+    pds70iso=dict(
+        id="1Xhunx2-eiW770p91OeTg1kvQdGwOOHhZ",
+        description="""PDS70c - isothermal protoplanetary disk and circumplanetary disk in [3D]""",
+        size=140,
     ),
 )
 
 if not fargopy.IN_COLAB:
-    signal.signal(signal.SIGCHLD, signal.SIG_IGN)
+    # SIGCHLD is only available on Unix-like systems (Linux, macOS)
+    if hasattr(signal, "SIGCHLD"):
+        signal.signal(signal.SIGCHLD, signal.SIG_IGN)
 
 ###############################################################
 # Classes
 ###############################################################
+
+
 class Simulation(fargopy.Fargobj):
+    """Manage a FARGO3D simulation and its filesystem, execution and I/O.
 
-    def __init__(self,**kwargs):
-        """Initialize a simulation.
+    The ``Simulation`` object centralizes configuration, units, setup
+    discovery, compilation and runtime control for a FARGO3D case. It
+    stores derived paths (setup directory, output directory), unit
+    conversion factors and exposes convenience methods to compile,
+    launch and inspect simulation outputs.
 
-        Examples:
-            Create an empty simulation (no setup chosen):
-            >>> sim = fargopy.Simulation()
+    Attributes
+    ----------
+    fargo3d_dir : str
+        Base directory of FARGO3D installation.
+    outputs_dir : str
+        Directory where outputs are stored.
+    setups_dir : str
+        Directory where setups are located.
+    setup : str
+        Active setup name.
+    fargo3d_binary : str
+        Path to the compiled binary.
+    units_system : str
+        Unit system ('CGS' or 'MKS').
 
-            Create a simulation using a specific base FARGO3D directory (no setup chosen):
-            >>> sim = fargopy.Simulation(fargo3d_dir='/tmp/public')
+    Parameters
+    ----------
+    **kwargs : dict
+        Configuration keywords. Typical keys include ``setup``,
+        ``fargo3d_dir``, ``output_dir`` and ``load``. When ``load=True``
+        the object attempts to read serialized metadata from the
+        specified setup directory.
 
-            Create a simulation starting with setup directory: 
-            >>> sim = fargopy.Simulation(setup='fargo')
-            
-            Connect to an existing simulation:
-            >>> sim = fargopy.Simulation(output_dir='/tmp/public/outputs/fargo')
+    Examples
+    --------
+    Create a simulation object:
 
-            Load an already existing simulation:
-            >>> sim = fargopy.Simulation(setup='fargo',load=True)
-        """
+    >>> import fargopy as fp
+    >>> sim = fp.Simulation()
+
+    Select a setup:
+
+    >>> sim.set_setup('fargo')
+
+    Compile the simulation:
+
+    >>> sim.compile(parallel=0, gpu=0)
+
+    Run the simulation (clean run):
+
+    >>> sim.run(cleanrun=True)
+
+    Check status:
+
+    >>> sim.status()
+
+    Stop simulation:
+
+    >>> sim.stop()
+    """
 
+    def __init__(self, **kwargs):
+        """Create and configure the simulation object.
+
+        The constructor initializes unit systems, default paths and
+        optionally loads previously saved simulation metadata when the
+        ``load`` flag is provided in ``kwargs``.
+
+        Parameters
+        ----------
+        **kwargs : dict
+            Same as described in the class-level docstring. When
+            ``load=True`` the object expects a valid ``setup`` name and
+            a FARGO3D base directory to locate a
+            ``fargopy_simulation.json`` file to restore state.
+        """
         super().__init__(**kwargs)
-        
+
+        # Default to CGS units
+        self.units_system = "CGS"
+        self._set_constants_cgs()
+
         # Load simulation configuration from a file
-        if ('load' in kwargs.keys()) and kwargs['load']:
-            
-            if not 'setup' in kwargs.keys():
+        if ("load" in kwargs.keys()) and kwargs["load"]:
+            if not "setup" in kwargs.keys():
                 raise AssertionError(f"You must provide a setup name.")
             else:
-                setup = kwargs['setup']
-            if 'fargo3d_dir' in kwargs.keys():
-                fargo3d_dir = kwargs['fargo3d_dir']
+                setup = kwargs["setup"]
+            if "fargo3d_dir" in kwargs.keys():
+                fargo3d_dir = kwargs["fargo3d_dir"]
             else:
                 fargo3d_dir = fargopy.Conf.FP_FARGO3D_DIR
-            
+
             # Load simulation
-            load_from = f"{fargo3d_dir}/setups/{setup}".replace('//','/')
+            load_from = os.path.join(fargo3d_dir, "setups", setup)
             if not os.path.isdir(load_from):
                 print(f"Directory for loading simulation '{load_from}' not found.")
-            json_file = f"{load_from}/fargopy_simulation.json"
+            json_file = os.path.join(load_from, "fargopy_simulation.json")
 
             print(f"Loading simulation from '{json_file}'")
             if not os.path.isfile(json_file):
@@ -117,8 +187,8 @@ class Simulation(fargopy.Fargobj):
                 attributes = json.load(file_handler)
 
                 # Check if there are not serializable items and set to None
-                for key,item in attributes.items():
-                    if item == '<not serializable>':
+                for key, item in attributes.items():
+                    if item == "<not serializable>":
                         attributes[key] = None
 
                 # Update the object
@@ -129,50 +199,52 @@ class Simulation(fargopy.Fargobj):
                 self.set_setup(self.setup)
                 self.set_output_dir(self.output_dir)
             return
-        
+
         # Set units by default
-        self.set_units(UL=AU,UM=MSUN)
-        
+        self.set_units(UL=AU, UM=MSUN)
+
         # Set properties
-        self.set_property('fargo3d_dir',
-                          fargopy.Conf.FP_FARGO3D_DIR,
-                          self.set_fargo3d_dir)
-        self.set_property('setup',
-                          None,
-                          self.set_setup)
-        self.set_property('output_dir',
-                          None,
-                          self.set_output_dir)
-        self.set_property('fargo3d_compilation_options',
-                          dict(parallel=0,gpu=0,options=''))
+        self.set_property(
+            "fargo3d_dir", fargopy.Conf.FP_FARGO3D_DIR, self.set_fargo3d_dir
+        )
+        self.set_property("setup", None, self.set_setup)
+        self.set_property("output_dir", None, self.set_output_dir)
+        self.set_property(
+            "fargo3d_compilation_options", dict(parallel=0, gpu=0, options="")
+        )
 
         # Check if binary is already compiled
-        fargo3d_binary,compile_options = self._generate_binary_name(parallel=self.fargo3d_compilation_options['parallel'],
-                                                    gpu=self.fargo3d_compilation_options['gpu'],
-                                                    options=self.fargo3d_compilation_options['options'])
+        fargo3d_binary, compile_options = self._generate_binary_name(
+            parallel=self.fargo3d_compilation_options["parallel"],
+            gpu=self.fargo3d_compilation_options["gpu"],
+            options=self.fargo3d_compilation_options["options"],
+        )
         if os.path.isfile(f"{self.fargo3d_dir}/{fargo3d_binary}"):
             print(f"FARGO3D binary '{fargo3d_binary}' found.")
             self.fargo3d_binary = fargo3d_binary
         else:
             self.fargo3d_binary = None
 
-        # Simulation process does not exist   
-        self.set_property('fargo3d_process',
-                          None)
-        
+        # Simulation process does not exist
+        self.set_property("fargo3d_process", None)
+
     # ##########################################################################
     # Set special properties
-    # ##########################################################################  
-    def set_fargo3d_dir(self,dir=None):
-        """Set fargo3d directory
-
-        Args:
-            dir: string, default = None:
-                Directory where FARGO3D is installed.
-
-        Returns:
-            True if the FARGO3D directory exists and the file
-            'src/<fargo_header>' is found. False otherwise.
+    # #########################################a#################################
+    def set_fargo3d_dir(self, dir=None):
+        """Set the base FARGO3D installation directory.
+
+        Parameters
+        ----------
+        dir : str or None
+            Path to the FARGO3D installation. When ``None`` the method
+            is a no-op.
+
+        Returns
+        -------
+        None
+            The method does not return meaningful data; it updates
+            internal path attributes and prints diagnostics.
         """
         if dir is None:
             return
@@ -180,47 +252,53 @@ class Simulation(fargopy.Fargobj):
             print(f"FARGO3D directory '{dir}' does not exist.")
             return
         else:
-            fargo_header = f"{dir}/{fargopy.Conf.FP_FARGO3D_HEADER}".replace('//','/')
+            fargo_header = os.path.join(dir, fargopy.Conf.FP_FARGO3D_HEADER)
             if not os.path.isfile(fargo_header):
                 print(f"No header file for FARGO3D found in '{fargo_header}'")
             else:
                 print(f"Your simulation is now connected with '{dir}'")
-        
+
         # Set derivative dirs
         self.fargo3d_dir = dir
-        self.outputs_dir = (self.fargo3d_dir + '/outputs').replace('//','/')
-        self.setups_dir = (self.fargo3d_dir + '/setups').replace('//','/')
-    
-    def set_setup(self,setup):
-        """Connect the simulation to a given setup.
-
-        Args:
-            setup: string:
-                Name of the setup.
-        
-        Returns:
-            True if the setup_dir <faro3d_dir>/setups/<setup> is found. 
-            False otherwise.
+        self.outputs_dir = os.path.join(self.fargo3d_dir, "outputs")
+        self.setups_dir = os.path.join(self.fargo3d_dir, "setups")
+
+    def set_setup(self, setup):
+        """Associate the simulation with a named setup directory.
+
+        Parameters
+        ----------
+        setup : str or None
+            Setup name present under the configured ``setups_dir``. If
+            ``None`` the setup association is cleared.
+
+        Returns
+        -------
+        str or None
+            The assigned setup name on success, otherwise ``None``.
         """
         if setup is None:
             self.setup_dir = None
             return None
-        setup_dir = f"{self.setups_dir}/{setup}".replace('//','/')
+        setup_dir = os.path.join(self.setups_dir, setup)
         if self.set_setup_dir(setup_dir):
             self.setup = setup
-            self.logfile = f"{self.setup_dir}/{self.setup}.log"
+            self.logfile = os.path.join(self.setup_dir, f"{self.setup}.log")
         return setup
-    
-    def set_setup_dir(self,dir):
-        """Set setup directory
 
-        Args:
-            dir: string:
-                Directory where setup is available.
+    def set_setup_dir(self, dir):
+        """Set the absolute path to a setup directory and validate it.
+
+        Parameters
+        ----------
+        dir : str
+            Filesystem path to the setup directory.
 
-        Returns:
-            True if the FARGO3D directory exists and the file
-            <fargo3d_dir>/src/<fargo_header> is found. False otherwise.
+        Returns
+        -------
+        bool
+            ``True`` when the directory exists and is accepted,
+            otherwise ``False``.
         """
         if dir is None:
             return False
@@ -233,53 +311,132 @@ class Simulation(fargopy.Fargobj):
         self.setup_dir = dir
         return True
 
-    def set_output_dir(self,dir):
-        """Connect a simulation with a directory where the outputs are stored.
+    def set_output_dir(self, dir):
+        """Set the output directory where FARGO3D stores run results.
+
+        Parameters
+        ----------
+        dir : str or None
+            Output directory path. When present the method attempts to
+            load simulation properties from a ``variables.par`` file.
         """
         if dir is None:
             return
         if not os.path.isdir(dir):
             print(f"Output directory '{dir}' does not exist.")
             return
-        else:        
+        else:
             print(f"Now you are connected with output directory '{dir}'")
             self.output_dir = dir
-            # Check if there are results 
-            par_file = f"{dir}/variables.par".replace('//','/')
+            # Check if there are results
+            par_file = os.path.join(dir, "variables.par")
             if os.path.isfile(par_file):
                 print(f"Found a variables.par file in '{dir}', loading properties")
                 self.load_properties()
-        
+
         return
 
-    def set_units(self,UM=MSUN,UL=AU,G=1,mu=2.35):
-        """Set units of the simulation
+    ################################
+    ##  UNITS
+    ################################
+
+    def units(self, system):
+        """Switch the simulation unit system between ``'CGS'`` and
+        ``'MKS'``.
+
+        Parameters
+        ----------
+        system : {'CGS','MKS'}
+            Unit system identifier.
+        """
+        if system.upper() == "CGS":
+            self._set_constants_cgs()
+            self.units_system = "CGS"
+            print("Units set to CGS.")
+        elif system.upper() == "MKS":
+            self._set_constants_mks()
+            self.units_system = "MKS"
+            print("Units set to MKS.")
+        else:
+            raise ValueError("Invalid units system. Use 'CGS' or 'MKS'.")
+
+    def _set_constants_cgs(self):
+        """Configure physical constants for the CGS unit system."""
+        self.KB = 1.380650424e-16  # Boltzmann constant: erg/K
+        self.MP = 1.672623099e-24  # Mass of the proton, g
+        self.GCONST = 6.67259e-8  # Gravitational constant, cm^3/g/s^2
+        self.RGAS = 8.314472e7  # Gas constant, erg/K/mol
+        self.MSUN = 1.9891e33  # Solar mass, g
+        self.AU = 1.49598e13  # Astronomical unit, cm
+        self.YEAR = 31557600.0  # Year, s
+
+    def _set_constants_mks(self):
+        """Configure physical constants for the MKS unit system."""
+        self.KB = 1.380649e-23  # Boltzmann constant: J/K
+        self.MP = 1.6726219e-27  # Mass of the proton, kg
+        self.GCONST = 6.67430e-11  # Gravitational constant, m^3/kg/s^2
+        self.RGAS = 8.314462618  # Gas constant, J/K/mol
+        self.MSUN = 1.9891e30  # Solar mass, kg
+        self.AU = 1.49598e11  # Astronomical unit, m
+        self.YEAR = 31557600.0  # Year, s
+
+    def set_units(self, UM=MSUN, UL=AU, G=1, mu=2.35):
+        """Define simulation base units and derived unit scales.
+
+        Parameters
+        ----------
+        UM : float
+            Mass unit (default: MSUN).
+        UL : float
+            Length unit (default: AU).
+        G : float
+            Dimensionless gravitational constant (default: 1).
+        mu : float
+            Mean molecular weight used to compute temperature units.
         """
         # Basic
         self.UM = UM
         self.UL = UL
         self.G = G
-        self.UT = (G*self.UL**3/(GCONST*self.UM))**0.5 # In seconds
+        self.UT = (G * self.UL**3 / (GCONST * self.UM)) ** 0.5  # In seconds
 
         # Thermodynamics
-        self.UTEMP = (GCONST*MP*mu/KB)*self.UM/self.UL # In K
-        
+        self.UTEMP = (GCONST * MP * mu / KB) * self.UM / self.UL  # In K
+
         # Derivative
-        self.USIGMA = self.UM/self.UL**2 # In g/cm^2
-        self.URHO = self.UM/self.UL**3 # In kg/m^3
-        self.UEPS = self.UM/(self.UL*self.UT**2)  # In J/m^3
-        self.UV = self.UL/self.UT
-    
+        self.USIGMA = self.UM / self.UL**2  # In g/cm^2
+        self.URHO = self.UM / self.UL**3  # In kg/m^3
+        self.UEPS = self.UM / (self.UL * self.UT**2)  # In J/m^3
+        self.UV = self.UL / self.UT
+
     # ##########################################################################
     # Control methods
-    # ##########################################################################  
-    def compile(self,setup=None,parallel=0,gpu=0,options='',force=False):
-        """Compile FARGO3D binary
+    # ##########################################################################
+    def compile(self, setup=None, parallel=0, gpu=0, options="", force=False):
+        """Compile the FARGO3D executable for the active setup.
+
+        Parameters
+        ----------
+        setup : str, optional
+            Setup name to compile. When provided the method will try to
+            set the setup before compiling.
+        parallel : int, optional
+            Parallel compilation flag (default: 0).
+        gpu : int, optional
+            GPU-enabled compilation flag (default: 0).
+        options : str, optional
+            Additional make options passed to the build system.
+        force : bool, optional
+            If ``True`` perform a clean before building.
+
+        Examples
+        --------
+        >>> sim.compile(parallel=1, gpu=0)
         """
         if setup is not None:
             if not self.set_setup(setup):
                 print("Failed")
-                return 
+                return
 
         # Clean directrory
         if force:
@@ -287,87 +444,141 @@ class Simulation(fargopy.Fargobj):
             cmd = f"make -C {self.fargo3d_dir} clean mrproper"
             # Clean all binaries
             compl = f"rm -rf {self.fargo3d_dir}/fargo3d_*"
-            error,output_clean = fargopy.Sys.run(cmd + '&&' + compl)
-            
+            error, output_clean = fargopy.Sys.run(cmd + "&&" + compl)
+
         # Prepare compilation
-        fargo3d_binary,compile_options = self._generate_binary_name(parallel=parallel,gpu=gpu,options=options)
-        #compile_options = f"SETUP={self.setup} PARALLEL={parallel} GPU={gpu} "+options
-        #fargo3d_binary = f"fargo3d-{compile_options.replace(' ','-').replace('=','_').strip('-')}"
+        fargo3d_binary, compile_options = self._generate_binary_name(
+            parallel=parallel, gpu=gpu, options=options
+        )
+        # compile_options = f"SETUP={self.setup} PARALLEL={parallel} GPU={gpu} "+options
+        # fargo3d_binary = f"fargo3d-{compile_options.replace(' ','-').replace('=','_').strip('-')}"
 
         # Compile binary
         print(f"Compiling {fargo3d_binary}...")
         cmd = f"cd {self.fargo3d_dir};make {compile_options} 2>&1 |tee {self.setup_dir}/compilation.log"
         compl = f"mv fargo3d {fargo3d_binary}"
         pipe = f""
-        error,output_compilation = fargopy.Sys.run(cmd+' && '+compl)
+        error, output_compilation = fargopy.Sys.run(cmd + " && " + compl)
 
         # Check compilation result
         if os.path.isfile(f"{self.fargo3d_dir}/{fargo3d_binary}"):
             self.fargo3d_binary = fargo3d_binary
             print(f"Succesful compilation of FARGO3D binary {self.fargo3d_binary}")
-            self.fargo3d_compilation_options=dict(
-                parallel=parallel,
-                gpu=gpu,
-                options=options
+            self.fargo3d_compilation_options = dict(
+                parallel=parallel, gpu=gpu, options=options
             )
         else:
-            print(f"Something failed when compiling FARGO3D. For details check '{self.setup_dir}/compilation.log")
-            
-    def _generate_binary_name(self,parallel=0,gpu=0,options=''):
-        """Generate binary name
+            print(
+                f"Something failed when compiling FARGO3D. For details check '{self.setup_dir}/compilation.log"
+            )
+
+    def _generate_binary_name(self, parallel=0, gpu=0, options=""):
+        """Produce the target binary filename and the make options.
+
+        Parameters
+        ----------
+        parallel : int
+            Parallel compilation flag.
+        gpu : int
+            GPU compilation flag.
+        options : str
+            Extra options appended to the make command.
+
+        Returns
+        -------
+        tuple
+            ``(binary_name, compile_options)`` where ``binary_name`` is
+            the filename to expect after a successful build and
+            ``compile_options`` is the command fragment passed to
+            ``make``.
         """
-        compile_options = f"SETUP={self.setup} PARALLEL={parallel} GPU={gpu} "+options
-        fargo3d_binary = f"fargo3d-{compile_options.replace(' ','-').replace('=','_').strip('-')}"
+        compile_options = f"SETUP={self.setup} PARALLEL={parallel} GPU={gpu} " + options
+        fargo3d_binary = (
+            f"fargo3d-{compile_options.replace(' ', '-').replace('=', '_').strip('-')}"
+        )
         return fargo3d_binary, compile_options
 
-    def run(self,
-            mode='async',
-            options='-m',
-            mpioptions='-np 1',
-            resume=False,
-            cleanrun=False,
-            test=False,
-            unlock=True):
+    def run(
+        self,
+        mode="async",
+        options="-m",
+        mpioptions="-np 1",
+        resume=False,
+        cleanrun=False,
+        test=False,
+        unlock=True,
+    ):
+        """Run the FARGO3D simulation.
+
+        Parameters
+        ----------
+        mode : str, optional
+            'async' for asynchronous or 'sync' for synchronous execution.
+        options : str, optional
+            Command-line options for FARGO3D.
+        mpioptions : str, optional
+            MPI options for parallel runs.
+        resume : bool, optional
+            Resume from previous run.
+        cleanrun : bool, optional
+            Clean output directory before running.
+        test : bool, optional
+            If True, do not actually run the simulation.
+        unlock : bool, optional
+            If True, unlock the simulation if locked.
+
+        Examples
+        --------
+        Run asynchronously:
+
+        >>> sim.run(cleanrun=True)
+
+        Run synchronously:
+
+        >>> sim.run(mode='sync', cleanrun=True)
+        """
 
         if self.fargo3d_binary is None:
-            print("You must first compile your simulation with: <simulation>.compile(<option>).")
+            print(
+                "You must first compile your simulation with: <simulation>.compile(<option>)."
+            )
             return
 
         if self._is_running():
             print(f"There is a running process. Please stop it before running/resuming")
             return
 
-        lock_info = fargopy.Sys.is_locked(self.setup_dir)        
+        lock_info = fargopy.Sys.is_locked(self.setup_dir)
         if lock_info:
             print(f"The process is locked by PID {lock_info['pid']}")
             return
 
         # Mandatory options
         options = options + " -t"
-        if 'fargo3d_run_options' not in self.__dict__.keys():
+        if "fargo3d_run_options" not in self.__dict__.keys():
             self.fargo3d_run_options = options
-        
+
         # Clean output if available
         if cleanrun:
             # Check if there is an output director
-            output_dir = f"{self.outputs_dir}/{self.setup}"
+            output_dir = os.path.join(self.outputs_dir, self.setup)
             if os.path.isdir(output_dir):
                 self.output_dir = output_dir
                 self.clean_output()
             else:
                 print(f"No output directory {output_dir} yet created.")
-        
+
         # Select command to run
-        precmd=''
-        if self.fargo3d_compilation_options['parallel']:
+        precmd = ""
+        if self.fargo3d_compilation_options["parallel"]:
             precmd = f"mpirun {mpioptions} "
 
         # Preparing command
         run_cmd = f"{precmd} ./{self.fargo3d_binary} {options} setups/{self.setup}/{self.setup}.par"
 
-        self.json_file = f"{self.setup_dir}/fargopy_simulation.json"
-        if mode == 'sync':
-            # Save object 
+        self.json_file = os.path.join(self.setup_dir, "fargopy_simulation.json")
+        if mode == "sync":
+            # Save object
             self.save_object(self.json_file)
 
             # Run synchronously
@@ -376,57 +587,71 @@ class Simulation(fargopy.Fargobj):
             fargopy.Sys.simple(cmd)
             self.fargo3d_process = None
 
-        elif mode == 'async':
+        elif mode == "async":
             # Run asynchronously
-            
+
             # Select logfile mode accroding to if the process is resuming
-            logmode = 'a' if resume else 'w'
-            logfile_handler=open(self.logfile,logmode)
+            logmode = "a" if resume else "w"
+            logfile_handler = open(self.logfile, logmode)
 
             # Launch process
             print(f"Running asynchronously (test = {test}): {run_cmd}")
             if not test:
-
                 # Check it is not locked
                 lock_info = fargopy.Sys.is_locked(dir=self.setup_dir)
                 if lock_info:
                     if unlock:
                         self.unlock_simulation(lock_info)
                     else:
-                        print(f"Output directory {self.setup_dir} is locked by a running process")
+                        print(
+                            f"Output directory {self.setup_dir} is locked by a running process"
+                        )
                         return
 
-                process = subprocess.Popen(run_cmd.split(),cwd=self.fargo3d_dir,
-                                           stdout=logfile_handler,stderr=logfile_handler,close_fds=True)
-                
+                process = subprocess.Popen(
+                    run_cmd.split(),
+                    cwd=self.fargo3d_dir,
+                    stdout=logfile_handler,
+                    stderr=logfile_handler,
+                    close_fds=True,
+                )
+
                 # Introduce a short delay to verify if the process has failed
                 time.sleep(1.0)
 
                 if process.poll() is None:
                     # Check if program is effectively running
-                    self.fargo3d_process = process            
-                    
+                    self.fargo3d_process = process
+
                     # Lock
                     fargopy.Sys.lock(
-                        dir=self.setup_dir,
-                        content=dict(pid=self.fargo3d_process.pid)
+                        dir=self.setup_dir, content=dict(pid=self.fargo3d_process.pid)
                     )
 
-                    # Setup output directory 
-                    self.set_output_dir(f"{self.outputs_dir}/{self.setup}".replace('//','/'))    
+                    # Setup output directory
+                    self.set_output_dir(os.path.join(self.outputs_dir, self.setup))
 
-                    # Save object 
+                    # Save object
                     self.save_object(self.json_file)
                 else:
-                    print(f"Process running failed. Please check the logfile {self.logfile}")
+                    print(
+                        f"Process running failed. Please check the logfile {self.logfile}"
+                    )
         else:
             print(f"Mode {mode} not recognized (valid are 'sync', 'async')")
             return
-                    
-    def stop(self):
+
+    def stop(self, verbose=False):
+        """Stop a running FARGO3D process and release the lock on the setup.
+
+        If a process is associated with the simulation the method attempts
+        to terminate it and then unlock the setup directory. If no
+        process handler is available the method simply tries to remove
+        any filesystem lock.
+        """
         # Check if the directory is locked
         lock_info = fargopy.Sys.is_locked(self.setup_dir)
-        
+
         if lock_info:
             print(f"The process is locked by PID {lock_info['pid']}")
 
@@ -446,42 +671,52 @@ class Simulation(fargopy.Fargobj):
             self.unlock_simulation(lock_info)
             print(f"The process has finished. Check logfile {self.logfile}.")
 
-    def unlock_simulation(self,lock_info=None,force=True):
-        """Unlock a simulation
+    def unlock_simulation(self, lock_info=None, force=True, verbose=False):
+        """Remove a simulation lock and optionally kill the owning PID.
+
+        Parameters
+        ----------
+        lock_info : dict or None
+            Lock metadata as returned by ``fargopy.Sys.is_locked``. If
+            ``None`` the method will attempt to discover the lock for
+            the active setup directory.
+        force : bool, optional
+            When ``True`` attempt to forcibly terminate the process
+            owning the lock (``kill -9``) before releasing the lock.
         """
         if lock_info is None and force:
             lock_info = fargopy.Sys.is_locked(self.setup_dir)
             if lock_info:
                 print(f"Unlocking simulation (pid = {lock_info['pid']})")
-                
+
         if lock_info:
-            pid = lock_info['pid']
+            pid = lock_info["pid"]
             fargopy.Debug.trace(f"Unlocking simulation (pid = {pid})")
-            error,output = fargopy.Sys.run(f"kill -9 {pid}")    
+            error, output = fargopy.Sys.run(f"kill -9 {pid}")
             fargopy.Sys.unlock(self.setup_dir)
-        
-    def status(self,mode='isrunning',verbose=True,**kwargs):
-        """Check the status of the running process
-
-        Parameters:
-            mode: string, defaul='isrunning':
-                Available modes:
-                    'isrunning': Just show if the process is running.
-                    'logfile': Show the latest lines of the logfile
-                    'outputs': Show (and return) a list of outputs
-                    'snapshots': Show (and return) a list of snapshots
-                    'progress': Show progress in realtime
-                    'locking': Show if the directory is locked
 
+    def status(self, mode="isrunning", verbose=True, **kwargs):
+        """Report process, logfile and output status for the simulation.
+
+        Parameters
+        ----------
+        mode : {'isrunning','logfile','outputs','progress','summary','locking','all'}, optional
+            Which status block to display. ``'all'`` prints every
+            section.
+        verbose : bool, optional
+            When ``True`` print human-readable diagnostics to stdout.
+        **kwargs : dict
+            Backend-specific options used by certain modes (for
+            instance ``numstatus`` for the ``'progress'`` mode).
         """
-        # Bar separating output 
-        bar = f"\n{''.join(['#']*80)}\n"
-        
+        # Bar separating output
+        bar = f"\n{''.join(['#'] * 80)}\n"
+
         # vprint
-        vprint = print if verbose else lambda x:x
+        vprint = print if verbose else lambda x: x
 
-        if 'isrunning' in mode or mode=='all':
-            vprint(bar+"Running status of the process:")
+        if "isrunning" in mode or mode == "all":
+            vprint(bar + "Running status of the process:")
             if self.fargo3d_process:
                 poll = self.fargo3d_process.poll()
                 if poll is None:
@@ -490,29 +725,29 @@ class Simulation(fargopy.Fargobj):
                     # Unlock any remaining process
                     lock_info = fargopy.Sys.is_locked(self.setup_dir)
                     self.unlock_simulation(lock_info)
-                    
+
                     vprint(f"\tThe process has ended with termination code {poll}.")
             else:
                 vprint(f"\tThe process is stopped.")
 
-        if 'logfile' in mode or mode=='all':
-            vprint(bar+"Logfile content:")
-            if 'logfile' in self.__dict__.keys() and os.path.isfile(self.logfile):
+        if "logfile" in mode or mode == "all":
+            vprint(bar + "Logfile content:")
+            if "logfile" in self.__dict__.keys() and os.path.isfile(self.logfile):
                 vprint("The latest 10 lines of the logfile:\n")
                 if verbose:
                     os.system(f"tail -n 10 {self.logfile}")
             else:
                 vprint("No log file created yet")
 
-        if 'outputs' in mode or mode=='all':
-            vprint(bar+"Output content:")
-            error,output = fargopy.Sys.run(f"ls {self.output_dir}/*.dat")
+        if "outputs" in mode or mode == "all":
+            vprint(bar + "Output content:")
+            error, output = fargopy.Sys.run(f"ls {self.output_dir}/*.dat")
             if not error:
-                files = [file.split('/')[-1] for file in output[:-1]]
+                files = [file.split("/")[-1] for file in output[:-1]]
                 file_list = ""
-                for i,file in enumerate(files):
+                for i, file in enumerate(files):
                     file_list += f"{file}, "
-                    if ((i+1)%10) == 0:
+                    if ((i + 1) % 10) == 0:
                         file_list += "\n"
                 file_list = file_list.strip("\n,")
                 vprint(f"\n{len(files)} available datafiles:\n")
@@ -521,107 +756,125 @@ class Simulation(fargopy.Fargobj):
             else:
                 vprint("No datafiles yet available")
 
-        if 'summary' in mode or mode=='all':
-            vprint(bar+"Summary:")
+        if "summary" in mode or mode == "all":
+            vprint(bar + "Summary:")
             nsnaps = self._get_nsnaps()
-            print(f"The simulation has been ran for {nsnaps} time-steps (including the initial one).")
+            print(
+                f"The simulation has been ran for {nsnaps} time-steps (including the initial one)."
+            )
 
-        if 'locking' in mode or mode=='all':
-            vprint(bar+"Locking status:")
+        if "locking" in mode or mode == "all":
+            vprint(bar + "Locking status:")
             lock_info = fargopy.Sys.is_locked(self.setup_dir)
             print(lock_info)
 
-        if 'progress' in mode:
+        if "progress" in mode:
             vprint(bar)
             numstatus = 5
-            if 'numstatus' in kwargs.keys():
-                numstatus = int(kwargs['numstatus'])
+            if "numstatus" in kwargs.keys():
+                numstatus = int(kwargs["numstatus"])
             self._status_progress(numstatus=numstatus)
 
-        print(f"\nOther status modes: 'isrunning', 'logfile', 'outputs', 'progress', 'summary'")
+        print(
+            f"\nOther status modes: 'isrunning', 'logfile', 'outputs', 'progress', 'summary'"
+        )
 
-    def _status_progress(self,minfreq=0.1,numstatus=100):
-        """Show a progress of the execution
+    def _status_progress(self, minfreq=0.1, numstatus=100):
+        """Display a live progress summary by tailing the simulation log.
 
-        Parameters:
-            minfreq: float, default = 0.1:
-                Minimum amount of seconds between status check.
+        The routine periodically reads the simulation logfile and prints
+        snapshot progress information until the process stops or the
+        requested number of updates is reached.
 
-            numstatus: int, default = 5:
-                Number of status shown before automatically stopping.
+        Parameters
+        ----------
+        minfreq : float, optional
+            Minimum polling interval in seconds.
+        numstatus : int, optional
+            Maximum number of status frames to emit.
         """
         # Prepare
-        if 'status_frequency' not in self.__dict__.keys():
+        if "status_frequency" not in self.__dict__.keys():
             frequency = minfreq
         else:
             frequency = self.status_frequency
-        previous_output = ''
+        previous_output = ""
         previous_resumable_snapshot = 1e100
         time_previous = time.time()
 
         # Infinite loop checking for output
         n = 0
-        print(f"Progress of the simulation (interrupt by pressing 'enter' or the stop button):")
-        while True and (n<numstatus):
-            
+        print(f"Progress of the simulation (interrupt by pressing the stop button):")
+        while True and (n < numstatus):
             # Check if the process is running locally
             if not self._is_running():
                 print("The simulation is not running anymore")
                 return
-            
-            error,output = fargopy.Sys.run(f"grep OUTPUT {self.logfile} |tail -n 1")
-            
+
+            error, output = fargopy.Sys.run(f"grep OUTPUT {self.logfile} |tail -n 1")
+
             if not error:
                 # Get the latest output
                 latest_output = output[-2]
                 if latest_output != previous_output:
-                    print(f"{n+1}:{latest_output} [output pace = {frequency:.1f} secs] <Press 'enter' to interrupt>")
-                    n += 1 
+                    print(
+                        f"{n + 1}:{latest_output} [output pace = {frequency:.1f} secs]"
+                    )
+                    n += 1
                     # Fun the number of the output
-                    find = re.findall(r'OUTPUTS\s+(\d+)',latest_output)
+                    find = re.findall(r"OUTPUTS\s+(\d+)", latest_output)
                     resumable_snapshot = int(find[0])
                     # Get the time elapsed since last status check
                     time_now = time.time()
-                    frequency = max(time_now - time_previous,minfreq)/2
-                    if (resumable_snapshot - previous_resumable_snapshot)>1:
+                    frequency = max(time_now - time_previous, minfreq) / 2
+                    if (resumable_snapshot - previous_resumable_snapshot) > 1:
                         # Reduce frequency if snapshots are accelerating
-                        frequency = frequency/2
+                        frequency = frequency / 2
                         self.status_frequency = frequency
                     previous_resumable_snapshot = resumable_snapshot
                     time_previous = time_now
                 previous_output = latest_output
-            
-            if fargopy.IN_COLAB:
-                # In colab the control is much more limited
-                try:
-                    time.sleep(frequency)
-                except KeyboardInterrupt:
-                    print("Interrupted by user. In some environment (IPython, Colab) stopping the progress status will stop the simulation. In that case just resume.")
-                    return
-                
-            else:
-                # Suitable for running in Jupyter and IPython
-                if fargopy.Sys.sleep_timeout(frequency):
-                    return
-                
-    def resume(self,snapshot=-1,mpioptions='-np 1'):
+
+            try:
+                time.sleep(frequency)
+            except KeyboardInterrupt:
+                print("Interrupted by user.")
+                return
+
+    def resume(self, snapshot=-1, mpioptions="-np 1"):
+        """Resume an interrupted simulation from the specified snapshot.
+
+        Parameters
+        ----------
+        snapshot : int, optional
+            Snapshot index to resume from. Use ``-1`` to resume from
+            the most recent resumable snapshot.
+        mpioptions : str, optional
+            MPI launch options for parallel executions.
+        """
         latest_snapshot_resumable = self._is_resumable()
-        if latest_snapshot_resumable<0:
+        if latest_snapshot_resumable < 0:
             return
         if self._is_running():
             print(f"There is a running process. Please stop it before resuming")
             return
-        if 'fargo3d_run_options' not in self.__dict__.keys():
+        if "fargo3d_run_options" not in self.__dict__.keys():
             print(f"The process has not been run before.")
             return
         # Resume
-        if snapshot<0:
+        if snapshot < 0:
             snapshot = latest_snapshot_resumable
         print(f"Resuming from snapshot {snapshot}...")
-        self.run(mode='async',mpioptions=mpioptions,resume=True,
-                 options=self.fargo3d_run_options+f' -S {snapshot}',test=False)
+        self.run(
+            mode="async",
+            mpioptions=mpioptions,
+            resume=True,
+            options=self.fargo3d_run_options + f" -S {snapshot}",
+            test=False,
+        )
 
     def _has_finished(self):
+        """Return ``True`` if the associated FARGO3D process has exited."""
         if self.fargo3d_process:
             poll = self.fargo3d_process.poll()
             if poll is None:
@@ -631,13 +884,24 @@ class Simulation(fargopy.Fargobj):
                 return True
 
     def _is_resumable(self):
+        """Determine the latest snapshot index from which the simulation can resume.
+
+        Returns
+        -------
+        int
+            Index of the latest resumable snapshot, or ``-1`` when the
+            simulation is not resumable or no outputs are present.
+        """
         if self.logfile is None:
-            print(f"The simulation has not been ran yet. Run <simulation>.run() before resuming")
+            print(
+                f"The simulation has not been ran yet. Run <simulation>.run() before resuming"
+            )
             return -1
         latest_snapshot_resumable = max(self._get_nsnaps() - 2, 0)
         return latest_snapshot_resumable
-        
+
     def clean_output(self):
+        """Remove all files from the configured output directory."""
         if self.output_dir is None:
             print(f"Output directory has not been set.")
             return
@@ -645,20 +909,33 @@ class Simulation(fargopy.Fargobj):
         if self._is_running():
             print(f"There is a running process. Please stop it before cleaning")
             return
-        
+
         print(f"Cleaning output directory {self.output_dir}")
         cmd = f"rm -rf {self.output_dir}/*"
-        error,output = fargopy.Sys.run(cmd)
+        error, output = fargopy.Sys.run(cmd)
+
+    def _is_running(self, verbose=False):
+        """Return ``True`` when a live FARGO3D process is detected.
 
-    def _is_running(self,verbose=False):
+        Parameters
+        ----------
+        verbose : bool, optional
+            When ``True`` print diagnostic messages.
+
+        Returns
+        -------
+        bool
+            ``True`` if a running process is associated with the
+            simulation, otherwise ``False``.
+        """
         lock_info = fargopy.Sys.is_locked(self.setup_dir)
         if lock_info:
             # Check if process is up
-            error,output = fargopy.Sys.run(f"ps -p {lock_info['pid']}")
+            error, output = fargopy.Sys.run(f"ps -p {lock_info['pid']}")
             if error == 0:
                 return True
 
-        if not self.has('fargo3d_process'):
+        if not self.has("fargo3d_process"):
             if verbose:
                 print("The simulation has not been run before.")
             return False
@@ -667,7 +944,9 @@ class Simulation(fargopy.Fargobj):
             poll = self.fargo3d_process.poll()
             if poll is None:
                 if verbose:
-                    print(f"The process is already running with pid '{self.fargo3d_process.pid}'")
+                    print(
+                        f"The process is already running with pid '{self.fargo3d_process.pid}'"
+                    )
                 return True
             else:
                 return False
@@ -675,6 +954,7 @@ class Simulation(fargopy.Fargobj):
             return False
 
     def _check_process(self):
+        """Return ``True`` if a process handler is present for the run."""
         if self.fargo3d_process is None:
             print(f"There is no FARGO3D process handler available.")
             return False
@@ -682,53 +962,84 @@ class Simulation(fargopy.Fargobj):
 
     # ##########################################################################
     # Operations on the FARGO3D directories
-    # ##########################################################################  
-    def list_fields(self,quiet=False):
+    # ##########################################################################
+    def list_fields(self, quiet=False):
+        """Return a list of data file names present in the output directory.
+
+        Parameters
+        ----------
+        quiet : bool, optional
+            When ``True`` avoid printing the detailed file list.
+
+        Returns
+        -------
+        list
+            Filenames found in the output directory.
+        """
         if self.output_dir is None:
-            print(f"You have to set forst the outputs directory with <sim>.set_outputs('<directory>')")
+            print(
+                f"You have to set forst the outputs directory with <sim>.set_outputs('<directory>')"
+            )
         else:
-            error,output = fargopy.Sys.run(f"ls {self.output_dir}")
+            error, output = fargopy.Sys.run(f"ls {self.output_dir}")
         if error == 0:
             files = output[:-1]
             print(f"{len(files)} files in output directory")
             if not quiet:
                 file_list = ""
-                for i,file in enumerate(files):
+                for i, file in enumerate(files):
                     file_list += f"{file}, "
-                    if ((i+1)%10) == 0:
+                    if ((i + 1) % 10) == 0:
                         file_list += "\n"
                 print(file_list)
             return files
-            
 
-    def load_macros(self, summaryfile='summary0.dat'):
-        """
-        Extracts global simulation parameters from the PREPROCESSOR MACROS SECTION in summary0.dat.
-        Returns a dictionary with the parameters found in the macros section.
-        Only the value after the last '=' is stored for each parameter.
+    def load_macros(self, summaryfile="summary0.dat"):
+        """Parse the PREPROCESSOR MACROS SECTION from a summary file.
+
+        Parameters
+        ----------
+        summaryfile : str, optional
+            Summary filename relative to the output directory (default
+            ``'summary0.dat'``).
+
+        Returns
+        -------
+        dict
+            Mapping of macro names to parsed values. Returns an empty
+            dict when the file is missing or parsing fails.
         """
-        summary_path = f"{self.output_dir}/{summaryfile}".replace('//', '/')
+        summary_path = os.path.join(self.output_dir, summaryfile)
         if not os.path.isfile(summary_path):
             print(f"No summary file '{summary_path}' found.")
             return {}
 
         macros = {}
         in_macros_section = False
-        with open(summary_path, 'r') as f:
+        with open(summary_path, "r") as f:
             for line in f:
                 # Detect the start of the macros section
-                if 'PREPROCESSOR MACROS SECTION' in line:
+                if "PREPROCESSOR MACROS SECTION" in line:
                     in_macros_section = True
                     continue
                 # Stop if another section starts
-                if in_macros_section and 'SECTION' in line and 'PREPROCESSOR MACROS SECTION' not in line:
+                if (
+                    in_macros_section
+                    and "SECTION" in line
+                    and "PREPROCESSOR MACROS SECTION" not in line
+                ):
                     break
                 if in_macros_section:
                     # Skip empty lines and lines that don't contain '='
-                    if line.strip() == '' or '=' not in line or line.strip().startswith('=') or line.strip().startswith('#'):
+                    if (
+                        line.strip() == ""
+                        or "=" not in line
+                        or line.strip().startswith("=")
+                        or line.strip().startswith("#")
+                    ):
                         continue
                     # Extract parameter name and value after the last '='
-                    parts = line.split('=')
+                    parts = line.split("=")
                     if len(parts) >= 2:
                         key = parts[0].strip()
                         value_str = parts[-1].strip()
@@ -741,99 +1052,150 @@ class Simulation(fargopy.Fargobj):
         self.simulation_macros = macros
         return macros
 
-                
-    def load_planet_summary(self, summaryfile='summary0.dat'):
+    def load_planets(self, snapshot=0, summaryfile=None):
         """
-        Loads planet information from summary.dat.
-        If not found, tries to extract from variables.par (PLANETMASS, ORBITALRADIUS, PLANETCONFIG).
-        Returns a list of dictionaries with name, distance, and mass.
+        Load planet data from a summary file.
+
+        Parameters
+        ----------
+        snapshot : int, optional
+            Snapshot index to load.
+        summaryfile : str, optional
+            Name of the summary file.
+
+        Returns
+        -------
+        list of Planet
+            List of Planet objects for the given snapshot.
         """
-        summary_path = f"{self.output_dir}/{summaryfile}".replace('//', '/')
-        if os.path.isfile(summary_path):
-            planets = []
-            with open(summary_path, 'r') as f:
-                lines = f.readlines()
-                for line in lines:
-                    # Skip comments, empty lines, and separators
-                    if line.strip().startswith('#') or not line.strip() or set(line.strip()) == set('-'):
-                        continue
-                    parts = line.split()
-                    if len(parts) >= 3:
-                        # Only accept if the first field is NOT a number
-                        try:
-                            float(parts[0])
-                            continue  # If it's a number, skip the line
-                        except ValueError:
-                            pass  # If not a number, it's a planet name
-                        try:
-                            planet = {
-                                'name': parts[0],
-                                'distance': float(parts[1]),
-                                'mass': float(parts[2])
-                            }
-                            planets.append(planet)
-                        except ValueError:
-                            continue
-            return planets
+        import os
 
-        # If summary.dat does not exist, try to read from variables.par
-        varfile = f"{self.output_dir}/variables.par".replace('//', '/')
-        if not os.path.isfile(varfile):
-            print(f"No summary file '{summary_path}' or variables file '{varfile}' found.")
-            return []
-
-        planet_mass = None
-        orbital_radius = None
-        planet_config = None
-        with open(varfile, 'r') as f:
-            for line in f:
-                # Look for PLANETMASS, ORBITALRADIUS, PLANETCONFIG lines
-                if line.strip().startswith('PLANETMASS'):
-                    try:
-                        planet_mass = float(line.split()[1])
-                    except Exception:
-                        pass
-                elif line.strip().startswith('ORBITALRADIUS'):
-                    try:
-                        orbital_radius = float(line.split()[1])
-                    except Exception:
-                        pass
-                elif line.strip().startswith('PLANETCONFIG'):
-                    try:
-                        planet_config = line.split()[1]
-                    except Exception:
-                        pass
-
-        if planet_mass is not None and orbital_radius is not None:
-            # Use the config file name as planet name if available
-            planet_name = planet_config.split('/')[-1].split('.')[0] if planet_config else 'planet'
-            return [{
-                'name': planet_name,
-                'distance': orbital_radius,
-                'mass': planet_mass
-            }]
-        else:
-            print("No planet data found in summary.dat or variables.par.")
+        # Determine summary file
+        if summaryfile is None:
+            summaryfile = f"summary{snapshot}.dat"
+        summary_path = os.path.join(self.output_dir, summaryfile)
+        if not os.path.isfile(summary_path):
+            print(f"No summary file '{summary_path}' found.")
             return []
 
+        # Get stellar mass
+        if (
+            not hasattr(self, "simulation_macros")
+            or "MSTAR" not in self.simulation_macros
+        ):
+            self.load_macros()
+        mstar = self.simulation_macros["MSTAR"]
+
+        # Parse initial positions from the top of the file (if available)
+        initial_planets = []
+        with open(summary_path, "r") as f:
+            lines = f.readlines()
+        for line in lines:
+            if (
+                line.strip().startswith("#")
+                or not line.strip()
+                or set(line.strip()) == set("-")
+            ):
+                continue
+            parts = line.split()
+            if len(parts) >= 3:
+                try:
+                    float(parts[0])
+                    continue  # skip if first field is a number
+                except ValueError:
+                    pass
+                try:
+                    name = parts[0]
+                    x0 = float(parts[1])
+                    y0 = float(parts[2]) if len(parts) > 3 else 0.0
+                    z0 = 0.0
+                    initial_planets.append({"name": name, "posi": (x0, y0, z0)})
+                except Exception:
+                    continue
 
-    def load_properties(self, quiet=False,
-                            varfile='variables.par',
-                            domain_prefix='domain_',
-                            dimsfile='dims.dat',
-                            summaryfile='summary0.dat'  # Nuevo parámetro opcional
-                            ):
+        # Find PLANETARY SYSTEM SECTION
+        planet_indices = []
+        in_section = False
+        for i, line in enumerate(lines):
+            if "PLANETARY SYSTEM SECTION" in line:
+                in_section = True
+                continue
+            if in_section and line.strip().startswith("#### Planet"):
+                planet_indices.append(i + 1)  # next line has the data
+            if in_section and line.strip().startswith("***"):
+                break
+
+        planets = []
+        for idx, data_idx in enumerate(planet_indices):
+            data_line = lines[data_idx].strip()
+            parts = data_line.split()
+            if len(parts) < 7:
+                continue
+            x, y, z = map(float, parts[0:3])
+            vx, vy, vz = map(float, parts[3:6])
+            mass = float(parts[-1])  # Always take the last column as mass
+
+            # Get initial position if available
+            if idx < len(initial_planets):
+                name = initial_planets[idx]["name"]
+                posi = initial_planets[idx]["posi"]
+            else:
+                name = f"Planet {idx}"
+                posi = (x, y, z)
+            planet = Planet(
+                name=name,
+                pos=(x, y, z),
+                vel=(vx, vy, vz),
+                mass=mass,
+                posi=posi,
+                mstar=mstar,
+            )
+            planets.append(planet)
+        return planets
+
+    def load_properties(
+        self,
+        quiet=False,
+        varfile="variables.par",
+        domain_prefix="domain_",
+        dimsfile="dims.dat",
+        summaryfile="summary0.dat",
+    ):
+        """
+        Load and print simulation properties, including variables, domains, dimensions, and planets.
+
+        Parameters
+        ----------
+        quiet : bool, optional
+            If True, suppress output.
+        varfile : str, optional
+            Name of the variables file.
+        domain_prefix : str, optional
+            Prefix for domain files.
+        dimsfile : str, optional
+            Name of the dimensions file.
+        summaryfile : str, optional
+            Name of the summary file.
+
+        Returns
+        -------
+        str
+            Summary information string.
+        """
+        info = []
         if self.output_dir is None:
-            print(f"You have to set first the outputs directory with <sim>.set_outputs('<directory>')")
-            return
+            msg = f"You have to set first the outputs directory with <sim>.set_outputs('<directory>')"
+            print(msg)
+            return msg
 
         # Read variables
         vars = self._load_variables(varfile)
         if not vars:
-            return
+            return "No variables found."
+        info.append(f"Simulation in {vars.DIM} dimensions")
         print(f"Simulation in {vars.DIM} dimensions")
-        
-        # Read domains 
+
+        # Read domains
         domains = self._load_domains(vars, domain_prefix)
 
         # Store the variables in the object
@@ -843,60 +1205,97 @@ class Simulation(fargopy.Fargobj):
         # Optionally read dims
         dims = self._load_dims(dimsfile)
         if len(dims):
-            self.dims = dims 
+            self.dims = dims
 
         # Read the summary files
         self.nsnaps = self._get_nsnaps()
+        info.append(f"Number of snapshots in output directory: {self.nsnaps}")
         print(f"Number of snapshots in output directory: {self.nsnaps}")
 
-        # Leer planetas del summary.dat
-        self.planets = self.load_planet_summary(summaryfile)
+        # Read planets from summary.dat using load_planet_states
+        self.planets = self.load_planets(snapshot=0, summaryfile=summaryfile)
         if self.planets:
+            info.append("Planets found in summary.dat:")
             print("Planets found in summary.dat:")
             for planet in self.planets:
-                print(f"  Name: {planet['name']}, Distance: {planet['distance']}, Mass: {planet['mass']}")
+                planet_info = f"  Name: {planet.name}, Initial pos: {planet.posi}, Mass: {planet.mass}"
+                info.append(planet_info)
+                print(planet_info)
         else:
+            info.append("No planet data found in summary.dat.")
             print("No planet data found in summary.dat.")
 
-        print("Configuration variables and domains load into the object. See e.g. <sim>.vars")
+        # Devuelve el string para mostrar en el cuadro de diálogo
+        return "\n".join(info)
 
     def _get_nsnaps(self):
-        """Get the number of snapshots in an output directory
         """
-        error, output = fargopy.Sys.run(f"ls {self.output_dir}/summary[0-9]*.dat")
-        if error == 0:
-            files = output[:-1]
+        Get the number of snapshots in the output directory.
+
+        Returns
+        -------
+        int
+            Number of snapshot files.
+        """
+        try:
+            # List all files in the output directory
+            files = [
+                f
+                for f in os.listdir(self.output_dir)
+                if f.startswith("summary") and f.endswith(".dat")
+            ]
             nsnaps = len(files)
             return nsnaps
-        else:
+        except FileNotFoundError:
             print(f"No summary file in {self.output_dir}")
             return 0
 
     def _load_dims(self, dimsfile):
-        """Parse the dim directory
         """
-        dimsfile = f"{self.output_dir}/{dimsfile}".replace('//','/')
+        Parse the dimensions file.
+
+        Parameters
+        ----------
+        dimsfile : str
+            Name of the dimensions file.
+
+        Returns
+        -------
+        np.ndarray or list
+            Array of dimensions or empty list if not found.
+        """
+        dimsfile = os.path.join(self.output_dir, dimsfile)
         if not os.path.isfile(dimsfile):
-            #print(f"No file with dimensions '{dimsfile}' found.")
+            # print(f"No file with dimensions '{dimsfile}' found.")
             return []
         dims = np.loadtxt(dimsfile)
         return dims
 
-    def _load_variables(self,varfile):
-        """Parse the file with the variables
+    def _load_variables(self, varfile):
         """
+        Parse the file with the simulation variables.
+
+        Parameters
+        ----------
+        varfile : str
+            Name of the variables file.
 
-        varfile = f"{self.output_dir}/{varfile}".replace('//','/')
+        Returns
+        -------
+        fargopy.Dictobj
+            Object containing simulation variables.
+        """
+        varfile = os.path.join(self.output_dir, varfile)
         if not os.path.isfile(varfile):
             print(f"No file with variables named '{varfile}' found.")
             return
 
         print(f"Loading variables")
         variables = np.genfromtxt(
-            varfile,dtype={'names': ("parameters","values"),
-            'formats': ("|S30","|S300")}
+            varfile,
+            dtype={"names": ("parameters", "values"), "formats": ("|S30", "|S300")},
         ).tolist()
-        
+
         vars = dict()
         for posicion in variables:
             str_value = posicion[1].decode("utf-8")
@@ -908,33 +1307,51 @@ class Simulation(fargopy.Fargobj):
                 except:
                     value = str_value
             vars[posicion[0].decode("utf-8")] = value
-        
+
         vars = fargopy.Dictobj(dict=vars)
         print(f"{len(vars.__dict__.keys())} variables loaded")
 
         # Create additional variables
-        variables = ['x', 'y', 'z']
-        if vars.COORDINATES == 'cylindrical':
-            variables = ['phi', 'r', 'z']
-        elif vars.COORDINATES == 'spherical':
-            variables = ['phi', 'r', 'theta']
+        variables = ["x", "y", "z"]
+        if vars.COORDINATES == "cylindrical":
+            variables = ["phi", "r", "z"]
+        elif vars.COORDINATES == "spherical":
+            variables = ["phi", "r", "theta"]
         vars.VARIABLES = variables
 
-        vars.__dict__[f'N{variables[0].upper()}'] = vars.NX
-        vars.__dict__[f'N{variables[1].upper()}'] = vars.NY
-        vars.__dict__[f'N{variables[2].upper()}'] = vars.NZ
+        vars.__dict__[f"N{variables[0].upper()}"] = vars.NX
+        vars.__dict__[f"N{variables[1].upper()}"] = vars.NY
+        vars.__dict__[f"N{variables[2].upper()}"] = vars.NZ
 
         # Dimension of the domain
         vars.DIM = 2 if vars.NZ == 1 else 3
-        
-        return vars
 
-    def _load_domains(self,vars,domain_prefix,
-                      borders=[[],[3,-3],[3,-3]],
-                      middle=True):
+        return vars
 
+    def _load_domains(self, vars, domain_prefix, borders=None, middle=True):
+        """
+        Load domain coordinate arrays from files.
+
+        Parameters
+        ----------
+        vars : fargopy.Dictobj
+            Simulation variables.
+        domain_prefix : str
+            Prefix for domain files.
+        borders : list, optional
+            List of border slices for each variable.
+        middle : bool, optional
+            If True, average between cell coordinates.
+
+        Returns
+        -------
+        fargopy.Dictobj
+            Object containing domain arrays.
+        """
+        if borders is None:
+            borders = [[], [3, -3], [3, -3]]
         # Coordinates
-        variable_suffixes = ['x', 'y', 'z']
+        variable_suffixes = ["x", "y", "z"]
         print(f"Loading domain in {vars.COORDINATES} coordinates:")
 
         # Correct dims in case of 2D
@@ -943,134 +1360,212 @@ class Simulation(fargopy.Fargobj):
 
         # Load domains
         domains = dict()
-        domains['extrema'] = dict()
+        domains["extrema"] = dict()
 
-        for i,variable_suffix in enumerate(variable_suffixes):
-            domain_file = f"{self.output_dir}/{domain_prefix}{variable_suffix}.dat".replace('//','/')
+        for i, variable_suffix in enumerate(variable_suffixes):
+            domain_file = os.path.join(
+                self.output_dir, f"{domain_prefix}{variable_suffix}.dat"
+            )
             if os.path.isfile(domain_file):
-
                 # Load data from file
                 domains[vars.VARIABLES[i]] = np.genfromtxt(domain_file)
 
                 if len(borders[i]) > 0:
                     # Drop the border of the domain
-                    domains[vars.VARIABLES[i]] = domains[vars.VARIABLES[i]][borders[i][0]:borders[i][1]]
+                    domains[vars.VARIABLES[i]] = domains[vars.VARIABLES[i]][
+                        borders[i][0] : borders[i][1]
+                    ]
 
                 if middle:
                     # Average between domain cell coordinates
-                    domains[vars.VARIABLES[i]] = 0.5*(domains[vars.VARIABLES[i]][:-1]+domains[vars.VARIABLES[i]][1:])
-                
+                    domains[vars.VARIABLES[i]] = 0.5 * (
+                        domains[vars.VARIABLES[i]][:-1] + domains[vars.VARIABLES[i]][1:]
+                    )
+
                 # Show indices and value map
-                domains['extrema'][vars.VARIABLES[i]] = [[0,domains[vars.VARIABLES[i]][0]],[-1,domains[vars.VARIABLES[i]][-1]]]
-                
-                print(f"\tVariable {vars.VARIABLES[i]}: {len(domains[vars.VARIABLES[i]])} {domains['extrema'][vars.VARIABLES[i]]}")
+                domains["extrema"][vars.VARIABLES[i]] = [
+                    [0, domains[vars.VARIABLES[i]][0]],
+                    [-1, domains[vars.VARIABLES[i]][-1]],
+                ]
+
+                print(
+                    f"\tVariable {vars.VARIABLES[i]}: {len(domains[vars.VARIABLES[i]])} {domains['extrema'][vars.VARIABLES[i]]}"
+                )
             else:
                 print(f"\tDomain file {domain_file} not found.")
         domains = fargopy.Dictobj(dict=domains)
 
-        return domains        
-
-
-    def load_field(self, fields, slice=None, snapshot=None, type=None, interpolate=False):
+        return domains
+
+    def load_field(
+        self,
+        fields,
+        slice=None,
+        snapshot=None,
+        type=None,
+        interpolate=None,
+        coords="cartesian",
+        cut=None,
+    ):
         """
-        Load a field from the simulation.
-
-        Parameters:
-            fields (str or list of str): Field name(s) to load.
-            slice (str, optional): Slice for 2D data (e.g., 'theta=1.5').
-            snapshot (int, optional): Snapshot index to load.
-            type (str, optional): Type of the field ('scalar' or 'vector'). Default is 'scalar'.
-            interpolate (bool, optional): Whether to interpolate the field. Default is False.
-
-        Returns:
-            fargopy.Field or fargopy.DataHandler: The loaded field(s) or DataHandler object(s).
+        Load a field or multiple fields from the simulation.
         """
-        # Ensure fields is a list
+
+        # Ensure fields is a list (but keep single-field compatibility)
+        single_input = False
         if isinstance(fields, str):
+            single_input = True
             fields = [fields]
 
-        # Interpolation case
-        if interpolate==True:
-            fields_data = []
-            for field in fields:
-                field_data = fargopy.FieldInterpolator(self)
-                # Pass the current field instead of the entire list
-                field_data.load_data(field, slice, snapshot)
-                fields_data.append(field_data)
-            return fields_data if len(fields_data) > 1 else fields_data[0]
-
-        # Non-interpolation case
-        else :
-            if not self.has('vars'):
-                # If the simulation has not loaded the variables
-                dims, vars, domains = self.load_properties()
-
-            # Use snapshot 0 by default if not provided
-            snapshot = 0 if snapshot is None else snapshot
-
-            loaded_fields = []
-            
-            for field in fields:
-                field_data = []
-
-                # Determine the field type if not specified
-                field_type = type
-                if field_type is None:
-                    if field in ['gasdens', 'gasenergy']:
-                        field_type = 'scalar'
-                    elif field == 'gasv':
-                        field_type = 'vector'
-                    else:
-                        raise ValueError(f"Field type for '{field}' could not be inferred. Please specify it explicitly.")
-
-                if field_type == 'scalar':
-                    file_name = f"{field}{str(snapshot)}.dat"
-                    file_field = f"{self.output_dir}/{file_name}".replace('//', '/')
-                    field_data = self._load_field_scalar(file_field)
-                elif field_type == 'vector':
-                    field_data = []
-                    variables = ['x', 'y']
-                    if self.vars.DIM == 3:
-                        variables += ['z']
-                    for variable in variables:
-                        file_name = f"{field}{variable}{str(snapshot)}.dat"
-                        file_field = f"{self.output_dir}/{file_name}".replace('//', '/')
-                        field_data.append(self._load_field_scalar(file_field))
+        # Default behavior: when interpolate is None, treat it as True
+        # to preserve the FieldInterpolator-based API by default.
+        # Backward compatibility:
+        #  - interpolate=True  -> return FieldInterpolator (explicit)
+        #  - interpolate=None  -> treated as True -> return FieldInterpolator
+        #  - interpolate=False -> return legacy Field or list of Fields
+
+        if interpolate is None:
+            interpolate = True
+
+        # --- INTERPOLATE == True : return FieldInterpolator ---
+        if interpolate is True:
+            handler = fargopy.FieldInterpolator(self)
+            handler.load_data(
+                fields=fields, slice=slice, snapshots=snapshot, cut=cut, coords=coords
+            )
+            return handler
+
+        # --- INTERPOLATE == False : legacy single-field behavior ---
+        if not self.has("vars"):
+            dims, vars, domains = self.load_properties()
+
+        snapshot = 0 if snapshot is None else snapshot
+        loaded_fields = []
+
+        for field in fields:
+            # Infer field type unless provided
+            field_type = type
+            if field_type is None:
+                if field in ["gasdens", "gasenergy"]:
+                    field_type = "scalar"
+                elif field == "gasv":
+                    field_type = "vector"
                 else:
-                    raise ValueError(f"fargopy.Field type '{field_type}' not recognized.")
-
-                # Create the field object
-                loaded_field = fargopy.Field(
-                    data=np.array(field_data),
-                    coordinates=self.vars.COORDINATES,
-                    domains=self.domains,
-                    type=field_type
-                )
+                    raise ValueError(f"Field type for '{field}' could not be inferred.")
+
+            # Load scalar
+            if field_type == "scalar":
+                file_name = f"{field}{snapshot}.dat"
+                file_field = os.path.join(self.output_dir, file_name)
+                data = self._load_field_scalar(file_field)
+
+            # Load vector
+            elif field_type == "vector":
+                data = []
+                components = ["x", "y"] + (["z"] if self.vars.DIM == 3 else [])
+                for comp in components:
+                    file_name = f"{field}{comp}{snapshot}.dat"
+                    file_field = os.path.join(self.output_dir, file_name)
+                    data.append(self._load_field_scalar(file_field))
+                data = np.array(data)
+
+            # Create Field
+            loaded_field = fargopy.Field(
+                data=np.array(data),
+                coordinates=self.vars.COORDINATES,
+                domains=self.domains,
+                type=field_type,
+            )
+
+            # Apply slicing
+            if slice:
+                sliced_data, mesh = loaded_field.meshslice(slice=slice)
+                loaded_field = fargopy.Dictobj(dict=dict(data=sliced_data, mesh=mesh))
 
-                # Apply meshslice if a slice is provided
-                if slice:
-                    sliced_data, mesh = loaded_field.meshslice(slice=slice)
-                    # Pass a single dictionary to Dictobj
-                    loaded_field = fargopy.Dictobj(dict=dict(data=sliced_data, mesh=mesh))
+            loaded_fields.append(loaded_field)
 
+        result = loaded_fields if len(loaded_fields) > 1 else loaded_fields[0]
+        return result
 
+    def _load_field_scalar(self, file):
+        """
+        Load a scalar field from a file.
 
-                loaded_fields.append(loaded_field)
-            #print(f'Slice: {slice}.')
-            # Return a single field if only one was provided, or a list if multiple were provided
-            return loaded_fields if len(loaded_fields) > 1 else loaded_fields[0]
+        Parameters
+        ----------
+        file : str
+            Path to the field file.
 
-    def _load_field_scalar(self,file):
-        """Load scalar field from file a file.
+        Returns
+        -------
+        np.ndarray
+            Field data array.
         """
         if os.path.isfile(file):
-            field_data = np.fromfile(file).reshape(int(self.vars.NZ),int(self.vars.NY),int(self.vars.NX))
+            field_data = np.fromfile(file).reshape(
+                int(self.vars.NZ), int(self.vars.NY), int(self.vars.NX)
+            )
             return field_data
         else:
             raise AssertionError(f"File with field '{file}' not found")
-        
-    def load_allfields(self,fluid,snapshot=None,type='scalar'):
-        """Load all fields in the output
+
+    def _load_field_raw(self, field, snapshot=0, field_type=None):
+        """
+        Internal helper to load a single field as a `fargopy.Field` without going
+        through `load_field` dispatching. This prevents recursion when higher-level
+        helpers request raw data.
+        """
+        # Infer type if not provided
+        if field_type is None:
+            if field in ["gasdens", "gasenergy"]:
+                field_type = "scalar"
+            elif field == "gasv":
+                field_type = "vector"
+            else:
+                raise ValueError(f"Field type for '{field}' could not be inferred.")
+
+        # Load scalar
+        if field_type == "scalar":
+            file_name = f"{field}{snapshot}.dat"
+            file_field = os.path.join(self.output_dir, file_name)
+            data = self._load_field_scalar(file_field)
+
+        # Load vector
+        elif field_type == "vector":
+            data = []
+            components = ["x", "y"]
+            if self.vars.DIM == 3:
+                components += ["z"]
+            for comp in components:
+                file_name = f"{field}{comp}{snapshot}.dat"
+                file_field = os.path.join(self.output_dir, file_name)
+                data.append(self._load_field_scalar(file_field))
+            data = np.array(data)
+
+        return fargopy.Field(
+            data=np.array(data),
+            coordinates=self.vars.COORDINATES,
+            domains=self.domains,
+            type=field_type,
+        )
+
+    def load_allfields(self, fluid, snapshot=None, type="scalar"):
+        """
+        Load all fields in the output directory for a given fluid.
+
+        Parameters
+        ----------
+        fluid : str
+            Name of the fluid (e.g., 'gas').
+        snapshot : int, optional
+            Snapshot index to load. If None, loads all snapshots.
+        type : str, optional
+            Field type ('scalar' or 'vector').
+
+        Returns
+        -------
+        fargopy.Dictobj
+            Object containing all loaded fields.
         """
         qall = False
         if snapshot is None:
@@ -1078,14 +1573,16 @@ class Simulation(fargopy.Fargobj):
             fields = fargopy.Dictobj()
         else:
             fields = fargopy.Dictobj()
-        
+
         # Search for field files
-        pattern = f"{self.output_dir}/{fluid}*.dat"
-        error,output = fargopy.Sys.run(f"ls {pattern}")
+        pattern = os.path.join(self.output_dir, f"{fluid}*.dat")
+        import glob
+
+        files_found = sorted(glob.glob(pattern))
 
-        if not error:
+        if files_found:
             size = 0
-            for file_field in output[:-1]:
+            for file_field in files_found:
                 comps = Simulation._parse_file_field(file_field)
                 if comps:
                     if qall:
@@ -1093,16 +1590,16 @@ class Simulation(fargopy.Fargobj):
                         field_name = comps[0]
                         field_snap = int(comps[1])
 
-                        if type == 'scalar':
+                        if type == "scalar":
                             field_data = self._load_field_scalar(file_field)
-                        elif type == 'vector':
+                        elif type == "vector":
                             field_data = []
-                            variables = ['x','y'] 
+                            variables = ["x", "y"]
                             if self.vars.DIM == 3:
-                                variables += ['z'] 
-                            for i,variable in enumerate(variables):
+                                variables += ["z"]
+                            for i, variable in enumerate(variables):
                                 file_name = f"{fluid}{variable}{str(field_snap)}.dat"
-                                file_field = f"{self.output_dir}/{file_name}".replace('//','/')
+                                file_field = os.path.join(self.output_dir, file_name)
                                 field_data += [self._load_field_scalar(file_field)]
                             field_data = np.array(field_data)
                             field_name = field_name[:-1]
@@ -1110,52 +1607,97 @@ class Simulation(fargopy.Fargobj):
                         if str(field_snap) not in fields.keys():
                             fields.__dict__[str(field_snap)] = fargopy.Dictobj()
                         size += field_data.nbytes
-                        (fields.__dict__[str(field_snap)]).__dict__[f"{field_name}"] = fargopy.Field(data=field_data, coordinates=self.vars.COORDINATES, domains=self.domains, type=type)
+                        (fields.__dict__[str(field_snap)]).__dict__[f"{field_name}"] = (
+                            fargopy.Field(
+                                data=field_data,
+                                coordinates=self.vars.COORDINATES,
+                                domains=self.domains,
+                                type=type,
+                            )
+                        )
 
                     else:
                         # Store a specific snapshot
                         if int(comps[1]) == snapshot:
                             field_name = comps[0]
 
-                            if type == 'scalar':
+                            if type == "scalar":
                                 field_data = self._load_field_scalar(file_field)
-                            elif type == 'vector':
+                            elif type == "vector":
                                 field_data = []
-                                variables = ['x','y'] 
+                                variables = ["x", "y"]
                                 if self.vars.DIM == 3:
-                                    variables += ['z'] 
-                                for i,variable in enumerate(variables):
-                                    file_name = f"{fluid}{variable}{str(field_snap)}.dat"
-                                    file_field = f"{self.output_dir}/{file_name}".replace('//','/')
+                                    variables += ["z"]
+                                for i, variable in enumerate(variables):
+                                    file_name = (
+                                        f"{fluid}{variable}{str(field_snap)}.dat"
+                                    )
+                                    file_field = os.path.join(
+                                        self.output_dir, file_name
+                                    )
                                     field_data += [self._load_field_scalar(file_field)]
                                 field_data = np.array(field_data)
                                 field_name = field_name[:-1]
 
                             size += field_data.nbytes
-                            fields.__dict__[f"{field_name}"] = fargopy.Field(data=field_data, coordinates=self.vars.COORDINATES, domains=self.domains, type=type)
+                            fields.__dict__[f"{field_name}"] = fargopy.Field(
+                                data=field_data,
+                                coordinates=self.vars.COORDINATES,
+                                domains=self.domains,
+                                type=type,
+                            )
 
         else:
-            raise ValueError(f"No field found with pattern '{pattern}'. Change the fluid")
-    
+            raise ValueError(
+                f"No field found with pattern '{pattern}'. Change the fluid"
+            )
+
         if qall:
-            fields.snapshots = sorted([int(s) for s in fields.keys() if s != 'size'])
-        fields.size = size/1024**2
+            fields.snapshots = sorted([int(s) for s in fields.keys() if s != "size"])
+        fields.size = size / 1024**2
         return fields
 
     @staticmethod
     def _parse_file_field(file_field):
+        """
+        Parse a field filename to extract the field name and snapshot number.
+
+        Parameters
+        ----------
+        file_field : str
+            Filename of the field.
+
+        Returns
+        -------
+        list or None
+            List with field name and snapshot number, or None if not matched.
+        """
         basename = os.path.basename(file_field)
         comps = None
-        match = re.match('([a-zA-Z]+)(\d+).dat',basename)
+        match = re.match("([a-zA-Z]+)(\d+).dat", basename)
         if match is not None:
-            comps = [match.group(i) for i in range(1,match.lastindex+1)]
+            comps = [match.group(i) for i in range(1, match.lastindex + 1)]
         return comps
 
     def __repr__(self):
+        """
+        String representation of the Simulation object.
+
+        Returns
+        -------
+        str
+        """
         repr = f"""FARGOPY simulation (fargo3d_dir = '{self.fargo3d_dir}', setup = '{self.setup}')"""
         return repr
 
     def __str__(self):
+        """
+        Detailed string with simulation information.
+
+        Returns
+        -------
+        str
+        """
         str = f"""Simulation information:
     FARGO3D directory: {self.fargo3d_dir}
         Outputs: {self.outputs_dir}
@@ -1178,74 +1720,503 @@ class Simulation(fargopy.Fargobj):
     # ##########################################################################
     @staticmethod
     def list_setups():
-        """List setups available in the FARGO3D directory
         """
-        error,output = fargopy.Sys.run(f"ls -d {fargopy.Conf.FP_FARGO3D_DIR}/setups/*")
-        list = ''
-        for setup_dir in output[:-1]:
-            setup_dir = setup_dir.replace('//','/')
-            setup_name = setup_dir.split('/')[-1]
-            setup_par = f"{setup_dir}/{setup_name}.par"
+        Print all valid setup directories detected under ``FP_FARGO3D_DIR``.
+
+        Returns
+        -------
+        None
+            The setups are printed to stdout; nothing is returned.
+        """
+        import glob
+
+        pattern = os.path.join(fargopy.Conf.FP_FARGO3D_DIR, "setups", "*")
+        output = sorted(glob.glob(pattern))
+        list_str = ""
+        for setup_dir in output:
+            setup_dir = setup_dir.replace("//", "/")
+            setup_name = setup_dir.split("/")[-1]
+            setup_par = os.path.join(setup_dir, f"{setup_name}.par")
             if os.path.isfile(setup_par):
-                list += f"Setup '{setup_name}' in '{setup_dir}'\n"
-        print(list)
-                 
+                list_str += f"Setup '{setup_name}' in '{setup_dir}'\n"
+        print(list_str)
+
     @staticmethod
     def list_precomputed():
-        """List the available precomputed simulations
         """
-        for key,item in PRECOMPUTED_SIMULATIONS.items():
-            print(f"{key}:\n\tDescription: {item['description']}\n\tSize: {item['size']} MB")
-    
-    @staticmethod
-    def download_precomputed(setup=None,download_dir='/tmp',quiet=False,clean=True):
-        """Download a precomputed output from Google Drive FARGOpy public repository.
-
-        Args:
-            setup: string, default = None:
-                Name of the setup. For a list see fargopu.PRECOMPUTED_SIMULATIONS dictionary.
+        Display the catalog of downloadable precomputed simulations with descriptions and sizes.
 
-            download_dir: string, default = '/tmp':
-                Directory where the output will be downloaded and uncompressed.
-
-        Optional args:
-            quiet: bool, default = True:
-                If True download quietly (no progress bar).
-            
-            clean: bool, default = False:
-                If True remove the tgz file after uncompressing it.
-
-        Return:
-            If successful returns the output directory.
+        Returns
+        -------
+        None
+            The listing is printed to stdout.
+        """
+        for key, item in PRECOMPUTED_SIMULATIONS.items():
+            print(
+                f"{key}:\n\tDescription: {item['description']}\n\tSize: {item['size']} MB"
+            )
 
+    @staticmethod
+    def download_precomputed(setup=None, download_dir=None, quiet=False, clean=True):
+        """
+        Download and extract a precomputed output archive from the public repository.
+
+        Parameters
+        ----------
+        setup : str, optional
+            Name of the entry in ``PRECOMPUTED_SIMULATIONS``.
+        download_dir : str, optional
+            Destination directory for the compressed file and extracted output.
+        quiet : bool, optional
+            When True, suppress download progress indicators.
+        clean : bool, optional
+            Remove the downloaded archive after extraction when set to True.
+
+        Returns
+        -------
+        str
+            Absolute path to the extracted output directory, or an empty string on failure.
         """
         if setup is None:
-            print(f"You must provide a setup name. Available setups: {list(PRECOMPUTED_SIMULATIONS.keys())}")
-            return ''
+            print(
+                f"You must provide a setup name. Available setups: {list(PRECOMPUTED_SIMULATIONS.keys())}"
+            )
+            return ""
+
+        # Set default download directory based on OS
+        if download_dir is None:
+            if os.name == "nt":  # Windows
+                download_dir = os.path.join(
+                    os.environ.get("TEMP", "C:\\temp"), "fargopy_data"
+                )
+            else:  # Unix-like systems
+                download_dir = "/tmp"
+            # Create directory if it doesn't exist
+            os.makedirs(download_dir, exist_ok=True)
+
         if not os.path.isdir(download_dir):
             print(f"Download directory '{download_dir}' does not exist.")
-            return ''
+            return ""
         if setup not in PRECOMPUTED_SIMULATIONS.keys():
-            print(f"Precomputed setup '{setup}' is not among the available setups: {list(PRECOMPUTED_SIMULATIONS.keys())}")
-            return ''
-        
-        output_dir = (download_dir + '/' + setup).replace('//','/')
+            print(
+                f"Precomputed setup '{setup}' is not among the available setups: {list(PRECOMPUTED_SIMULATIONS.keys())}"
+            )
+            return ""
+
+        output_dir = os.path.join(download_dir, setup)
         if os.path.isdir(output_dir):
             print(f"Precomputed output directory '{output_dir}' already exist")
             return output_dir
         else:
-            filename = setup + '.tgz'
-            fileloc = download_dir + '/' + filename
+            filename = setup + ".tgz"
+            fileloc = os.path.join(download_dir, filename)
             if os.path.isfile(fileloc):
                 print(f"Precomputed file '{fileloc}' already downloaded")
             else:
                 # Download the setups
-                print(f"Downloading {filename} from cloud (compressed size around {PRECOMPUTED_SIMULATIONS[setup]['size']} MB) into {download_dir}")
-                url = PRECOMPUTED_BASEURL + PRECOMPUTED_SIMULATIONS[setup]['id']
-                gdown.download(url,fileloc,quiet=quiet)
-            # Uncompress the setups
-            print(f"Uncompressing {filename} into {output_dir}") 
-            fargopy.Sys.simple(f"cd {download_dir};tar zxf {filename}")
-            print(f"Done.")
-            fargopy.Sys.simple(f"cd {download_dir};rm -rf {filename}")
+                print(
+                    f"Downloading {filename} from cloud (compressed size around {PRECOMPUTED_SIMULATIONS[setup]['size']} MB) into {download_dir}"
+                )
+                url = PRECOMPUTED_BASEURL + PRECOMPUTED_SIMULATIONS[setup]["id"]
+                gdown.download(url, fileloc, quiet=quiet)
+
+            # Uncompress the setups - Windows compatible
+            print(f"Uncompressing {filename} into {output_dir}")
+            try:
+                import tarfile
+
+                with tarfile.open(fileloc, "r:gz") as tar:
+                    tar.extractall(path=download_dir)
+                print(f"Done.")
+
+                # Clean up the tar file if requested
+                if clean:
+                    os.remove(fileloc)
+
+            except Exception as e:
+                print(f"Error uncompressing file: {e}")
+                # Fallback to system command for Unix-like systems
+                if os.name != "nt":
+                    fargopy.Sys.simple(f"cd {download_dir};tar zxf {filename}")
+                    if clean:
+                        fargopy.Sys.simple(f"cd {download_dir};rm -rf {filename}")
+                else:
+                    print(
+                        "Failed to decompress on Windows. Please install tar or 7-zip."
+                    )
+                    return ""
+
+            return output_dir
+
+    def time_scale(self, scale="orbits"):
+        """
+        Calculates the time scale of the simulation in different units.
+
+        Parameters
+        ----------
+        scale : str, optional
+            'orbits' to calculate the number of orbits completed by the planet,
+            'duration' for the total simulation time in simulation units.
+
+        Returns
+        -------
+        float
+            Number of orbits or total simulation time, depending on scale.
+        """
+        import contextlib
+        import io
+        import numpy as np
+
+        with contextlib.redirect_stdout(io.StringIO()):
+            # Load necessary parameters
+            self.load_macros()
+            self.load_planet_summary()
+
+            # Extract parameters from macros and planet summary
+            G = self.G  # Gravitational constant in simulation units
+            M_star = self.simulation_macros["MSTAR"]  # Stellar mass in simulation units
+            planet = self.planets[0]  # Assume the first planet for calculations
+            a = planet["distance"]  # Orbital radius in simulation units
+
+            # Calculate orbital period (T) using Kepler's third law
+            T = 2 * np.pi * np.sqrt(a**3 / (G * M_star))
+
+            # Extract simulation parameters
+            NINTERM = self._load_variables("variables.par").NINTERM
+            DT = self._load_variables("variables.par").DT
+            NTOT = self._load_variables("variables.par").NTOT
+
+            # Calculate total simulation time
+            total_time = NTOT * DT  # Total time in simulation units
+
+            if scale == "orbits":
+                # Calculate the number of orbits completed by the planet
+                orbits_num = total_time / T
+                return orbits_num
 
+            elif scale == "duration":
+                # Return the total simulation time in simulation units
+                return total_time
+
+            else:
+                raise ValueError("Invalid scale. Choose either 'orbits' or 'duration'.")
+
+    def to_paraview(self, snapshot=0, dir=".", basename=None):
+        """
+        Export a snapshot to a VTU (UnstructuredGrid) file with physical XYZ coordinates
+        and point data arrays for density and Cartesian velocity.
+
+        Parameters
+        ----------
+        snapshot : int
+            Snapshot index to export (default: 0).
+        dir : str
+            Output directory where the .vtu file will be written (default: current directory).
+        basename : str or None
+            Base name for the output file (without extension). If None, a default name
+            using the simulation setup and snapshot is created.
+
+        Notes
+        -----
+        - Requires the 'vtk' Python package (and vtk.util.numpy_support).
+        - Expects self.load_field('gasdens') and self.load_field('gasv') to return
+          fargopy.Field-like objects where .data is a numpy array with shapes:
+            rho: (nt, nr, nphi)
+            vel: either (3, nt, nr, nphi) or (nt, nr, nphi, 3)
+        - Domains expected in self.domains as theta, r, phi arrays.
+        - Output is a single .vtu unstructured grid with VTK_VOXEL cells and point data arrays:
+            - "rho" (scalar)
+            - "vel_cart" (3-component vector)
+        """
+        import os
+        import numpy as np
+
+        try:
+            import vtk
+            from vtk.util import numpy_support as ns
+        except Exception as e:
+            raise RuntimeError(
+                "VTK is required for to_paraview. Install the 'vtk' package."
+            ) from e
+
+        # prepare output path
+        os.makedirs(dir, exist_ok=True)
+        if basename is None:
+            base = getattr(self, "setup", "simulation")
+            basename = f"{base}_snap{snapshot}"
+
+        filename = os.path.join(dir, basename)
+
+        # Load fields (tolerant to different return types)
+        gasdens = self.load_field(
+            fields="gasdens", snapshot=snapshot, interpolate=False
+        )
+        gasv = self.load_field(fields="gasv", snapshot=snapshot, interpolate=False)
+
+        rho = np.log10(
+            getattr(gasdens, "data", gasdens) * self.URHO
+        )  # convert to physical units and log10
+        vel = getattr(gasv, "data", gasv) * self.UL / self.UT * 1e-5
+
+        # Normalize shapes
+        if rho.ndim == 4 and rho.shape[0] == 1:
+            rho = rho[0]
+        if rho.ndim != 3:
+            raise ValueError(
+                f"Unexpected rho shape: {rho.shape}. Expected (nt,nr,nphi)."
+            )
+
+        # velocity can be (3,nt,nr,nphi) or (nt,nr,nphi,3)
+        if vel.ndim == 4 and vel.shape[0] == 3:
+            vel_components = vel
+        elif vel.ndim == 4 and vel.shape[-1] == 3:
+            vel_components = np.moveaxis(vel, -1, 0)
+        else:
+            raise ValueError(
+                f"Unexpected vel shape: {vel.shape}. Expected (3,nt,nr,nphi) or (nt,nr,nphi,3)."
+            )
+
+        # Get spherical coordinate grids from self.domains
+        try:
+            theta = self.domains.theta
+            r = self.domains.r * self.UL / self.AU
+            phi = self.domains.phi
+        except Exception:
+            # If domain attribute names differ, try common alternatives
+            try:
+                theta = self.domains.theta
+                r = self.domains.r * self.UL / self.AU
+                phi = self.domains.phi
+            except Exception as e:
+                raise RuntimeError(
+                    "Could not find theta, r, phi in self.domains"
+                ) from e
+
+        # Create meshgrid (vectorized)
+        TT, RR, PP = np.meshgrid(theta, r, phi, indexing="ij")  # shape (nt,nr,nphi)
+
+        # Coordinates in Cartesian
+        X = (RR * np.sin(TT) * np.cos(PP)).ravel(order="C").astype(np.float64)
+        Y = (RR * np.sin(TT) * np.sin(PP)).ravel(order="C").astype(np.float64)
+        Z = (RR * np.cos(TT)).ravel(order="C").astype(np.float64)
+        pts = np.column_stack([X, Y, Z])
+
+        # Cartesian velocity components (vectorized)
+        v_theta = vel_components[0]
+        v_r = vel_components[1]
+        v_phi = vel_components[2]
+
+        v_x = (
+            (
+                v_r * np.sin(TT) * np.cos(PP)
+                + v_theta * np.cos(TT) * np.cos(PP)
+                - v_phi * np.sin(PP)
+            )
+            .ravel(order="C")
+            .astype(np.float64)
+        )
+
+        v_y = (
+            (
+                v_r * np.sin(TT) * np.sin(PP)
+                + v_theta * np.cos(TT) * np.sin(PP)
+                + v_phi * np.cos(PP)
+            )
+            .ravel(order="C")
+            .astype(np.float64)
+        )
+
+        v_z = (
+            (v_r * np.cos(TT) - v_theta * np.sin(TT))
+            .ravel(order="C")
+            .astype(np.float64)
+        )
+
+        vel_cart = np.column_stack([v_x, v_y, v_z])
+
+        # Density flattened
+        rho_flat = rho.ravel(order="C").astype(np.float64)
+
+        # VTK points
+        vtk_points = vtk.vtkPoints()
+        vtk_points.SetData(ns.numpy_to_vtk(pts, deep=True))
+
+        # Build VOXEL connectivity (vectorized)
+        nt, nr, nphi = rho.shape
+        ntm = nt - 1
+        nrm = nr - 1
+        npm = nphi - 1
+        ncells = ntm * nrm * npm
+
+        it, ir_, ip = np.meshgrid(
+            np.arange(ntm), np.arange(nrm), np.arange(npm), indexing="ij"
+        )
+        it = it.ravel()
+        ir_ = ir_.ravel()
+        ip = ip.ravel()
+        base = it * nr * nphi + ir_ * nphi + ip
+
+        p000 = base
+        p001 = base + 1
+        p010 = base + nphi
+        p011 = base + nphi + 1
+        p100 = base + nr * nphi
+        p101 = base + nr * nphi + 1
+        p110 = base + nr * nphi + nphi
+        p111 = base + nr * nphi + nphi + 1
+
+        cells = np.column_stack(
+            [
+                np.full(ncells, 8, dtype=np.int64),
+                p000,
+                p001,
+                p010,
+                p011,
+                p100,
+                p101,
+                p110,
+                p111,
+            ]
+        ).ravel()
+
+        vtk_cells = vtk.vtkCellArray()
+        vtk_cells.SetCells(ncells, ns.numpy_to_vtkIdTypeArray(cells, deep=True))
+
+        # Unstructured grid
+        grid = vtk.vtkUnstructuredGrid()
+        grid.SetPoints(vtk_points)
+        grid.SetCells(vtk.VTK_VOXEL, vtk_cells)
+
+        # Point data arrays
+        vtk_rho = ns.numpy_to_vtk(rho_flat, deep=True)
+        vtk_rho.SetName("rho")
+        vtk_vel = ns.numpy_to_vtk(vel_cart, deep=True)
+        vtk_vel.SetNumberOfComponents(3)
+        vtk_vel.SetName("vel_cart")
+
+        grid.GetPointData().AddArray(vtk_rho)
+        grid.GetPointData().AddArray(vtk_vel)
+
+        # Write VTU
+        writer = vtk.vtkXMLUnstructuredGridWriter()
+        writer.SetFileName(filename + ".vtu")
+        writer.SetInputData(grid)
+        if writer.Write() == 0:
+            raise RuntimeError(f"Failed writing VTU file '{filename}.vtu'.")
+
+        return filename + ".vtu"
+
+
+class Planet:
+    """
+    Represents a planet in the simulation, holding its physical state and properties.
+
+    Attributes
+    ----------
+    name : str
+        Name or label of the planet.
+    mass : float
+        Planet mass (in Msun or simulation units).
+    pos : Planet.Vector
+        Current cartesian position (x, y, z) in AU.
+    vel : Planet.Vector
+        Current cartesian velocity (vx, vy, vz) in AU/UT.
+    posi : Planet.Vector
+        Initial cartesian position (x, y, z) in AU.
+    mstar : float
+        Stellar mass (in Msun or simulation units).
+
+    Properties
+    ----------
+    hill_radius : float
+        The Hill radius of the planet (in AU), computed from its current position and mass.
+
+    Example
+    -------
+    >>> jupiter = planets[0]
+    >>> print(jupiter.pos.x, jupiter.vel.y, jupiter.hill_radius)
+    """
+
+    def __init__(self, name, pos, vel, mass, posi, mstar):
+        self.name = name
+        self.mass = mass
+        self.pos = Planet._Vector(*pos)
+        self.vel = Planet._Vector(*vel)
+        self.posi = Planet._Vector(*posi)
+        self.mstar = mstar
+
+    class _Vector:
+        """
+        Simple vector class for 3D coordinates, allowing attribute and index access.
+
+        Attributes
+        ----------
+        x : float
+            X coordinate.
+        y : float
+            Y coordinate.
+        z : float
+            Z coordinate.
+        """
+
+        def __init__(self, x, y, z):
+            self.x = x
+            self.y = y
+            self.z = z
+
+        def __getitem__(self, idx):
+            """
+            Return the coordinate at index ``idx`` (0→x, 1→y, 2→z).
+            """
+            return [self.x, self.y, self.z][idx]
+
+        def __array__(self):
+            """
+            Expose the vector as a NumPy array for downstream numerical routines.
+            """
+            import numpy as np
+
+            return np.array([self.x, self.y, self.z])
+
+        def __repr__(self):
+            """
+            Debug-friendly string showing the three Cartesian components.
+            """
+            return f"[{self.x}, {self.y}, {self.z}]"
+
+    @property
+    def hill_radius(self):
+        """
+        .. :no-index:
+
+        Returns the Hill radius in AU, using the current position and mass.
+
+        Returns
+        -------
+        float
+            Hill radius in AU.
+        """
+        AU_to_cm = 1.495978707e13
+        Mjup_to_g = 1.898e30
+        Msun_to_g = 1.989e33
+
+        # Distance to star (AU)
+        r_au = (self.pos.x**2 + self.pos.y**2 + self.pos.z**2) ** 0.5
+        m_jup = self.mass * 1e3  # Msun to Mjup
+        mstar_g = float(self.mstar) * Msun_to_g
+        r_cm = r_au * AU_to_cm
+        m_p = m_jup * Mjup_to_g
+
+        r_hill_cm = r_cm * (m_p / (3 * mstar_g)) ** (1 / 3)
+        r_hill_au = r_hill_cm / AU_to_cm
+        return r_hill_au
+
+    def __repr__(self):
+        """
+        String representation of the Planet object.
+
+        Returns
+        -------
+        str
+        """
+        return f"Planet(name={self.name}, mass={self.mass}, pos={self.pos}, vel={self.vel}, posi={self.posi})"