Adding Process ID so PyAEDT can point to a specific instance of Electronics Desktop

[camuher]

The capability is meant to facilitate the use of a launcher(not part of the pull request) from within Electronics Desktop to launch a PyAEDT script that can access all the external libraries available to an installation of CPython while retaining a reference to the instance of Electronics desktop that launched it.

[maxcapodi78]

Please extend the aedt_process_id to all AEDT apps

[maxcapodi78]

@camuher I like this approach. Anyway you should add the aedt_process_id to any of the tools (q2d, q3d, maxwell3d etc...) otherwise you will be able to initialize the app using process id only from Hfss

[camuher]

@maxcapodi78 thank you for the quick reply. As specified in my latest commit I've added it to almost all the tools(and parent classes as needed) which makes for a uniform variable across the board. The only one I couldn't add it to was siwave whose init differs significantly from the structure used by the other tools. I hope this is sufficient to address your request.

[maxcapodi78]

@camuher
you have to ignore siwave
it is different approach.
Please merge from main to get a fix on UT and then I can approve the PR

[camuher]

@maxcapodi78 I've merged to be up to date with main now. I also addressed the error that caused the UT for rmxport and twinbuilder to fail. I hadn't added the variable to their parent classes. There were also miscellaneous edits to make the code more uniform with the style.

[maxcapodi78]

@camuher I still see some style error. I suggest to install pre-commit on your environment.
pip install pre-commit
pre-commit install

It will automatically fix style errors on commit.

PS can you ping me on Teams? I'd like to know who you are :-)

[camuher]

this last commit should correct the remaining edits and (if I'm interpreting the output correctly on the pre-commit) passes the style check

[MaxJPRey]

Suggested change

	Only used when new_desktop_session == False, specifies by process id which instance
	Only used when new_desktop_session = False, specifies by process id which instance

[camuher]

@MaxJPRey in my latest commit i've followed this style while addressing Massimo's comment.

[codecov]

Codecov Report

Merging #1187 (c125bf8) into main (20b47c2) will increase coverage by 0.00%.
The diff coverage is 87.50%.

@@           Coverage Diff           @@
##             main    #1187   +/-   ##
=======================================
  Coverage   80.97%   80.97%           
=======================================
  Files         135      135           
  Lines       44376    44380    +4     
=======================================
+ Hits        35932    35937    +5     
+ Misses       8444     8443    -1     

[maxcapodi78]

Good Improvement

[PipKat]

Suggested change

	print("Launching PyAEDT outside Electronics Desktop with CPython and Pythonnet")
	print("Launching PyAEDT outside Electronics Desktop with CPython and Python.Net.")

[camuher]

Hi @PipKat , while I don't mind committing the small change to this print statement, it is outside the scope of changes I contributed to the code. I'd appreciate the (somewhat redundant) affirmation that it is ok to proceed with this commit.

[PipKat]

Suggested change

	Only used when new_desktop_session = False, specifies by process id which instance
	of electronics desktop to point PyAEDT at.
	Only used when ``new_desktop_session = False``, specifies by process ID which instance
	of Electronics Desktop to point PyAEDT at.