FEAT: Farfield refactoring

[Samuelopez-ansys]

No description provided.

[ansys-reviewer-bot]

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

[codecov]

Codecov Report

Attention: Patch coverage is 90.60078% with 97 lines in your changes missing coverage. Please review.

Project coverage is 83.65%. Comparing base (b0f022b) to head (bd57746).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4893      +/-   ##
==========================================
+ Coverage   82.26%   83.65%   +1.38%     
==========================================
  Files         120      121       +1     
  Lines       54561    54799     +238     
==========================================
+ Hits        44886    45843     +957     
+ Misses       9675     8956     -719     

[Samuelopez-ansys]

@SMoraisAnsys Any idea why documentation style is failing?

[SMoraisAnsys]

Thanks for this huge rework and great use of "scope" with variable/method names !
Warning: while this is a great change, you have to really rack your brain to ensure that the scope you defined won't change later on as this will be though to handle !

I left some minor comments. Among them is the fact of returning directly a value instead of using else statement before. JFYI, if we don't specify a value to be returned, Python returns None by default. So, we can discard return None if you want !