Skip to content

multimodal: Data sync between vision and ts#1930

Draft
sathyendranv wants to merge 2 commits intomainfrom
feature/svellais/data_sync
Draft

multimodal: Data sync between vision and ts#1930
sathyendranv wants to merge 2 commits intomainfrom
feature/svellais/data_sync

Conversation

@sathyendranv
Copy link
Contributor

Description

Please include a summary of the changes and the related issue. List any dependencies that are required for this change.

Fixes # (issue)

Any Newly Introduced Dependencies

Please describe any newly introduced 3rd party dependencies in this change. List their name, license information and how they are used in the project.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes.
  • I have not introduced any 3rd party components incompatible with APACHE-2.0.
  • I have not included any company confidential information, trade secret, password or security token.
  • I have performed a self-review of my code.

Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
LOG_LEVEL: ${LOG_LEVEL}
volumes:
- ./weld-data-simulator/simulation-data:/simulation-data
- ./weld-data-simulator/publisher.py:/publisher.py:ro
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's remove this, you may have added primarily for debug purpose this.

Comment on lines +398 to +402
# dlsps_proc = start_dlsps_pipeline_server(dlsps_host, dlsps_port)

# if dlsps_proc is None:
# logger.error("DLStreamPipelineServer failed to start. Exiting.")
# exit(1)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this commented code and the function def, also line 376?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants