Non-isothermal charge simulations

[marc-flex]

Greptile Summary

Added support for non-isothermal charge simulations by introducing SteadyChargeDCAnalysis base class and refactoring the existing IsothermalSteadyChargeDCAnalysis, enabling temperature variation effects in charge simulations.

• Split analysis classes in spice/analysis/dc.py to support both temperature-varying and fixed-temperature simulations while maintaining backward compatibility
• Added validation in tcad/simulation/heat_charge.py to ensure structures have both electrical and thermal properties for non-isothermal simulations
• Enhanced test fixtures in test_heat_charge.py with comprehensive thermal conductivity and medium components
• Modified package exports in __init__.py to include new SteadyChargeDCAnalysis class

[github-actions]

Diff Coverage

Diff: origin/develop...HEAD, staged and unstaged changes

• tidy3d/components/spice/analysis/dc.py (100%)
• tidy3d/components/spice/types.py (100%)
• tidy3d/components/tcad/simulation/heat_charge.py (92.0%): Missing lines 985,990

Summary

• Total: 30 lines
• Missing: 2 lines
• Coverage: 93%

tidy3d/components/tcad/simulation/heat_charge.py

  981                     "The current simulation is defined as non-isothermal but no solid "
  982                     "materials with heat properties have been defined. "
  983                 )
  984             elif not has_elec and has_heat:
! 985                 raise SetupError(
  986                     "The current simulation is defined as non-isothermal but no "
  987                     "semiconductor materials have been defined. "
  988                 )
  989             elif not has_heat and not has_elec:
! 990                 raise SetupError(
  991                     "The current simulation is defined as non-isothermal but no "
  992                     "solid or semiconductor materials have been defined. "
  993                 )
  994         return values

---

[yaugenst-flex]

@greptileai

[greptile-apps]

_{5 files reviewed, 3 comments
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

style: Base class docstring should be more descriptive, mentioning this is for both isothermal and non-isothermal simulations

[greptile-apps]

style: Docstring should clarify this is specifically for isothermal (constant temperature) simulations