Skip to content

Commit 7712396

Browse files
authored
RFSG examples default value fix and clean up (#2130)
* example changes * changelog update * made rsrc name not required and removed changelog mention --------- Co-authored-by: vnktshr21 <[email protected]>
1 parent b5ffa43 commit 7712396

File tree

3 files changed

+15
-15
lines changed

3 files changed

+15
-15
lines changed

src/nirfsg/examples/nirfsg_arb_waveform.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,11 @@ def example(resource_name, options, frequency, power_level, number_of_samples):
1919

2020
def _main(argsv):
2121
parser = argparse.ArgumentParser(description='Continuously generates an arbitrary waveform using NI-RFSG.', formatter_class=argparse.ArgumentDefaultsHelpFormatter)
22-
parser.add_argument('-n', '--resource-name', default='5841', help='Resource name of the NI RF signal generator.')
22+
parser.add_argument('-n', '--resource-name', default='PXI1Slot2', help='Resource name of the NI RF signal generator.')
2323
parser.add_argument('-f', '--frequency', default=1e9, type=float, help='Frequency in Hz.')
2424
parser.add_argument('-p', '--power-level', default=-10.0, type=float, help='Power level in dBm.')
2525
parser.add_argument('-s', '--number-of-samples', default=1000, type=int, help='Number of samples.')
26-
parser.add_argument('-op', '--option-string', default='Simulate=1, DriverSetup=Model:5841', type=str, help='Option string for the session.')
26+
parser.add_argument('-op', '--option-string', default='', type=str, help='Option string for the session.')
2727
args = parser.parse_args(argsv)
2828
example(args.resource_name, args.option_string, args.frequency, args.power_level, args.number_of_samples)
2929

@@ -33,12 +33,12 @@ def main():
3333

3434

3535
def test_example():
36-
options = "Simulate=1, DriverSetup=Model:5841"
37-
example('5841', options, 1e9, -10.0, 1000)
36+
options = {'simulate': True, 'driver_setup': {'Model': '5841', }, }
37+
example('simulated5841', options, 1e9, -10.0, 1000)
3838

3939

4040
def test_main():
41-
cmd_line = ['--resource-name', '5841', '--frequency', '1e9', '--power-level', '-10', '--number-of-samples', '1000', '--option-string', 'Simulate=1, DriverSetup=Model:5841']
41+
cmd_line = ['--resource-name', 'simulated5841', '--frequency', '1e9', '--power-level', '-10', '--number-of-samples', '1000', '--option-string', 'Simulate=1, DriverSetup=Model:5841']
4242
_main(cmd_line)
4343

4444

src/nirfsg/examples/nirfsg_cw.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,10 @@ def example(resource_name, options, frequency, power_level):
1919

2020
def _main(argsv):
2121
parser = argparse.ArgumentParser(description='Generates a continuous wave (CW) signal using NI-RFSG.', formatter_class=argparse.ArgumentDefaultsHelpFormatter)
22-
parser.add_argument('-n', '--resource-name', default='5841', help='Resource name of the NI RF signal generator.')
22+
parser.add_argument('-n', '--resource-name', default='PXI1Slot2', help='Resource name of the NI RF signal generator.')
2323
parser.add_argument('-f', '--frequency', default=1e9, type=float, help='Frequency in Hz.')
2424
parser.add_argument('-p', '--power-level', default=-10.0, type=float, help='Power level in dBm.')
25-
parser.add_argument('-op', '--option-string', default='Simulate=1, DriverSetup=Model:5841', type=str, help='Option string for the session.')
25+
parser.add_argument('-op', '--option-string', default='', type=str, help='Option string for the session.')
2626
args = parser.parse_args(argsv)
2727
example(args.resource_name, args.option_string, args.frequency, args.power_level)
2828

@@ -32,12 +32,12 @@ def main():
3232

3333

3434
def test_example():
35-
options = "Simulate=1, DriverSetup=Model:5841"
36-
example('5841', options, 1e9, -10.0)
35+
options = {'simulate': True, 'driver_setup': {'Model': '5841', }, }
36+
example('simulated5841', options, 1e9, -10.0)
3737

3838

3939
def test_main():
40-
cmd_line = ['--resource-name', '5841', '--frequency', '1e9', '--power-level', '-10', '--option-string', 'Simulate=1, DriverSetup=Model:5841']
40+
cmd_line = ['--resource-name', 'simulated5841', '--frequency', '1e9', '--power-level', '-10', '--option-string', 'Simulate=1, DriverSetup=Model:5841']
4141
_main(cmd_line)
4242

4343

src/nirfsg/examples/nirfsg_script.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,11 @@ def example(resource_name, options, frequency, power_level, number_of_samples):
2828

2929
def _main(argsv):
3030
parser = argparse.ArgumentParser(description='Generates a signal based on the script provided.', formatter_class=argparse.ArgumentDefaultsHelpFormatter)
31-
parser.add_argument('-n', '--resource-name', default='5841', help='Resource name of the NI RF signal generator.')
31+
parser.add_argument('-n', '--resource-name', default='PXI1Slot2', help='Resource name of the NI RF signal generator.')
3232
parser.add_argument('-f', '--frequency', default=1e9, type=float, help='Frequency in Hz.')
3333
parser.add_argument('-p', '--power-level', default=-10.0, type=float, help='Power level in dBm.')
3434
parser.add_argument('-s', '--number-of-samples', default=1000, type=int, help='Number of samples.')
35-
parser.add_argument('-op', '--option-string', default='Simulate=1, DriverSetup=Model:5841', type=str, help='Option string for the session.')
35+
parser.add_argument('-op', '--option-string', default='', type=str, help='Option string for the session.')
3636
args = parser.parse_args(argsv)
3737
example(args.resource_name, args.option_string, args.frequency, args.power_level, args.number_of_samples)
3838

@@ -42,12 +42,12 @@ def main():
4242

4343

4444
def test_example():
45-
options = "Simulate=1, DriverSetup=Model:5841"
46-
example('5841', options, 1e9, -10.0, 1000)
45+
options = {'simulate': True, 'driver_setup': {'Model': '5841', }, }
46+
example('simulated5841', options, 1e9, -10.0, 1000)
4747

4848

4949
def test_main():
50-
cmd_line = ['--resource-name', '5841', '--frequency', '1e9', '--power-level', '-10', '--number-of-samples', '1000', '--option-string', 'Simulate=1, DriverSetup=Model:5841']
50+
cmd_line = ['--resource-name', 'simulated5841', '--frequency', '1e9', '--power-level', '-10', '--number-of-samples', '1000', '--option-string', 'Simulate=1, DriverSetup=Model:5841']
5151
_main(cmd_line)
5252

5353

0 commit comments

Comments
 (0)