Skip to content

Commit

Permalink
Merge pull request #117 from jakeogh/AFG3000_fix_interim
Browse files Browse the repository at this point in the history
AFG3000: use PermissiveMultiples until MultiType can AND
  • Loading branch information
jenshnielsen committed Feb 10, 2022
2 parents 7fcb326 + 193676d commit 0519fe3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions qcodes_contrib_drivers/drivers/Tektronix/AFG3000.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ def __init__(self, name, address, **kwargs):
get_cmd=f'SOURce{src}:AM:DEPTh?',
get_parser=float,
set_cmd=f'SOURce{src}:AM:DEPTh {{}}PCT',
vals=vals.Multiples(divisor=0.1, min_value=0, max_value=120)
vals=vals.PermissiveMultiples(divisor=0.1) # TODO min_value=0, max_value=120
)

# Frequency modulation
Expand Down Expand Up @@ -104,7 +104,7 @@ def __init__(self, name, address, **kwargs):
get_cmd=f'SOURce{src}:{mod_type}:INTernal:FREQuency?',
get_parser=float,
set_cmd=f'SOURce{src}:{mod_type}:INTernal:FREQuency {{}}Hz',
vals=vals.Multiples(divisor=1e-3, min_value=2e-3, max_value=5e4)
vals=vals.PermissiveMultiples(divisor=1e-3) # TODO min_value=2e-3, max_value=5e4
)
self.add_parameter(
name=f'{mod_type.lower()}_internal_function{src}',
Expand Down

0 comments on commit 0519fe3

Please sign in to comment.