Adding Improve DC Noise Rejection example in NI-DMM

[divakar-selvam07]

• 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?

• Adds nidmm_improve_dc_noise_rejection.py, a Python adaptation of the LabVIEW “Improve DC Noise Rejection” example.
• Updates CHANGELOG.md to call out the new examples in the unreleased section.

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

None

What testing has been done?

• Manually validated with a simulated instrument and an actual hardware instrument.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.18%. Comparing base (cdbd8f2) to head (c7ae2d0).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2217      +/-   ##
==========================================
- Coverage   88.11%   87.18%   -0.94%     
==========================================
  Files          88       88              
  Lines       22832    22832              
==========================================
- Hits        20119    19905     -214     
- Misses       2713     2927     +214     

Flag	Coverage Δ
codegenunittests	84.78% <ø> (ø)
nidcpowersystemtests	94.40% <ø> (+0.04%)	⬆️
nidcpowerunittests	89.53% <ø> (ø)
nidigitalsystemtests	92.04% <ø> (ø)
nidigitalunittests	68.43% <ø> (ø)
nidmmsystemtests	92.33% <ø> (ø)
nifakeunittests	86.08% <ø> (ø)
nifgensystemtests	94.34% <ø> (ø)
nimodinstsystemtests	73.85% <ø> (ø)
nimodinstunittests	94.20% <ø> (ø)
nirfsasystemtests	82.71% <ø> (ø)
nirfsaunittests	63.40% <ø> (ø)
nirfsgsystemtests	72.66% <ø> (-8.52%)	⬇️
niscopesystemtests	89.99% <ø> (ø)
niscopeunittests	43.20% <ø> (ø)
nisesystemtests	91.50% <ø> (ø)
niswitchsystemtests	81.66% <ø> (ø)
nitclksystemtests	94.87% <ø> (ø)
nitclkunittests	98.26% <ø> (ø)

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

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

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

[divakar-selvam07]

This PR is ready for @Avinash2Suresh 's review.

[divakar-selvam07]

@jarnoldNI , the PR is ready for your review.

[ni-jfitzger]

If we're going to copy the example, should we adapt the example description, too?
"""
By varying the weighting of the samples that compose a single measurement, noise can be removed from the measurement. The DC Noise Rejection attribute controls the sample weighting. The signal frequencies removed from the measurement are related to the aperture time selected.
"""

@marcoskirsch What do you think?

[jarnoldNI]

If we're going to copy the example, should we adapt the example description, too? """ By varying the weighting of the samples that compose a single measurement, noise can be removed from the measurement. The DC Noise Rejection attribute controls the sample weighting. The signal frequencies removed from the measurement are related to the aperture time selected. """

@marcoskirsch What do you think?

@ni-jfitzger @marcoskirsch - I like the idea of adding a description like this to the examples. If we do it for this one, we should do it for all of the ones we're working on.

[ni-jfitzger]

We're setting auto_zero, way up above. Does this comment need to be split?
Or maybe we set auto_zero down here?

[ni-jfitzger]

Maybe I'm nitpicking, but I feel like this comment could also be clearer. The property that gets set is auto_zero. AutoZero is the enum and it's not like we have a feature with that spelling. The user manual page title it "Auto Zero".
https://www.ni.com/docs/en-US/bundle/ni-dmm/page/auto-zero.html

@marcoskirsch Do you have any thoughts on this?