-
Notifications
You must be signed in to change notification settings - Fork 37
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
Update v2 #19
Update v2 #19
Conversation
Former-commit-id: e43ed6987551bce997ce1b03fe80c0d9c52f1d40
Former-commit-id: 52cb00d34eb708b9acb01831d0be0c0190bdfde5
Former-commit-id: 92cda3b7c976cf87f898cd33ae25be6ba52887c9
Former-commit-id: 72bc96c5811125e038afebea7c39fc66a518fd06
Former-commit-id: 81bf06e9296415a8e47a69844cb3a9f965a62f41
Former-commit-id: 6fd5abfdbfc05b46b98fd3c441c980265178c020
Former-commit-id: 2980460f292d8f1e515583ffda69d1eeeb4520e0
Former-commit-id: 8e030db2cc926f5a5bc2b0186b78e7f309238fbc
Former-commit-id: f1dd4b5bcdc6f5f63db4ff5fa3e16e0f0799b35c
Former-commit-id: e00f44353c011245ac2e2fc2c98cf0fa91f5436a
Former-commit-id: 9901dfe83a99d37f110efc35673a23082831791d
Model train
@mchadwick-iqt, does this look right to you? Of course, this operation is not as simple as I would like it to be. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jspeed-meyers this looks correct to me!
@jspeed-meyers - I pushed a new commit to remove a number of obsolete files that had been retained with the merge. Can you do a quick once-over and make sure we're still good from your perspective? |
@mchadwick-iqt, It looks good to me. I am going to look at it one more time with fresh eyes tomorrow morning and then I'll merge. |
@@ -4,5 +4,5 @@ services: | |||
axis-ptz: | |||
command: /bin/sh -c "while sleep 1000; do :; done" | |||
volumes: | |||
- /home/pi/Projects/SkyScan/axis-ptz:/workspace:cached # Additional bind mount | |||
- /home/pi/Projects/SkyScan-Private/axis-ptz:/workspace:cached # Additional bind mount |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rcaudill, is this use of a different volume path a problem?
Web status Former-commit-id: dbf96fb6813361d883c76a7fe11203503e92556c
@rob, this is an attempt to merge back in the progress from the private fork into the public fork. Does it look right to you? Can you please take a look and let me know?
JS