-
Notifications
You must be signed in to change notification settings - Fork 13
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
Task05 Игорь Логинов SPbU #27
Closed
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Все хорошо, 9/10 баллов 👍 (т.к. после дедлайна) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
cameras.out видимо сейчас есть только для датасета temple47, вот облака точек для части камер и для всех камер
на первом с какой-то стороны видно лучше, с других сторон - хуже (вроде это логично)
на втором местами заметны отслоения, вроде это ок т.к. мы их пока не фильтруем (но кажется у них обычно другие причины появления должны быть вида "кто-то прошёл"/"машина проехала" - тут ничего такого нет)
куча тёмных точек наверное появляется из-за того что все снимки на тёмном фоне и тёмный фон может быть воспринят как "тёмная жидкость вокруг предмета"
почему то в какой-то момент происходит что-то странное:
log
вроде сейчас я немного подкрутил CI, чтобы библиотеки не пересобирались каждый раз - может быть получится поэкспериментировать с какими-то изменениями (локально на виртуалке это слишком долго работает)
а, я пока писал PR понял, что cameras.out для других датасетов есть в task06, попробую их тоже запустить
saharov32, все камеры
saharov32, часть камер (заметно с какой стороны снимали)
saharov32, если точки снаружи содержат какую-то информацию об окружении, то внутри точек почти нет
herzjesu25:
тут сильно меньше точек от окружения (наверное потому что снимали с одной стороны), лучше заметны ошибочные точки. также видно, откуда примерно снимали + машина почему-то достаточно неплохо сохранилась
saharov32 c dataset_downscale=2 - тут даже надпись "А. Сахаров" можно разглядеть (наверное с большим dataset_downscale тоже можно, но заметил только сейчас); надпись "Академик" сложнее заметить т.к. рядом больше точек фона
TODO 105:
Запустил разные варианты "случайных пертурбаций" и длины отрезка из камер:
Результат для (to_shift = 5, ref_camera_shift = 2, w decreases from 2 to 1):
Результат для (to_shift = 7, ref_camera_shift = 3, w decreases from 2.3 to 1.5):
Результат для (to_shift = 7, ref_camera_shift = 3, w is fixed at 2):
Кажется от w тут вообще ничего не зависит (там логи чуть ли не полностью одинаковые), при увеличении длины отрезка камер вроде стоимость становится лучше ценой большего времени работы (визуально отличия не заметны)