acmerobotics/ftc-dashboard

documentation for teams migrating from 0.4.2

Closed this issue · 8 comments

If you used dashboard 0.4.2 or earlier, there were numerous places you had to sprinkle integration code into ftcrobotcontroller.java.

That seems to have been done away with by using annotation hooks, but it took a bit of work to figure that out and remove those bits. Please add some documentation to help previous users migrate.

But we haven't been fully successful. When we go to the dash page we are getting the response:

Mime type unknown: uri='/dash' path='dash'

In case it matters, this is for an OpenRC project

In case it matters, this is for an OpenRC project

Check that your OpenRC distribution includes the RC web server.

All OpenRC variations should include the webserver, beginning with OpenRC 5.5B.

Oh right, the web server is obviously running to display that message. I still suspect some OpenRC-specific issue though... I'll see if I can investigate in the next few days.

Further documentation refinement since OpenRCTurbo doesn't include a build.dependencies.gradle file:

In build.gradle at the project root add to the end of allprojects { repositories {
maven { url = 'https://maven.brott.dev/' }

add to the end of dependencies in build.gradle for both FTCRobotController and TeamCode:
implementation('com.acmerobotics.dashboard:dashboard:0.4.3') {
exclude group: 'org.firstinspires.ftc'
}

If you can provide any advice on how to debug the hooks mechanism, that would be helpful. But don't put further effort into that. I went ahead and downloaded pristine copies of the official ftcrobotcontroller v7 as well as the current OpenRCTurbo and configured the current ftc-dashboard on both. It worked in both cases. So there must be something we need to correct in our codebase's migration for all the updates.

I would make sure your merged FtcRobotControllerActivity includes all of the hook calls (e.g., this menu one). If you haven't otherwise modified FtcRobotControllerActivity, I would copy over the latest version.

Thanks for the help, there were merge errors in update to 7.0. It's working now.