Add default value to nislsc python api

[zoechanzy]

What does this Pull Request accomplish?

• Update nislscapi_full.json
• Update function_helper.py and templates files to consume default value from nislscapi_full.json
• Added default value of None for the library parameter
• Regenerate generated python modules files
• Refactor parameter ordering in unit tests and examples

Why should this Pull Request be merged?

nislscapi.json is updated to have default values. Function helper and template files should be updated so that the default value can be used in code-gen.

What testing has been done?

• Passed unit tests
• Passed examples:
  -- Result of poetry run .\examples\show_command_and_property_tree.py SLSC-12001-0368CF24

{
   "Name": "SLSC-12001-0368CF24",
   "Commands": [],
   "Properties": [
       {
           "name": "PowerOnHours",
           "current_value": 1014,
           "datatype": "INT32",
           "range": "[0, ]",
           "access": "READ_WRITE",
           "description": "Recorded power-on hours of the chassis since last reset of this property. This property is persistent across reboots and factory resets.",
           "documentation": ""
       },
       {
           "name": "SupportedSlotCoolingCapacities",
           "current_value": [
               50
           ],
           "datatype": "INT32_ARRAY",
           "range": "",
           "access": "READ_ONLY",
           "description": "The cooling capacities supported by the chassis. The cooling capacity is used to determine the fan curve for the chassis. The first value listed is the default cooling capacity.",
           "documentation": ""
       },
       {
           "name": "CurrentSlotCoolingCapacity",
           "current_value": 50,
           "datatype": "INT32",
           "range": "",
           "access": "READ_WRITE",
           "description": "The current cooling capacity of the chassis. The cooling capacity is used to determine the fan curve for the chassis.",
           "documentation": ""
       },
       {
           "name": "SupportsProtectiveEarthing",
           "current_value": false,
           "datatype": "BOOL",
           "range": "",
           "access": "READ_ONLY",
           "description": "Describes whether or not this chassis supports Protective Earthing.",
           "documentation": ""
       }
   ],
   "SLSC-12001-0368CF24/BatteryVoltageSensor": {
       "Commands": [],
       "Properties": [
           {
               "name": "SensorReading",
               "current_value": 3.664550778248,
               "datatype": "DOUBLE",
               "range": "",
               "access": "READ_ONLY",
               "description": "The current voltage of the battery in unit voltage.",
               "documentation": ""
           },
           {
               "name": "SensorNominal",
               "current_value": 3.6,
               "datatype": "DOUBLE",
               "range": "",
               "access": "READ_ONLY",
               "description": "The nominal voltage of the battery in unit voltage.",
               "documentation": ""
           },
           {
               "name": "SensorLowerCritical",
               "current_value": 2.9,
               "datatype": "DOUBLE",
               "range": "",
               "access": "READ_ONLY",
               "description": "The lower battery threshold in unit voltage.",
               "documentation": ""
           },
           {
               "name": "HealthState",
               "current_value": 0,
               "datatype": "INT32",
               "range": "",
               "access": "READ_ONLY",
               "description": "Health State of the chassis battery",
               "documentation": ""
           }
       ]
   },
   "SLSC-12001-0368CF24/FanVoltageSensor": {
       "Commands": [],
       "Properties": [
           {
               "name": "SensorReading",
               "current_value": 0.105590820226,
               "datatype": "DOUBLE",
               "range": "",
               "access": "READ_ONLY",
               "description": "The current voltage of the fan in unit voltage.",
               "documentation": ""
           },
           {
               "name": "SensorLowerCritical",
               "current_value": 0.094,
               "datatype": "DOUBLE",
               "range": "",
               "access": "READ_ONLY",
               "description": "The fan's lower threshold of the operating voltage in unit voltage.",
               "documentation": ""
           },
           {
               "name": "SensorUpperCritical",
               "current_value": 0.12,
               "datatype": "DOUBLE",
               "range": "",
               "access": "READ_ONLY",
               "description": "The fan's upper threshold of the operating voltage in unit voltage.",
               "documentation": ""
           },
           {
               "name": "HealthState",
               "current_value": 0,
               "datatype": "INT32",
               "range": "",
               "access": "READ_ONLY",
               "description": "Health State of the chassis fan.",
               "documentation": ""
           }
       ]
   }
}

-- Result of poetry run .\examples\reset_device.py SLSC-12001-0368CF24

Battery health state: Normal
Sensor reading: 3.664550778248 V
Battery needs replacement?: False

-- Result of poetry run .\examples\check_chassis_battery.py SLSC-12001-0368CF24

Reset command sent to device(s) SLSC-12001-0368CF24

Related work item:
USER STORY 3221873

[ni-github-admins]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scan Engine	Critical	High	Medium	Low	Total (0)
✅	Code Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[github-actions]

Test Results

   14 files  ±0     14 suites  ±0   14m 47s ⏱️ -49s
  224 tests ±0    224 ✅ ±0  0 💤 ±0  0 ❌ ±0 
3 136 runs  ±0  3 136 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit abc88dd. ± Comparison against base commit 940ed0b.

♻️ This comment has been updated with latest results.

[Copilot]

Pull request overview

This PR updates the codegen metadata and templates so generated NI-SLSC Python APIs can expose parameter default values (notably library=None, timeouts, and “default resource” sentinels), and regenerates the affected modules and tests accordingly.

Changes:

• Added default fields to nislscapi_full.json for various parameters (timeouts, default resources).
• Updated codegen helpers/templates to emit defaults in generated Python signatures (and to reorder required vs defaulted parameters).
• Regenerated generated/nislsc/session.py and updated unit tests/examples to match the new call signatures.

Reviewed changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
tests/unit/test_session.py	Updates test calls to match reordered/generated Session method signatures and new keyword usage.
tests/unit/conftest.py	Updates the session fixture to use new initialize_session_with_devices signature/keywords.
src/codegen/utilities/function_helpers.py	Adds “include defaults” support, default formatting helpers, and signature reordering logic.
src/codegen/templates/utils.py.mako	Enables default emission for module-level generated functions.
src/codegen/templates/session.py.mako	Enables default emission for generated Session methods/classmethods.
src/codegen/templates/property.py.mako	Enables default emission for generated PropertyReference APIs.
src/codegen/templates/library.py.mako	Enables default emission for generated Library APIs.
src/codegen/templates/command.py.mako	Enables default emission for generated CommandReference APIs.
src/codegen/metadata/nislscapi_full.json	Adds default values in metadata (timeouts, $Default* resource sentinels).
generated/nislsc/session.py	Regenerated Session API with defaults and reordered parameters.
examples/show_command_and_property_tree.py	Updates example calls to match new Session method parameter order.
examples/reset_device.py	Updates example initialization call to use new keyword/default patterns.
examples/check_chassis_battery.py	Updates example property calls to match new parameter ordering/defaults.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[tjying95]

Perform the tests on actual hardware and make sure the examples still work. Also try writing a simple python script to test if the default parameter is working as intended, especially the $DefaultDevices and $DefaultPhysChans.