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

Docs #6

Merged
merged 5 commits into from
Jul 8, 2021
Merged

Docs #6

merged 5 commits into from
Jul 8, 2021

Conversation

vvrubel
Copy link
Owner

@vvrubel vvrubel commented Jul 8, 2021

документация

@vvrubel vvrubel requested a review from churnikov July 8, 2021 10:03
@@ -6,7 +6,7 @@ authors = ["vvrubel <vasilyeva1510@gmail.com>"]
license = "free"

[tool.poetry.dependencies]
python = "^3.7.1"
python = "^3.7.10"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Мне казалось у нас тут 3.9

Copy link
Owner Author

Choose a reason for hiding this comment

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

Мне казалось у нас тут 3.9

а там же все пришлось менять, тк это надо mongo-rdkit

Copy link
Collaborator

Choose a reason for hiding this comment

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

Понятно
Код ты поменяла, соответственно?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Понятно
Код ты поменяла, соответственно?

ды

README.md Outdated
| Переменная | Описание | Значение по умолчанию |
|---|---|---|
|ENV|---|---|
|SERVICE_URL|URL-адрес запущенного приложения; рекомендуется устанавливать API из сети BIOCAD (например, http://10.249.233.40:8888/). Не забудьте указать порт (т.е. 8888) и протокол (http или https).|"http://localhost:8888"|
Copy link
Collaborator

Choose a reason for hiding this comment

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

А зачем это?..

Copy link
Owner Author

Choose a reason for hiding this comment

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

А зачем это?..

весь блок или только эти две переменные?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Конкретно эта переменная

Copy link
Owner Author

Choose a reason for hiding this comment

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

Конкретно эта переменная

хз, могу убрать в принципе, пока не пригодилась

Copy link
Collaborator

Choose a reason for hiding this comment

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

Оно не пригодится

README.md Outdated
Comment on lines 91 to 92
$ mongod --dbpath ~/MongoDB/data/db
Copy link
Collaborator

Choose a reason for hiding this comment

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

разные пути

Copy link
Collaborator

Choose a reason for hiding this comment

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

Как я смотрю, у тебя предполагается использования Enterprise Server.

Так то папка с данными монги может быть в любом месте, независимо от установки

Copy link
Owner Author

Choose a reason for hiding this comment

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

Как я смотрю, у тебя предполагается использования Enterprise Server.

Так то папка с данными монги может быть в любом месте, независимо от установки

ну да, тут из контекста выдернуто

```

В результате выполнения команды `fetch` будет создан JSON-файл в директории `out_dir`;
Copy link
Collaborator

Choose a reason for hiding this comment

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

out_dir должен существовать перед запуском?
Я что-то не увидел

Copy link
Owner Author

Choose a reason for hiding this comment

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

out_dir должен существовать перед запуском?
Я что-то не увидел

пропишу об этом, но вообще папки наоборот должно не быть, если она есть - он кинет ошибку, это для того чтобы различать например когда что было скачано, а то с наименованиями с помощью uuid это сложно - во избежании кучи дублей с разными названиями

@vvrubel vvrubel merged commit 6d5b8ac into master Jul 8, 2021
@vvrubel vvrubel mentioned this pull request Jul 14, 2021
@vvrubel vvrubel deleted the docs branch August 8, 2021 12:05
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.

None yet

2 participants