Go to Post Give me enough gearboxes and I'll move the world. - SenorZ [more]
Home
Go Back   Chief Delphi > Technical > Programming
CD-Media   CD-Spy  
portal register members calendar search Today's Posts Mark Forums Read FAQ rules

 
Reply
 
Thread Tools Rate Thread Display Modes
  #1   Spotlight this post!  
Unread 01-02-2017, 08:50
mikearnold514 mikearnold514 is offline
Registered User
FRC #0514
 
Join Date: Jan 2017
Location: Miller Place, NY
Posts: 4
mikearnold514 is an unknown quantity at this point
Re: No robot code

@Austin
We got around the error we were having with the NT bind() as originally reported here. I updated the GitHub ticket #465 with our code and findings. In short, for anyone looking here, we moved the CameraServer.getInstance().startAutomaticCapture() out of the class constructor of our subsystem and into a regular public method in our subsystem then called that method from RobotInit() like this: Robot.visionUtil.startImageProcessing(); Obviously, folks will use whatever naming is suitable for them. Also, not everyone is using the CommandBasedRobot framework so it might be different for those folks but this is the general idea.

I hope folks find this information helpful and I appreciate the support you have given. You will see in the ticket that I did ask you to review the code as we are not getting any meaningful data from the Pipeline but we can move that to a different topic thread if you think that is best.

Thanks again Austin and kindest regards,
Mike Arnold, Team 514.
Reply With Quote
  #2   Spotlight this post!  
Unread 01-02-2017, 18:16
cnc4 cnc4 is online now
Registered User
FRC #2231 (OnyxTronix)
Team Role: Programmer
 
Join Date: Jul 2016
Rookie Year: 2015
Location: Israel
Posts: 21
cnc4 is an unknown quantity at this point
Re: No robot code

Quote:
Originally Posted by mikearnold514 View Post
@Austin
We got around the error we were having with the NT bind() as originally reported here. I updated the GitHub ticket #465 with our code and findings. In short, for anyone looking here, we moved the CameraServer.getInstance().startAutomaticCapture() out of the class constructor of our subsystem and into a regular public method in our subsystem then called that method from RobotInit() like this: Robot.visionUtil.startImageProcessing(); Obviously, folks will use whatever naming is suitable for them. Also, not everyone is using the CommandBasedRobot framework so it might be different for those folks but this is the general idea.

I hope folks find this information helpful and I appreciate the support you have given. You will see in the ticket that I did ask you to review the code as we are not getting any meaningful data from the Pipeline but we can move that to a different topic thread if you think that is best.

Thanks again Austin and kindest regards,
Mike Arnold, Team 514.
https://github.com/wpilibsuite/allwp...ment-276674779

You wanted a feedback on your code:
My team and me made a visionSensor that implements PIDSource, this sensor is not an subsystem and it return a pidValue in pidGet.
So you can use it inside a PIDController.
Make a pidVisionSourceType to choose what will get in pidValue.
Hope this helps you .
Reply With Quote
  #3   Spotlight this post!  
Unread 02-02-2017, 09:46
mikearnold514 mikearnold514 is offline
Registered User
FRC #0514
 
Join Date: Jan 2017
Location: Miller Place, NY
Posts: 4
mikearnold514 is an unknown quantity at this point
Re: No robot code

Thanks for the feedback. We were also considering PID Control and would likely try a PID Subsystem. We will definitely consider your approach. We happen to be stuck on trying to get meaningful data from our VisionPipeline and it is likely something silly we are doing that is holding us up now.

As for the original error, I am concluding that by your reply you are past this issue. If that is the case then I would recommend we tell Austin and the WPILib development team that they can close issue #465, if it has not been closed already. But since you opened the issue originally, I will leave that action to you.

Kind Regards,
Mike
Reply With Quote
  #4   Spotlight this post!  
Unread 02-02-2017, 19:04
cnc4 cnc4 is online now
Registered User
FRC #2231 (OnyxTronix)
Team Role: Programmer
 
Join Date: Jul 2016
Rookie Year: 2015
Location: Israel
Posts: 21
cnc4 is an unknown quantity at this point
Re: No robot code

Quote:
Originally Posted by mikearnold514 View Post
As for the original error, I am concluding that by your reply you are past this issue. If that is the case then I would recommend we tell Austin and the WPILib development team that they can close issue #465, if it has not been closed already. But since you opened the issue originally, I will leave that action to you.

Kind Regards,
Mike
No, please see my last commant on the issue.

Any way good luck
Reply With Quote
Reply


Thread Tools
Display Modes Rate This Thread
Rate This Thread:

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Forum Jump


All times are GMT -5. The time now is 15:27.

The Chief Delphi Forums are sponsored by Innovation First International, Inc.


Powered by vBulletin® Version 3.6.4
Copyright ©2000 - 2017, Jelsoft Enterprises Ltd.
Copyright © Chief Delphi