Skip to content

Point set diameter - #30

Open
dvbobrov wants to merge 1 commit into
mcquay239:masterfrom
dvbobrov:diameter
Open

Point set diameter#30
dvbobrov wants to merge 1 commit into
mcquay239:masterfrom
dvbobrov:diameter

Conversation

@dvbobrov

Copy link
Copy Markdown

No description provided.

@antonkov

Copy link
Copy Markdown
Collaborator

I think that diametr shouldn't spoil input set of points

@dvbobrov

Copy link
Copy Markdown
Author

Point set remains the same, it is just reordered. If you need to save order, you can always copy your container outside.

@antonkov

Copy link
Copy Markdown
Collaborator

Yes, I meant reordering, and I don't expect that diametr will shuffle my elements. It's just subjective.

@ZumZoom

ZumZoom commented Sep 29, 2013

Copy link
Copy Markdown
Collaborator

Господа, простите за мой английский, но ваша дискуссия в учебном репозитории вгоняет меня в тоску)
А по теме: на мой взгляд гораздо хуже то, что нет нормального предиката для сравнения расстояний. Антон, думается мне, пристрелит за такое get_lr_distance.

@dvbobrov

Copy link
Copy Markdown
Author

Он вроде бы говорил забить на точность где-то

@ZumZoom

ZumZoom commented Sep 29, 2013

Copy link
Copy Markdown
Collaborator

В Дугласе-Пеккере, потому что там все неточно. Топология не сохраняется, и все такое, так что там никакого толку от точных вычислений нет.

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.

3 participants