Skip to content

Publish Current Sensor Calibration Article#117

Merged
elsevers merged 8 commits intomainfrom
user/elsevers/partial-add-control-content
Dec 8, 2024
Merged

Publish Current Sensor Calibration Article#117
elsevers merged 8 commits intomainfrom
user/elsevers/partial-add-control-content

Conversation

@elsevers
Copy link
Contributor

@elsevers elsevers commented Dec 8, 2024

This PR publishes the Current Sensor Calibration Article from #85.

This is being added now, while other articles are staged in #85.

npetersen2 and others added 8 commits June 5, 2023 17:00
* Initial commit

* Update readme

* Update readme

* Update readme

* Add files

* Update files

* Changes

* Changes

* Update files

* Get rif of scope image

* Minor updates

* Address review comments

* Remove resources

* Add back resources

* Address review comments

* Address review comments

* Update index.md

* Edit current sensor calibration method section

* Edit calibration method section

* Add section on Recalcilating current sensor offset

* Clarify that we measure a voltage.

* Edit conclusion

* Update index.md

* Update index.md

* Address review comments

* Update index.md

* Update index.md

* Update index.md

* Apply suggestions from code review

Co-authored-by: Eric Severson <eric.severson@wisc.edu>

* Update source/getting-started/control-with-amdc/current-sensor-cal/index.md

* Update source/getting-started/control-with-amdc/current-sensor-cal/index.md

---------

Co-authored-by: Nathan Petersen <npetersen2@wisc.edu>
Co-authored-by: Eric Severson <eric.severson@wisc.edu>
Copy link
Contributor

@noguchi-takahiro noguchi-takahiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @elsevers. Looks good.

@elsevers elsevers merged commit ff6b7f8 into main Dec 8, 2024
@elsevers elsevers deleted the user/elsevers/partial-add-control-content branch December 8, 2024 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants