made more robust the launch_ironpython_server. now it is checking the aedt_path

[maxcapodi78]

… and it is checking if connection to the client is successful before initializing it.

[codecov]

Codecov Report

Merging #983 (0ec3cf7) into main (ab885f8) will decrease coverage by 14.58%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main     #983       +/-   ##
===========================================
- Coverage   80.74%   66.16%   -14.59%     
===========================================
  Files         129      198       +69     
  Lines       38350    46867     +8517     
===========================================
+ Hits        30965    31008       +43     
- Misses       7385    15859     +8474     

[akaszynski]

These should raise errors.

[akaszynski]

These should raise errors.

[akaszynski]

This should be logged.

[akaszynski]

This should be logged.

[MaxJPRey]

@maxcapodi78 If we decide to go with the log option instead of the print. We cannot use the AEDT message logger because it won't be available.
We could use the root logger instead.
Something like:

import logging
logger = logging.getLogger(__name__)
logger.setLevel(logging.CRITICAL)

[maxcapodi78]

@akaszynski That's is the only part of pyaedt not logget because it is the linux client_server. this cannot be logged because nothing is started yet. the logs starts after the client is up and running

[MaxJPRey]

Suggested change

# sys.path.insert(0, os.path.join(pyaedt_path, "third_party", "ironpython"))

[MaxJPRey]

Suggested change

#sys.path.insert(0, os.path.join(pyaedt_path, "pyaedt", "third_party", "ironpython"))

[maxcapodi78]

@maxcapodi78 If we decide to go with the log option instead of the print. We cannot use the AEDT message logger because it won't be available. We could use the root logger instead. Something like:

import logging
logger = logging.getLogger(__name__)
logger.setLevel(logging.CRITICAL)

@MaxJPRey If it is ok for you we can take in another PR. We need to test the log and see if no issues are in both Windows and Linux, CPython and IPY