Removing unnecessary exception log. (#676)

* Removing innecesary exception log.

* Adding a meaningful error message.
And cleaning up a bit logging.py

* Added more info in comments.

* Grammar fixing

* Update ansys/mapdl/core/mapdl.py

Co-authored-by: Alex Kaszynski <[email protected]>

Co-authored-by: Alex Kaszynski <[email protected]>

Author: germa89

ansys/mapdl/core/logging.py

@@ -130,7 +130,7 @@
 
 ## Formatting
 
-STDOUT_MSG_FORMAT = '%(levelname)s - %(instance_name)s -  %(module)s - %(funcName)s - %(message)s, defaults={"instance_name": None}'
+STDOUT_MSG_FORMAT = '%(levelname)s - %(instance_name)s -  %(module)s - %(funcName)s - %(message)s'
 
 FILE_MSG_FORMAT = STDOUT_MSG_FORMAT
 
@@ -424,7 +424,6 @@ def _add_mapdl_instance_logger(self, name, mapdl_instance, level):
             raise ValueError("You can only input 'str' classes to this method.")
 
         return instance_logger
-        # return.level = None # TODO: To remove
 
     def add_instance_logger(self, name, mapdl_instance, level=None):
         """Create a logger for a MAPDL instance.

ansys/mapdl/core/mapdl.py

@@ -2113,8 +2113,11 @@ def run(self, command, write_to_log=True, **kwargs):
                 response = re.sub(err_str, "", response)
 
             if "*** ERROR ***" in response:
-                self._log.error(self._response)
-                raise MapdlRuntimeError(self._response)
+                # We don't need to log exception because they already included in the main logger.
+                # logger.error(self._response)
+                # However, exceptions are recorded in the global logger which do not record
+                # information of the instances name, hence we edit the error message.
+                raise MapdlRuntimeError(f"\n\nError in instance {self._name}\n\n" + self._response)
             else:
                 warnings.warn(
                     "MAPDL returned non-abort errors.  Please " "check the logs."