FIX: Bug located in primitives circuit module

[ramin4667]

Description

**This PR fixes the error due to non-digit circuit element IDs. **

Issue linked

**The post-processing call for the simulated circuit of this example https://examples.aedt.docs.pyansys.com/version/dev/examples/high_frequency/radiofrequency_mmwave/lumped_element.html raises an error. **

Checklist

• I have tested my changes locally.
• I have added necessary documentation or updated existing documentation.
• I have followed the coding style guidelines of this project.
• I have added appropriate tests (unit, integration, system).
• I have reviewed my changes before submitting this pull request.
• I have linked the issue or issues that are solved by the PR if any.
• I have agreed with the Contributor License Agreement (CLA).

[codecov]

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 83.49%. Comparing base (3241dff) to head (2896265).

❌ Your patch check has failed because the patch coverage (75.00%) is below the target coverage (85.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6775   +/-   ##
=======================================
  Coverage   83.49%   83.49%           
=======================================
  Files         245      245           
  Lines       77511    77511           
=======================================
+ Hits        64714    64718    +4     
+ Misses      12797    12793    -4     

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[gmalinve]

Hi @ramin4667 ,

Do you have an issue linked to this PR? if yes please link it.

Thanks,

KR,

Giulia

[ramin4667]

Hi @ramin4667 ,

Do you have an issue linked to this PR? if yes please link it.

Thanks,

KR,

Giulia

Hi @gmalinve, The PR is updated with a link to an issue.

[gmalinve]

Hi @ramin4667, yes. I linked it before.

[myoung301]

I see that the other occurrences of setting id have been updated and the tests are passing still.

[myoung301]

@ramin4667 There is at least one change needed for the tests. I saw a failure caused by the test looking for an int where the type is now a string. Please review the failing tests.

[ramin4667]

@ramin4667 There is at least one change needed for the tests. I saw a failure caused by the test looking for an int where the type is now a string. Please review the failing tests.

@myoung301 Tests are updated and passing.

[myoung301]

Reviewed changes and looks good.