Skip to content

Issue1177 debug - #1178

Open
Didou09 wants to merge 17 commits into
develfrom
Issue1177_Debug
Open

Didou09 wants to merge 17 commits into
develfrom
Issue1177_Debug

Conversation

@Didou09

@Didou09 Didou09 commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

Main changes:

  • coll.add_rays_from_diagnostic(key_cam=str or list) implemented
  • spectro.spectral_range2():
    • restructured with _check.py, _compute.py and _plot.py
    • More robust vs inputs
    • More explicit keywords
    • Allows for multiple combinations of aperture, crystal, camera
    • Ray-tracing now focuses on lamb0, lamb0_min and lamb0_max
    • Markers now show which components is limiting (aperture with semi_angle_max, crystal length or camera)
  • Magnetic sheilding:
    • physics_tools.magnetic.shielding.single() operational
    • physics_tools.magnetic.shielding.double() exists

Issues:

Fixes, in devel:

Magnetic shielding routines should be merge into a single one handling {transverse + axial} for multiple layers
=> #1179 opened

Examples:

spectro.spectral_range_2d()
image
image

physics_tools.magnetic.shielding.single()

dout, dax = tf.physics_tools.magnetic.shielding.single(
    diam_inner_ref=0.15, 
    diam_outer=0.15 + np.linspace(0.001, 0.05, 31), 
    diam_outer_ref=0.16, 
    Bin_lim_T=1e-3,
)
image

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants