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

Deployment Verifier #36

Closed
tthiery opened this issue Jun 30, 2020 · 2 comments
Closed

Deployment Verifier #36

tthiery opened this issue Jun 30, 2020 · 2 comments
Assignees
Milestone

Comments

@tthiery
Copy link
Member

tthiery commented Jun 30, 2020

When you build a user interface or something related, you expect the connected sensors, actors and hubs to follow a pre-defined model (which you then control by your UI).

var model = new DeploymentModelBuilder()
                 .AddHub<TechnicMediumHub>(hubBuilder => hubBuilder
                     .AddDevice<TechnicLinearMotor>(hub => hub.A)
                     .AddDevice<TechnicXlargeLinearMotor>()
                 )
                .Build();

var errors = model.Verify(protocol);

// or reactive

var errorObservable = protocol.VerifyObservable(model);

With errors being something like ..

class DeploymentError
{
    int ErrorCode;
    byte HubId;
    byte? PortId;
    string Message;
}
@tthiery tthiery added this to the v2.0 milestone Jun 30, 2020
This was referenced Jul 4, 2020
@tthiery tthiery self-assigned this Jul 4, 2020
tthiery added a commit that referenced this issue Jul 4, 2020
- Added DeploymentModel model ;)
- Added Builder for deployment model
- Added HubExtensions to enable a simple usage
- Add verification for examples

#36 non-breaking
@tthiery
Copy link
Member Author

tthiery commented Jul 4, 2020

  • Initial verification on request
  • Update README.md
  • Observable verification stream (reacting on connecting and disconnecting devices)
  • Verify Hub/System Type
  • Verify "Any" device (e.g. for selective port discovery)
  • Verify "Any" hub (e.g. for hub does not matter as long as motor is connected)
  • Documentation
  • re-work of protocol knowledge to include hubId
  • Multi-Hub Verification

tthiery added a commit that referenced this issue Jul 5, 2020
tthiery added a commit that referenced this issue Jul 5, 2020
- Non Breaking change (outside of SharpBrick.PoweredUp namespace)

#36 (non-breaking)
tthiery added a commit that referenced this issue Jul 5, 2020
tthiery added a commit that referenced this issue Jul 5, 2020
tthiery added a commit that referenced this issue Jul 5, 2020
@tthiery
Copy link
Member Author

tthiery commented Jul 5, 2020

  • Multi Hub Validation is a non-goal since no hardware available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant