-
Notifications
You must be signed in to change notification settings - Fork 19
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
Using full vector from mdp travel service #156
Conversation
Needs merging soon. |
Has this been tested by @Jailander or anyone else at UOL? Please report findings here. |
This should address strands-project#157
@Jailander @cdondrup I know you're testing loads ATM, so can you give this one a go, too? |
Will test it tomorrow when we have the info_terminal and walking_group running on schedule. |
…t directly does not have concurrency issues with the other expected time call.
@bfalacerda changes are in and run fine here in simulation, but I'm probably missing stuff that is important on the robot. |
When starting
|
When trying to use the top version of the scheduler, it starts:
So it updates correctly from google calendar and sends it to the scheduler. Afterwards I run
but |
I think I found where it dies. I removed the task from the calendar, started the schedule in the top mode again. Set it to execute and the it started publishing an empty schedule under
And nothing ever happened thereafter. Also |
See if you have any other instances of PRISM running and then kill them if you do. The first time I run the mdp on any system it gives me an error, but then runs next time. I have a suspicion on the other issue, but let's try to get the mdp bit working first as that's the intended system. |
this has been working in bob throughout the day. @cdondrup , the issues you posted don't seem to be directly to this PR, so it is ok if I merge it? Is the issue with launching |
I'm in the process of trying it, but there apparently was a prism process running and now it launched. Can't tell if there is any thing intelligent happening though. Have to tie up a few loose ends first. |
ok, i'll wait for some feedback then |
…tore service. This should fix strands-project#160
OK, so the top version seems to work if there is task schedule not via the google interface. at least we restarted everything and tried the top version with info_terminal and that worked. I think you can merge this, once the tests pass, and I will keep on trying the mdp version tomorrow. |
retest this please |
As said here: #155 the test is itself unstable, it seems. |
Using full vector from mdp travel service
This should speed up scheduling quite a lot since the proper edge statistics stuff was introduced.