Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

Mechanisms return Status from run() #149

Open
wants to merge 1 commit into
base: rcahoon/mf3-statuses
Choose a base branch
from

Conversation

rcahoon
Copy link
Member

@rcahoon rcahoon commented Dec 13, 2024

Description

Encourages the user into the code patterns that we want to use for data flow

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Be detailed so that your code reviewer can understand exactly how much and what kinds of testing were done, and which might still be worthwhile to do.

  • Unit tests: Units tests for new functionality are included in this PR
  • Simulator testing: [Add your description here]
  • On-robot bench testing: [Add your description here]
  • On-robot field testing: [Add your description here]

@rcahoon rcahoon requested a review from dejabot December 13, 2024 08:17
@rcahoon rcahoon force-pushed the rcahoon/mf3-mechanism-status branch 2 times, most recently from 49d255b to c47c20e Compare December 13, 2024 10:10
throw new NoSuchElementException(getName() + " has not published a status yet");
}
return status;
}
Copy link
Member Author

Choose a reason for hiding this comment

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

Remove this. Just have one way of doing things

@rcahoon rcahoon force-pushed the rcahoon/mf3-statuses branch from 3abc687 to 2069112 Compare December 14, 2024 21:24
@rcahoon rcahoon force-pushed the rcahoon/mf3-mechanism-status branch from c47c20e to d4029e5 Compare December 14, 2024 21:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants