Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Decadac #869

Merged
merged 3 commits into from
Nov 14, 2017
Merged

Decadac #869

merged 3 commits into from
Nov 14, 2017

Conversation

ThorvaldLarsen
Copy link
Contributor

  • Changed number of slots to 5 (slot 6 doesn't exist).
  • Changed voltage label to refer to channel number rather than 'Voltage'.

- Changed number of slots to 5 (slot 6 doesnt exist).
- Changed voltage label to refer to channel name.
Copy link
Contributor

@WilliamHPNielsen WilliamHPNielsen left a comment

Choose a reason for hiding this comment

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

If what you say is true (that the number of slots is indeed 5), then this is A-OK.

@Dominik-Vogel
Copy link
Contributor

Great you could make this PR so quickly! Looks perfect to me :-)

@WilliamHPNielsen WilliamHPNielsen merged commit 27567bf into microsoft:master Nov 14, 2017
giulioungaretti pushed a commit that referenced this pull request Nov 14, 2017
Author: ThorvaldLarsen <[email protected]>

    Slot number and label (#869)
giulioungaretti pushed a commit that referenced this pull request Nov 14, 2017
Author: ThorvaldLarsen <[email protected]>

    Slot number and label (#869)
@ThorvaldLarsen ThorvaldLarsen deleted the Decadac branch June 8, 2021 18:34
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.

3 participants