nirfsg: Enable complex number support for gRPC and update system tests

[rahur-NI]

• This contribution adheres to CONTRIBUTING.md.
• I've updated CHANGELOG.md if applicable.
• I've added tests applicable for this pull request

What does this Pull Request accomplish?

• Updated numpy_read_method.py.mako template with snippets copied over from default_method.py.mako and also included gRPC generation support for numpy complex methods.
• Updated _grpc_stub_interpreter.py.mako template to import nidevice_pb2 as grpc_complex_types only if complex numbers are used in the module.
• Updated 'included_in_proto' in functions.py of niFake to generate the gRPC changes in functions which has complex numpy values. Also updated nifake.proto file to generate the necessary changes to include in the nifake unit tests.
• Added grpc_server_config.json file for nirfsg to include the address and port for server listening.

List issues fixed by this Pull Request below, if any.

NA

What testing has been done?

• Ensured the codegen modifications are as expected.
• Added a new TestGrpc class in test_system_nirfsg.py to include gRPC-based system test session setup.
• As the fix for a defect in nirfsg grpc-device metadata was merged recently, test_get_all_named_waveform_names test will be only passing when the next release version of grpc-device server is used for nimi-python system tests. So this test case was moved from common path to TestLibrary class in test_system_nirfsg.py file to skip being tested against grpc. This movement will be reverted once we update the grpc-device version being used later.
• Existing system tests are passing when tested against TestLibrary and TestGrpc class included in simulated session.
• Existing system tests are tested against hardware by keeping use_simulated_session option as False, all the nirfsg test cases are passing as expected.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 46.15385% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.90%. Comparing base (448825e) to head (0217316).

Files with missing lines	Patch %	Lines
generated/nirfsg/nirfsg/_grpc_stub_interpreter.py	0.00%	13 Missing ⚠️
generated/nifake/nifake/_grpc_stub_interpreter.py	92.30%	1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (46.15%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage.
❌ Your project status has failed because the head coverage (78.90%) is below the target coverage (85.00%). You can increase the head coverage or adjust the target coverage.

❗ There is a different number of reports uploaded between BASE (448825e) and HEAD (0217316). Click for more details.

HEAD has 31 uploads less than BASE

Flag	BASE (448825e)	HEAD (0217316)
nifakeunittests	5	1
nidcpowerunittests	5	1
nidigitalunittests	5	1
nimodinstunittests	5	1
niscopeunittests	5	1
nitclkunittests	5	1
nimodinstsystemtests	1	0
nisesystemtests	1	0
nidmmsystemtests	1	0
nidigitalsystemtests	1	0
nidcpowersystemtests	1	0
nifgensystemtests	1	0
nitclksystemtests	1	0

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2172      +/-   ##
==========================================
- Coverage   88.76%   78.90%   -9.87%     
==========================================
  Files          73       46      -27     
  Lines       18986    10119    -8867     
==========================================
- Hits        16853     7984    -8869     
- Misses       2133     2135       +2     

Flag	Coverage Δ
codegenunittests	84.90% <ø> (ø)
nidcpowersystemtests	?
nidcpowerunittests	89.53% <ø> (ø)
nidigitalsystemtests	?
nidigitalunittests	68.44% <ø> (ø)
nidmmsystemtests	?
nifakeunittests	86.01% <92.30%> (+0.33%)	⬆️
nifgensystemtests	?
nimodinstsystemtests	?
nimodinstunittests	94.20% <ø> (ø)
nirfsgsystemtests	70.17% <0.00%> (-2.85%)	⬇️
niscopesystemtests	84.94% <ø> (-8.00%)	⬇️
niscopeunittests	43.20% <ø> (ø)
nisesystemtests	?
niswitchsystemtests	71.26% <ø> (-10.78%)	⬇️
nitclksystemtests	?
nitclkunittests	98.26% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
generated/nifake/nifake/_grpc_stub_interpreter.py	85.11% <92.30%> (+1.71%)	⬆️
generated/nirfsg/nirfsg/_grpc_stub_interpreter.py	0.00% <0.00%> (ø)

... and 39 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 448825e...0217316. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ni-jfitzger]

You should add new tests to src\nifake\unit_tests\test_grpc.py

[ni-jfitzger]

I'm worried about the system test results. We seem to have hung as soon as we reached the first nirfsg gRPC test for 32-bit. You may want to investigate on a local system.

[ni-jfitzger]

Can you add a comment about why this test was moved to TestLibrary and when it can be moved back?

[rahur-NI]

Updated

[rahur-NI]

Tried to move this back within the common path to check whether the failure in system test is anyway related to this. But it doesnt seems to be. Will investigate further regarding the nirfsg system test failure and update again.

[ni-jfitzger]

travis says you need to run codegen again.