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

property extraction annotator #202

Merged
merged 50 commits into from
Feb 21, 2023
Merged

property extraction annotator #202

merged 50 commits into from
Feb 21, 2023

Conversation

dmitrijeuseew
Copy link
Contributor

Annotator for extraction of user attributes triplet from utterances.

- "./annotators/property_extraction:/src"
- "~/.deeppavlov:/root/.deeppavlov"
ports:
- 8126:8126
Copy link
Collaborator

Choose a reason for hiding this comment

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

до 8128 включительно заняты порты.
Бери 8129 - еще свободен

@@ -19,7 +19,8 @@ services:
dff-funfact-skill:8104, dff-bot-persona-skill:8105, news-api-annotator:8112,
dff-gossip-skill:8109, dff-wiki-skill:8111, dff-gaming-skill:8115, topic-recommendation:8113,
user-persona-extractor:8114, wiki-facts:8116, dff-music-skill:8099, entity-detection:8103, dff-art-skill:8117,
midas-predictor:8121, dialogpt:8125, storygpt:8126, prompt-storygpt:8127, infilling:8122, dff-template-skill:8120"
midas-predictor:8121, dialogpt:8125, storygpt:8126, prompt-storygpt:8127, infilling:8122,
property-extraction:8126, dff-template-skill:8120"
Copy link
Collaborator

Choose a reason for hiding this comment

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

до 8128 включительно заняты порты.
Бери 8129 - еще свободен

command: flask run -h 0.0.0.0 -p 8126
environment:
- FLASK_APP=server
- CUDA_VISIBLE_DEVICES=7
Copy link
Collaborator

Choose a reason for hiding this comment

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

здесь 0. Не нулевой номер девайса - в гпу1 и тест ямл

Copy link
Collaborator

@dilyararimovna dilyararimovna left a comment

Choose a reason for hiding this comment

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

Надо добавить в test.yml и прочие и добавить в список тестируемых компонент в run_tests

@@ -0,0 +1,20 @@
FROM deeppavlov/base-gpu
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
Contributor Author

Choose a reason for hiding this comment

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

Зафиксировал

assistant_dists/dream/test.yml Show resolved Hide resolved
ARG CONFIG
ARG PORT
ARG SRC_DIR
ARG SED_ARG=" | "
Copy link
Collaborator

Choose a reason for hiding this comment

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

неиспользуемые

assistant_dists/dream/test.yml Show resolved Hide resolved
command: flask run -h 0.0.0.0 -p 8129
environment:
- FLASK_APP=server
- CUDA_VISIBLE_DEVICES=0
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
| Spacy Annotator | 250 MB RAM | token-wise annotations by Spacy |
black --line-length=120 --check annotators/entity_detection - проверка

black --line-length=120 annotators/entity_detection - форматирование| Spacy Annotator | 250 MB RAM | token-wise annotations by Spacy |
Copy link
Collaborator

Choose a reason for hiding this comment

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

артефакты случайной вставки эта и две предыдущие строки

@@ -1084,7 +1085,6 @@ services:
context: ./
dockerfile: annotators/entity_detection/Dockerfile
command: flask run -h 0.0.0.0 -p 8103
environment:
Copy link
Collaborator

Choose a reason for hiding this comment

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

строчку удалил случайно

env_file: [.env]
build:
args:
CONFIG: t5_generative_ie_lite_infer.json
Copy link
Collaborator

Choose a reason for hiding this comment

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

а ему карта нужна? t5 вроде написано

SRC_DIR: annotators/property_extraction/
context: ./
dockerfile: annotators/property_extraction/Dockerfile
command: flask run -h 0.0.0.0 -p 8129
Copy link
Collaborator

Choose a reason for hiding this comment

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

сорри, порт заняли уже, возьми другой, пожалуйста.
Например 8136

@dilyararimovna dilyararimovna merged commit 9e64c6c into dev Feb 21, 2023
@dilyararimovna dilyararimovna deleted the feat/property_extraction branch February 21, 2023 05:35
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