Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add sizemode property for Sensor, pixel and Dipole #658

Merged
merged 21 commits into from
Aug 23, 2023

Conversation

Alexboiboi
Copy link
Member

@Alexboiboi Alexboiboi commented Aug 17, 2023

import magpylib as magpy

dips = [
    magpy.misc.Dipole(
        moment=(0, 0, 1),
        position=(size, 0 if sizemode == "scaled" else 1, 0),
        style_size=size,
        style_sizemode=sizemode,
        style_pivot="tail",
        style_label=f"{sizemode=}, {size=}",
        style_description_show=False,
    )
    for size in range(1, 4)
    for sizemode in ("scaled", "absolute")
]
magpy.show(*dips)
image
import magpylib as magpy

sens = [
    magpy.Sensor(
        position=(size - 3, 0 if sizemode == "scaled" else 2, 0),
        style_size=size,
        style_sizemode=sizemode,
        style_label=f"{sizemode=}, {size=}",
        style_description_show=False,
    )
    for size in range(1, 4)
    for sizemode in ("scaled", "absolute")
]
magpy.show(*sens)
image
import magpylib as magpy
import numpy as np

sens = [
    magpy.Sensor(
        pixel=[(0, 0, 0), (1, 0, 0), (2, 0, 0)],
        position=(0, 0 if pixel_sizemode == "scaled" else 1, 0),
        style_pixel_size=0.5,
        style_pixel_sizemode=pixel_sizemode,
        style_label=f"{pixel_sizemode=}, pixel_size=0.5",
        style_description_show=False,
    )
    for pixel_sizemode in ("scaled", "absolute")
]
magpy.show(*sens)
image

@Alexboiboi Alexboiboi added this to the 4.4 milestone Aug 17, 2023
@Alexboiboi Alexboiboi self-assigned this Aug 17, 2023
@Alexboiboi Alexboiboi marked this pull request as draft August 17, 2023 21:09
@codecov
Copy link

codecov bot commented Aug 17, 2023

Codecov Report

Merging #658 (7d779c5) into main (b9c8be1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #658   +/-   ##
=======================================
  Coverage   99.26%   99.26%           
=======================================
  Files          55       55           
  Lines        7073     7095   +22     
=======================================
+ Hits         7021     7043   +22     
  Misses         52       52           
Files Changed Coverage Δ
magpylib/_src/defaults/defaults_values.py 100.00% <ø> (ø)
magpylib/_src/display/backend_matplotlib.py 100.00% <100.00%> (ø)
magpylib/_src/display/traces_core.py 100.00% <100.00%> (ø)
magpylib/_src/style.py 99.74% <100.00%> (+<0.01%) ⬆️

magpylib/_src/obj_classes/class_current_Line.py Dismissed Show dismissed Hide dismissed
magpylib/_src/obj_classes/class_current_Loop.py Dismissed Show dismissed Hide dismissed
magpylib/_src/obj_classes/class_magnet_Cuboid.py Dismissed Show dismissed Hide dismissed
magpylib/_src/obj_classes/class_magnet_Cylinder.py Dismissed Show dismissed Hide dismissed
magpylib/_src/obj_classes/class_magnet_CylinderSegment.py Dismissed Show dismissed Hide dismissed
magpylib/_src/obj_classes/class_magnet_Sphere.py Dismissed Show dismissed Hide dismissed
magpylib/_src/obj_classes/class_misc_Dipole.py Dismissed Show dismissed Hide dismissed
magpylib/_src/display/traces_core.py Dismissed Show dismissed Hide dismissed
magpylib/_src/display/traces_generic.py Dismissed Show dismissed Hide dismissed
Copy link
Member

@OrtnerMichael OrtnerMichael left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool stuff

@Alexboiboi
Copy link
Member Author

@Alexboiboi Alexboiboi marked this pull request as ready for review August 23, 2023 10:23
@Alexboiboi Alexboiboi merged commit 85a3ac5 into main Aug 23, 2023
12 checks passed
@Alexboiboi Alexboiboi deleted the sizemode_sensor&dipole branch August 24, 2023 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants