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

Улицы Могилева #41

Open
wants to merge 8 commits into
base: pis
Choose a base branch
from
Open

Улицы Могилева #41

wants to merge 8 commits into from

Conversation

KopinHruk
Copy link

Пашкевич Влад

@KopinHruk KopinHruk changed the title ДОТы Линии Сталина Минский УР Линия Сталина Минский УР Feb 9, 2022
Vikort
Vikort requested changes Feb 15, 2022
View reviewed changes
Copy link
Owner

@Vikort Vikort left a comment

Choose a reason for hiding this comment

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

по указанным в понятиях названиях ничего не отображается на этой карте.

@KopinHruk
Copy link
Author

У меня есть координаты объетов, как сделать так чтобы агент инеграции osm их подхватывал?

@KopinHruk
Copy link
Author

по указанным в понятиях названиях ничего не отображается на этой карте.

Или как вариант, я могу дополнительно сделать понятия по другой теме, например Мастерские по ремонту электроники. Они будут на карте. Потому что я максимально пытался заставить агента интеграции юзать коориднаты, но в текущей реализации он это не поддерживает.

@KopinHruk KopinHruk changed the title Линия Сталина Минский УР Улицы Могилева Feb 17, 2022
@KopinHruk
Copy link
Author

Тема Улицы Могилева. Пашкевич Владислав, Тимофеенко Денис, 821701

@KopinHruk KopinHruk requested a review from Vikort February 17, 2022 15:52
@KopinHruk
Copy link
Author

KopinHruk commented Feb 22, 2022 •
edited
Loading

по указанным в понятиях названиях ничего не отображается на этой карте.

Понятия изменены, добавлен транслятор

@KopinHruk KopinHruk requested a review from SHtress March 1, 2022 04:29
Vikort
Vikort requested changes Mar 1, 2022
View reviewed changes
Copy link
Owner

@Vikort Vikort left a comment

Choose a reason for hiding this comment

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

Вторая часть принята

...in_belarus/области/moghilev_region/moghilev/streets_moghilev/agrarnaja_street_moghilev_8.scs Outdated Show resolved Hide resolved
Vlad Pashkevich added 2 commits March 2, 2022 11:56
Changed 'улица *' to '* улица'
50490b7
Merge remote-tracking branch 'origin/pis' into pis
f33b59e 
# Conflicts:
#	kb/section_subjdomain_openstreetmap/subject_domain_belarus/области/moghilev_region/moghilev/streets_moghilev/volochaevskaja_street_moghilev_238.scs
@KopinHruk KopinHruk requested a review from Vikort March 2, 2022 08:58
Vikort
Vikort reviewed Mar 7, 2022
View reviewed changes
Copy link
Owner

@Vikort Vikort left a comment

Choose a reason for hiding this comment

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

Перавя часть принята

@KopinHruk
Copy link
Author

Screenshot from 2022-03-08 23-54-23
Screenshot from 2022-03-08 23-55-56
Screenshot from 2022-03-08 23-56-16
Пример работы

@KopinHruk KopinHruk requested a review from Vikort March 8, 2022 21:05
@Vikort
Copy link
Owner

Vikort commented Mar 9, 2022

Вы сделали агента по запуску транслятора, а не интерфейс
То, что на скринах это интерфейс остиса, а не ваш

@KopinHruk
Copy link
Author

Вы сделали агента по запуску транслятора, а не интерфейс То, что на скринах это интерфейс остиса, а не ваш

))) Я прошлым коммитом как раз только и агента залил, не ожидал что ревью будет так быстро. Вот последний коммит залил с интерфейсом, мы сделали и агента запуска и интерфейс. Работали над интерфейсом: Тимофеенко Денис, Пашкевич Владислав, Илья Самойлов.

Vikort
Vikort approved these changes Mar 9, 2022
View reviewed changes
Copy link
Owner

@Vikort Vikort left a comment

Choose a reason for hiding this comment

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

Все принято

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

@Vikort Vikort Vikort approved these changes

@SHtress SHtress SHtress 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.
3 participants
@KopinHruk @Vikort @SHtress