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

Add speed monitor #64

Merged
merged 3 commits into from
Nov 25, 2020
Merged

Conversation

ianmaciel
Copy link
Contributor

@ianmaciel ianmaciel commented Nov 23, 2020

Add feature: speed monitor/sign.

It also included a container to make easier to install the dependencies. The container will work on Linux (under Windows or Mac it won't be able to connect to USB devices - still possible to run using RTSP) and does not support GUI yet.

@ianmaciel ianmaciel mentioned this pull request Nov 23, 2020
@ianmaciel ianmaciel force-pushed the feature/add-speed-monitor branch from c6b8c0c to c1f35d7 Compare November 23, 2020 00:07
@pageauc pageauc merged commit 99640df into pageauc:master Nov 25, 2020
@pageauc
Copy link
Owner

pageauc commented Nov 25, 2020

Can you supply instructions for implementing container or supply a script to do it. Currently, I am not familiar with vscode container implementation and usage. I would like to add wiki to explain usage and implementation details
Thanks Claude ...

@ianmaciel
Copy link
Contributor Author

ianmaciel commented Nov 26, 2020

Can you supply instructions for implementing container or supply a script to do it. Currently, I am not familiar with vscode container implementation and usage. I would like to add wiki to explain usage and implementation details
Thanks Claude ...

Yes, sure... I'll write this and update here.
(Actually I was going to do that but I forgot)

@ianmaciel ianmaciel deleted the feature/add-speed-monitor branch November 26, 2020 18:41
@ianmaciel
Copy link
Contributor Author

ianmaciel commented Dec 7, 2020

I'm sorry about my delay. I couldn't write anything.

In the meantime while I don't write, I would like to introduce devcontainer to you. So I recorded this quick video:

ezgif-3-92094a35f158

After these steps, you will have all required dependencies (python, opencv), and it is just start coding. the only two prior dependency needed are: Docker and VS Code

@ianmaciel
Copy link
Contributor Author

Hi @pageauc, sorry about my delay.

I tried to update the wiki but looks like I cant: I thought I would be able to fork, update the wiki and then create a pull request, but this is not available on GitHub.

So I will right here and you can copy/edit if you want to. 😃

@ianmaciel
Copy link
Contributor Author

ianmaciel commented Jan 28, 2021

Suggested topic about devcontaier.

## Using Visual Studio devcontainer

Using VSCode IDE it is possible to run the project without all the setup needed. All you need to run is [VSCode](https://code.visualstudio.com/) and [Docker](https://www.docker.com/) installed on your machine.

VSCode will create a docker container with all dependencies needed (specified on .devcontainer folder). With this environment you it is only possible to connect to USB devices on Linux. If you are running on Windows or Mac you still can use this environment with RTSP protocol. 

Detailed information are available on https://code.visualstudio.com/docs/remote/containers

### Getting Started

- Install VSCode (https://code.visualstudio.com/download)
- Install Docker (https://docker.com), If you are using Windows you will also need WSL2 (https://docs.microsoft.com/pt-br/windows/wsl/install-win10)
- Open the project root with Visual Studio. Visual Studio will find the "devcontainer" folder and suggest to open it in Docker environment.

See the animation below:

image

@pageauc
Copy link
Owner

pageauc commented Jan 29, 2021 via email

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

Successfully merging this pull request may close these issues.

2 participants