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

Updated installation.md with respect to information about CVAT_HOST for local installation #4014

Merged
merged 4 commits into from
Dec 10, 2021

Conversation

anugrahasinha
Copy link
Contributor

@anugrahasinha anugrahasinha commented Dec 10, 2021

Resolves #4013

Motivation and context

The installation guide should mention about exporting CVAT_HOST variable so that CVAT can be access over the network. The current installation method only supports accessing CVAT using http://localhost:8080. This information about exporting CVAT_HOST variable can be derived from AWS-Deployment Guide
source here
actual here

How has this been tested?

This is document level change, therefore the markdown has been checked for sanity.

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

Added information about *exporting* `CVAT_HOST` variable so that accessing CVAT portal can be enabled over local network.
Updated changes for exporting `CVAT_HOST` variable to be in passive voice (language/grammar) related change.
Added changelog specific information related to cvat-ai#4014
@nmanovic
Copy link
Contributor

@anugrahasinha , thanks for your contribution! Great job. I will merge the PR after all linter checks are passed.

CHANGELOG.md Outdated Show resolved Hide resolved
@nmanovic nmanovic merged commit 8cdd5d6 into cvat-ai:develop Dec 10, 2021
@anugrahasinha
Copy link
Contributor Author

Thanks for merging the PR. 👍

@nmanovic nmanovic mentioned this pull request Mar 4, 2022
7 tasks
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.

Update installation guide for exporting CVAT_HOST env variable before docker compose command.
3 participants