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

Improved loop syntax, fixed code to update the part info whenever the part combo is changed #1077

Merged
merged 1 commit into from
Dec 1, 2020

Conversation

desertgreg
Copy link
Contributor

@desertgreg desertgreg commented Nov 24, 2020

Description

Updates my recent changes to ReferenceStripFeederWizard to use nicer loop syntax and fix a bug where the part count did not update correctly when the part combo box is changed.

Justification

Syntax change was suggested by mark and this code fixes an inconsistency in the UI.

Instructions for Use

ReferenceStripFeederWizard displays the number of parts used by the currently loaded job next to the part combo box.

Implementation Details

  1. Tested by interacting with some reference strip feeders and a test board. Changing the part in each feeder correctly updates the count.
  2. This code follows the coding style.
  3. There are no model changes.
  4. mvn test passes with no errors

@vonnieda vonnieda merged commit b7e9883 into openpnp:develop Dec 1, 2020
@desertgreg desertgreg deleted the StripFeederImprovements branch December 4, 2020 03:58
vonnieda added a commit that referenced this pull request Dec 23, 2020
* develop:
  Added possibility to disable ContactProbeNozzle actuator temporarily. To (#1026)
  ReferenceDragFeeder works with 0402 parts with vision enabled and dis… (#1088)
  a feeder in feeder's list is grayed out if no active placement uses it. (#1068)
  Closes #1005: Replace actuator text field with combobox (#1080)
  Feature / Global Axes and Framework for Enhanced Motion Control (#1035)
  Fix for issue 755 "ReferenceStripFeeder Breaks When Part Pitch > Hole to Hole Distance" (#1083)
  Improved loop syntax, fixed code to update the part info whenever the combo box is changed. (#1077)
  MjpgCaptureCamera was missing image transform support.  Added the missing code so the image trasnform settings now work correctly. (#1078)
  set list length of combo boxes to 20 items. (#1069)
  Implemented part count display on the ReferenceStripFeederConfigurationWizard (#1076)
  Mjpg camera feature (#1075)

# Conflicts:
#	src/main/java/org/openpnp/machine/reference/ReferenceMachine.java
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.

None yet

2 participants