Adding Improve DC Noise Rejection example in NI-DMM by divakar-selvam07 · Pull Request #2217 · ni/nimi-python

divakar-selvam07 · 2026-10-08T10:55:21Z

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.

…into users/diva/add-nidmm-improve-dc-noise-rejection-example

codecov-commenter · 2026-10-08T10:59:47Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.12%. Comparing base (49c8447) to head (a793aa8).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2217      +/-   ##
==========================================
+ Coverage   86.40%   88.12%   +1.71%     
==========================================
  Files          86       88       +2     
  Lines       21563    22832    +1269     
==========================================
+ Hits        18631    20120    +1489     
+ Misses       2932     2712     -220

Flag	Coverage Δ
codegenunittests	`84.78% <ø> (ø)`
nidcpowersystemtests	`94.36% <ø> (ø)`
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	`81.22% <ø> (ø)`
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.

…dmm-improve-dc-noise-rejection-example # Conflicts: # docs/nidmm/examples.rst

divakar-selvam07 · 2026-10-09T11:17:40Z

This PR is ready for @Avinash2Suresh 's review.

divakar-selvam07 · 2026-10-09T12:16:53Z

@jarnoldNI , the PR is ready for your review.

ni-jfitzger · 2026-10-12T21:48:35Z

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 · 2026-10-14T14:57:02Z

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.

…' of https://github.com/NI-velmurugan/nimi-python into users/diva/add-nidmm-improve-dc-noise-rejection-example

…dmm-improve-dc-noise-rejection-example

ni-jfitzger · 2026-10-30T20:08:14Z

+        session.aperture_time = aperture_time
+        # NORMAL weighs all samples equally; SECOND_ORDER and HIGH_ORDER weigh samples in the middle of the aperture more.
+        session.dc_noise_rejection = nidmm.DCNoiseRejection[dc_noise_rejection]
+        # The DMM averages shorter measurements to produce each returned reading.


Isn't this only True if Auto Zero is enabled?

NI-velmurugan and others added 4 commits October 7, 2026 19:45

Add @Avinash2Suresh as a code owner

4eb8624

add: add Improve DC Noise Rejection example for nidmm

cdf7fa8

Remove @Avinash2Suresh from CODEOWNERS

c965114

Merge branch 'master' of https://github.com/NI-velmurugan/nimi-python …

9d4d325

…into users/diva/add-nidmm-improve-dc-noise-rejection-example

divakar-selvam07 added 3 commits October 8, 2026 16:45

fix: fix tests

b12451d

Trigger CI build

944eda7

Merge remote-tracking branch 'upstream/master' into users/diva/add-ni…

8173be4

…dmm-improve-dc-noise-rejection-example # Conflicts: # docs/nidmm/examples.rst

Avinash2Suresh suggested changes Oct 9, 2026

View reviewed changes

resolve PR comments

f97bec3

divakar-selvam07 requested a review from Avinash2Suresh October 9, 2026 12:09

Avinash2Suresh approved these changes Oct 9, 2026

View reviewed changes

divakar-selvam07 added 3 commits October 9, 2026 18:06

Trigger CI build

eeb4882

Trigger CI build

5b30a87

Trigger CI build

a39ffd0

jarnoldNI reviewed Oct 9, 2026

View reviewed changes

Comment thread src/nidmm/examples/nidmm_improve_dc_noise_rejection.py Resolved

add: add number of averages parameter

5c9e94f

divakar-selvam07 requested review from Avinash2Suresh and jarnoldNI October 12, 2026 06:03

Avinash2Suresh approved these changes Oct 12, 2026

View reviewed changes

jarnoldNI reviewed Oct 12, 2026

View reviewed changes

Comment thread src/nidmm/examples/nidmm_improve_dc_noise_rejection.py Resolved

doc: add self explainatory comments

ab9bcff

divakar-selvam07 requested a review from jarnoldNI October 12, 2026 15:45

jarnoldNI approved these changes Oct 12, 2026

View reviewed changes

divakar-selvam07 marked this pull request as ready for review October 12, 2026 16:46

divakar-selvam07 requested review from marcoskirsch, ni-jfitzger and vnktshr21 as code owners October 12, 2026 16:46

ni-jfitzger reviewed Oct 14, 2026

View reviewed changes

Comment thread src/nidmm/examples/nidmm_improve_dc_noise_rejection.py Resolved

ni-jfitzger reviewed Oct 14, 2026

View reviewed changes

Comment thread src/nidmm/examples/nidmm_improve_dc_noise_rejection.py Resolved

NI-velmurugan and others added 6 commits October 15, 2026 12:50

Updating the comments for aperture time

c7ae2d0

Trigger CI build

4f00eb4

Merge branch 'users/diva/add-nidmm-improve-dc-noise-rejection-example…

ec9fbc7

…' of https://github.com/NI-velmurugan/nimi-python into users/diva/add-nidmm-improve-dc-noise-rejection-example

Trigger CI build

db7c272

nit: apply suggestions

c8757ec

Merge remote-tracking branch 'upstream/master' into users/diva/add-ni…

a793aa8

…dmm-improve-dc-noise-rejection-example

divakar-selvam07 requested a review from ni-jfitzger October 26, 2026 15:48

ni-jfitzger reviewed Oct 30, 2026

View reviewed changes

Navigation Menu

Repository navigation

Adding Improve DC Noise Rejection example in NI-DMM - #2217

Adding Improve DC Noise Rejection example in NI-DMM#2217
divakar-selvam07 wants to merge 19 commits into
ni:masterfrom
NI-velmurugan:users/diva/add-nidmm-improve-dc-noise-rejection-example

divakar-selvam07 commented Oct 8, 2026

Uh oh!

codecov-commenter commented Oct 8, 2026 •

edited

Loading

Uh oh!

divakar-selvam07 commented Oct 9, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

divakar-selvam07 commented Oct 9, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

ni-jfitzger commented Oct 12, 2026

Uh oh!

jarnoldNI commented Oct 14, 2026

Uh oh!

Uh oh!

Uh oh!

ni-jfitzger Oct 30, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Conversation

divakar-selvam07 commented Oct 8, 2026

What does this Pull Request accomplish?

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

What testing has been done?

Uh oh!

codecov-commenter commented Oct 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

divakar-selvam07 commented Oct 9, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

divakar-selvam07 commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ni-jfitzger commented Oct 12, 2026

Uh oh!

jarnoldNI commented Oct 14, 2026

Uh oh!

Uh oh!

Uh oh!

ni-jfitzger Oct 30, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

codecov-commenter commented Oct 8, 2026 •

edited

Loading

divakar-selvam07 commented Oct 9, 2026 •

edited

Loading