New architecture developed #31
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…vices.bash, and seperate configs
…i.Dockerfile, as well as x86-openvino.Dockerfile - IP address of api server [processor] in config-frontend.ini should be carefully resolved. IP and port should be reachable from where the dashboard is runnning - e.g. if you are forwarding local ports on your server, proper ports also should be forwarded for the api, as it currently looks for 0.0.0.0:8003 from the user's browser ... I also changed a few other things including yarn's timeout and som dockerfile modifications to avoid multiple copies of files related to frontend, it was messing up the dockers if one did not want to build all related dockers [I added scripts for building and running, a good idea is to add them to a docker compose or sth.
…d problems using openvino so moved the _serve out of thread. Let's see when this will cause problems. Added some logic, merged back all the files to the same dockerfile, and made this huge commit. See the diff. It is now working with openvino's dockerfile, and you have to manually send get requests to api to make it work.
|
This PR had some unnecessary changes, let me fix it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi,
I have designed the new architecture in which processor's concept is separated into an API and a CORE, both running in a container which actually represents the "processor"'s concept (now runs only on x86 using OpenVino, other platforms should be added). Dashboard is separated based on branch of separate_components.
Please see how the code works, and let me know where the architecture may improve.
Next, I am going to write support for all other platforms, and then I will try to merge #17 into my branch.
Also thanks @emma-w-dev , @mats-claassen and @mhejrati for their help.
Maybe it is a good idea to make a branch for the new-architecture development and remove/merge extra branches into it so we can test all the changes afterwards.