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

Дублирует участки #8

Closed
denny123 opened this issue Sep 10, 2018 · 12 comments
Closed

Дублирует участки #8

denny123 opened this issue Sep 10, 2018 · 12 comments

Comments

@denny123
Copy link

При запросе например номера 60:18:0060201:3306, png как надо, а вот координаты участков в geojson дублирует и в результате получаются дырки вместо участков. Также хотел узнать про точность(вроде параметр -e), как им пользоваться(на данном участке также не все точки видит)?

@denny123
Copy link
Author

export.py

       for fry in range(len(coords)):
            for j in range(len(coords[fry])):
                xy = coords[fry][j]
                xy.append(xy[0])
            close_xy.append(xy)    <------------------------подвинул тут влево
        multi_polygon.append(close_xy)   <-----------------------и тут влево
        feature = {"type": "Feature",

вроде как помогло, требуется визирование автора)

@rendrom
Copy link
Owner

rendrom commented Sep 10, 2018

@denny123 хорошо что заметили, спасибо. Предлагаю отправить Pull Request с изменениями

@denny123
Copy link
Author

Я ничем не против, знать бы что это такое Pull Request(гитхаб не юзаю).

@rendrom
Copy link
Owner

rendrom commented Sep 10, 2018

Там вверху есть кнопка Fork, как нажмёшь, в твоём профиле будет копия репозитория,

  • клонируешь её к себе на компьютер git clone [название репозитория];
  • вносишь изменения;
  • сохраняешь git commit -a -m 'Дублирует участки #8 message' (Дублирует участки #8 - номер этого бага, message - название коммита, короткий список изменений не более 80 символов)
  • заливаешь к себе в форк git push origin master
  • в интерфесе github нажимаешь Create Pull Request на странице своего форка этого репозитория.

Это всё для того чтобы сохранить авторстов твоих изменений в проекте. Если не хочется этим заниматься, могу я поправить и накатить

@denny123
Copy link
Author

не не не, не надо мне авторства, правь сам, даю согласие и все права)

@rendrom
Copy link
Owner

rendrom commented Sep 11, 2018

параметр -e увеличивает точность распознавания угловых точек с изображения участка.

https://docs.opencv.org/2.4/modules/imgproc/doc/structural_analysis_and_shape_descriptors.html
epsilon – Parameter specifying the approximation accuracy. This is the maximum distance between the original curve and its approximation.

Для увелечения точности определения координат нет специального параметра, но в коде можно поменять значения параметра tile_size
https://github.com/rendrom/rosreestr2coord/blob/master/scripts/merge_tiles.py#L315

@denny123
Copy link
Author

менял tile_size - ничего не изменилось, как будто он берет из другого места...

@rendrom
Copy link
Owner

rendrom commented Sep 11, 2018

Проверю. Это только для больших участков работает. Но точность все равно будет приблизительной при любых настройках.

@denny123
Copy link
Author

я все на этом 60:18:0060201:3306 экспериментирую, там верхний участок и у него верхний правый угол с изгибом должен быть.....я там кстати баг создал уже по пакетной аппроксимации...

@denny123
Copy link
Author

на счет точности, лучше просто сделать тайлы для заданных приближений, будет точность как в росреестре)

@denny123
Copy link
Author

denny123 commented Sep 12, 2018

export.py

   for fry in range(len(coords)):
        close_xy = [] <----------------------------------- добавить строку
        for j in range(len(coords[fry])):
            xy = coords[fry][j]
            xy.append(xy[0])
            close_xy.append(xy)    <------------------------ оставить как было
        multi_polygon.append(close_xy)   <----------------------- оставить как было
    feature = {"type": "Feature",

по крайней мере пока работает и для дыр и для нескольких участков под одним номером...
Просьба автору поправить Журнал и скрипты, мой косяк - плохо проверил.

@rendrom
Copy link
Owner

rendrom commented Sep 12, 2018

Спасибо, обязательно поправлю как доберусь до компьютера

rendrom added a commit that referenced this issue Sep 27, 2018
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

No branches or pull requests

2 participants