WIP - Feat/ main logger (#640)

* Added procedure for silently omitting some commands (`/NOPR` for example.

* Added test cases for invalid commands and invalid silent commands.

* Format fixing

* Added names to pool instances.

* First implementation of main logger.

* Second revision.

* small changes in log.

* Replacing default logger with pyansys logger.

* Replacing logger with pyansys logger.

* adding last_logger atributte.

* Fixing logger hierachy and file header format.

* Replacing old logger by new one.

* Replacing old logger by new one.

* Fixing instance name input.

* Replacing old logger. Fix.

* Creating tests for log

* Added log tests.

* Using `__name__` instead of hardcoding the name of the loggers.

* Added uncaught exception recording.

* Fixing style

* Adding ``file_record`` and ``console_output`` options to suppress file or console output.

* Adding thread naming.

* Major revamp.
`getLogger` got splitted in several functions to simplify reading.
Added documentation at the head of the file.

* Added documentation.

* Updating modules to match the new log module calling method.

* New revamp of the logger classes

* Updating the calls to the logger.

* Implementing alex comment.

We forgot about distintion between threaded and normal mapdl execution.
We use now instances, which has unique name.

* Updating log module calls

* updating log module calls and removing arguments from logs.

* Added property name and logger.

Also the __getitem__ to get the name from the instance, this is later used for LoggerAdapter

* Adding call to loger and name attribute.

* Again, big changes. Now the format is not conditional upon having or not threads.
It is all based in instances of MAPDL.

* Renaming the library and retooling some parts of the code.

* Updating the logging library call.

* Externalized function to add handlers to logger.
Fix issue with conflicting ``supress_logging`` function.

* Updated modules to function with new logging module.

* Added functions for logging to file or stdout

Added test

* Added info.

* Cleaning up

* Added configuration for copying handlers.

* Updated LOG call.

* Added loglevel to instance logger.

* Fixing style

* Fixing style

* Fix style.

* Fix style.

* Grammar fix.

* Removing walrus operator.

* Deleting old test log.

* Added review suggestions.

* Added review suggestions.

* Applying review comments.

Removed the __getitem__ method to use 'get_mame'.

* Spacing changes.

* Fixing logging tests.

* Spacing fixing.

* Added logging documentation.

* Fixed rst format.

* fix py37-38 difference; fix CORBA

* misc cleanup

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

Author: germa89

ansys/mapdl/core/__init__.py

@@ -1,6 +1,12 @@
 import appdirs
 import os
 
+# Importing logging
+import logging
+from ansys.mapdl.core.logging import Logger
+LOG = Logger(level=logging.ERROR, to_file=False, to_stdout=True)
+LOG.debug('Loaded logging module as LOG')
+
 _LOCAL_PORTS = []
 
 # Per contract with Sphinx-Gallery, this method must be available at top level

ansys/mapdl/core/errors.py

@@ -1,5 +1,7 @@
 """pymapdl specific errors"""
-import logging
+
+from ansys.mapdl.core import LOG as logger
+
 import threading
 import signal
 from functools import wraps
@@ -8,8 +10,6 @@
 
 SIGINT_TRACKER = []
 
-logger = logging.getLogger(__name__)
-
 
 LOCKFILE_MSG = """
 Another ANSYS job with the same job name is already running in this

ansys/mapdl/core/launcher.py

@@ -1,4 +1,5 @@
 """Module for launching MAPDL locally or connecting to a remote instance with gRPC."""
+
 import platform
 from glob import glob
 import re
@@ -36,14 +37,14 @@
 LOCALHOST = "127.0.0.1"
 MAPDL_DEFAULT_PORT = 50052
 
-INTEL_MSG = """Due to incompatibilites between 'DMP', Windows and VPN connections,
+INTEL_MSG = """Due to incompatibilities between 'DMP', Windows and VPN connections,
 the flat '-mpi INTELMPI' is overwritten by '-mpi msmpi'.
 
 If you still want to use 'INTEL', set:
 
 launch_mapdl(..., force_intel=True, additional_switches='-mpi INTELMPI')
 
-Be aware of possible errors or unexpected behaviour with this configuration.
+Be aware of possible errors or unexpected behavior with this configuration.
 """
 
 
@@ -123,7 +124,7 @@ def close_all_local_instances(port_range=None):
         port_range = range(50000, 50200)
 
     @threaded
-    def close_mapdl(port):
+    def close_mapdl(port, name='Closing mapdl thread.'):
         try:
             mapdl = MapdlGrpc(port=port, set_no_abort=False)
             mapdl.exit()