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.

Sign up for GitHub

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

Jump to bottom

Ru mtl #566

Merged
merged 111 commits into from
Oct 10, 2023
Merged

Ru mtl #566

merged 111 commits into from
Oct 10, 2023

Conversation

dimakarp1996
Copy link
Contributor

@dimakarp1996 dimakarp1996 commented Sep 5, 2023 •
edited
Loading

Russian MTL.

AndriiHura and others added 30 commits January 24, 2022 11:22
@AndriiHura
Fix requirements.txt (#84)
72842e3
@mtalimanchuk
fix itsdangerous requirements
a67e15c
@mtalimanchuk
pin itsdangerous requirements for all flask==1.1.1 servers
0f8ef0e
@mtalimanchuk
Merge pull request #102 from deepmipt/fix/combined-classification-fla…
6f0684a 
…sk-server
@dilyararimovna
Merge pull request #103 from deepmipt/dev
d237711 
Release v0.1.0
@dilyararimovna
Merge pull request #107 from deepmipt/dev
e990264 
Release v0.1.1
@dilyararimovna
Merge pull request #119 from deepmipt/dev
3208f71 
Release v0.1.2
@dilyararimovna
Merge pull request #123 from deepmipt/dev
ab44553 
Release v0.1.3
@dilyararimovna
Merge pull request #137 from deepmipt/dev
1c9a463 
Release v0.1.4
@dilyararimovna
Merge pull request #145 from deepmipt/dev
f8e4a59 
Release v0.1.4
@dilyararimovna
Merge pull request #150 from deepmipt/dev
48872a6 
Release v0.1.5
@dilyararimovna
Merge pull request #153 from deepmipt/dev
ed42f0c 
Release v0.1.6
@dilyararimovna
Merge pull request #155 from deepmipt/dev
30f290c 
Release v0.1.7
@dilyararimovna
Merge pull request #158 from deepmipt/dev
de510bc 
Release v0.1.8
@dilyararimovna
Merge pull request #165 from deepmipt/dev
ab2dcbd 
Release v0.1.9
@dilyararimovna
Merge pull request #174 from deepmipt/dev
525783a 
Release v0.1.10
@dilyararimovna
Merge pull request #177 from deepmipt/dev
7e87a36 
Release v0.2.0
@dilyararimovna
Merge pull request #188 from deepmipt/dev
82a19df 
Release v0.2.1
@dilyararimovna
Merge pull request #192 from deepmipt/dev
2b50d6d 
Release v0.2.2
@dilyararimovna
Merge pull request #14 from deeppavlovteam/dev
3e3dd71 
Release v0.3.0
@dilyararimovna
Merge pull request #201 from deeppavlov/dev
d42e1d6 
Release v0.3.1
@dilyararimovna
Merge pull request #206 from deeppavlov/dev
4d5cb36 
Release 0.4.0
@dilyararimovna
Merge pull request #216 from deeppavlov/dev
5c36b99 
Release v0.4.1
@dilyararimovna
Merge pull request #247 from deeppavlov/dev
2961057 
Release v0.5.0
@dilyararimovna
Merge pull request #248 from deeppavlov/dev
c8632e9 
Release v0.6.0
@dilyararimovna
Merge pull request #251 from deeppavlov/dev
e4df912 
Release v0.6.1
@dilyararimovna
Merge pull request #258 from deeppavlov/dev
c8dc1fa 
Release v0.6.2
@dilyararimovna
Merge pull request #259 from deeppavlov/dev
6a85fe4 
Release v0.7.0
@dilyararimovna
Merge pull request #267 from deeppavlov/dev
92b29eb 
Release v0.8.0
@dilyararimovna
Merge pull request #272 from deeppavlov/dev
d74eca8 
Release v0.9.0
dilyararimovna
dilyararimovna requested changes Sep 12, 2023
View reviewed changes
assistant_dists/dream_russian/pipeline_conf.json Outdated
"service": "annotators/toxic_classification_ru/service_configs/toxic-classification-ru"
"directory": "annotators/combined_classification_ru",
"container": "combined-classification-ru",
"endpoint": "model"
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.

ok

annotators/combined_classification_ru/service_configs/combined-classification-ru/service.yml Outdated
SERVICE_PORT: 8198
SERVICE_NAME: combined_classification_ru
CONFIG: combined_classifier.json
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.

можно убрать отсюда

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

...tators/combined_classification_ru/service_configs/combined-classification-ru/environment.yml Outdated
@@ -0,0 +1,4 @@
SERVICE_PORT: 8198
SERVICE_NAME: combined_classification_ru
CONFIG: combined_classifier.json
Copy link
Collaborator

Choose a reason for hiding this comment

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

_ru

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

annotators/combined_classification_ru/service_configs/combined-classification-ru/service.yml Outdated
args:
SERVICE_PORT: 8198
SERVICE_NAME: combined_classification_ru
CONFIG: combined_classifier.json
Copy link
Collaborator

Choose a reason for hiding this comment

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

_ru

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

dilyararimovna
dilyararimovna requested changes Sep 24, 2023
View reviewed changes
assistant_dists/dream_russian/pipeline_conf.json Outdated
"service": "annotators/toxic_classification_ru/service_configs/toxic-classification-ru"
"directory": "annotators/combined_classification_ru",
"container": "combined-classification-ru",
"endpoint": "batch_model"
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

@dimakarp1996 dimakarp1996 Sep 24, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

Принято, поправил

dilyararimovna
dilyararimovna requested changes Oct 1, 2023
View reviewed changes
components/5eb485871be3.yml Outdated
previous_services:
- annotators.spelling_preprocessing
required_previous_services: null
state_manager_method: add_annotation
Copy link
Collaborator

Choose a reason for hiding this comment

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

если это карточка для аннотатора гипотез (в соответствии с пайпплайн конфом), то здесь должно быть по-другому

dilyararimovna
dilyararimovna requested changes Oct 3, 2023
View reviewed changes
assistant_dists/dream_russian/pipeline_conf.json Show resolved Hide resolved
assistant_dists/dream_russian/pipeline_conf.json Show resolved Hide resolved
assistant_dists/dream_russian/pipeline_conf.json Outdated
@@ -161,7 +161,7 @@
"timeout": 1.0,
"url": "http://intent-catcher-ru:8014/detect"
},
"dialog_formatter": "state_formatters.dp_formatters:last_utt_sentseg_segments_dialog",
"dialog_formatter": "state_formatters.dp_formatters:preproc_last_human_utt_dialog_w_hist",
Copy link
Collaborator

Choose a reason for hiding this comment

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

а тут почему поменял? тут не должно меняться

components/5eb485871be3.yml Outdated
connector:
protocol: http
timeout: 2.0
url: http://combined-classification-ru:8087/batch_model
Copy link
Collaborator

Choose a reason for hiding this comment

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

неправильный порт

dilyararimovna
dilyararimovna requested changes Oct 10, 2023
View reviewed changes
components/5eb485871be3.yml Outdated
required_previous_services: null
state_manager_method: add_hypothesis_annotation_batch
tags: null
endpoint: model
Copy link
Collaborator

Choose a reason for hiding this comment

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

batch_model

@dilyararimovna dilyararimovna merged commit f589120 into dev Oct 10, 2023
2 checks passed
@dilyararimovna dilyararimovna deleted the ru_mtl2 branch October 10, 2023 12:32
oserikov pushed a commit that referenced this pull request Nov 21, 2024
@dimakarp1996 @AndriiHura
@mtalimanchuk @dilyararimovna
Ru mtl (#566)
5979493 
* Fix requirements.txt (#84)

* fix itsdangerous requirements

* pin itsdangerous requirements for all flask==1.1.1 servers

* Theoretically working version

* f

* j

* j

* h

* Avoid circular imports

* Update combined_classifier_ru.json

* Removed circular imports

* Removed circular imports

* Test skill passed

* j

* Moved join_functions to separate file to avoid circular import

* Moved join_functions to separate file to avoid circular import

* passing codestyle

* h

* h

* gg

* gg

* gg

* gg

* codestyle

* Not to wait russian toxic_classification as it was replaced

* j

* Tests must pass now

* Update service.yml

* Update environment.yml

* Create docker_compose_id.yml

* Rename docker_compose_id.yml to 5eb485871be3.yml

* Update pipeline_conf.json

* Update pipeline_conf.json

* Create 3dcc0944c3e0.yml

* Update pipeline_conf.json

* Update 5eb485871be3.yml

* Update pipeline_conf.json

* Update 3dcc0944c3e0.yml

* Update 5eb485871be3.yml

* Update pipeline_conf.json

* was erroneous edit

* Update 5eb485871be3.yml

---------

Co-authored-by: Andrii.Hura <54397922+AndriiHura@users.noreply.github.com>
Co-authored-by: mtalimanchuk <mtalimanchuk@gmail.com>
Co-authored-by: Dilyara Baymurzina <dilyara.rimovna@gmail.com>
Co-authored-by: Dmitry <dimakarp1996@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dilyararimovna dilyararimovna dilyararimovna approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dimakarp1996 @dilyararimovna @AndriiHura @mtalimanchuk