SASGIS - SAS.Планета
View Issue Details
0001105SAS.Планета[All Projects] Хотелкаpublic12-01-2012 07:3829-10-2012 12:45
vasketsov 
vdemidov 
normalfeaturealways
resolvedfixed 
WindowsVistaUltimate
110418 
131111131111 
0001105: Разделить настройку "Алгоритм изменения размера изображений изображений"
Сделать настройку алгоритма ресайза изображений выбираемого по-умолчанию при генерации вышележащих зумов.
зум, настройки
Issue History
12-01-2012 07:38vasketsovNew Issue
12-01-2012 09:25vasketsovNote Added: 0004877
12-01-2012 12:13gpsMaxTag Attached: зум
12-01-2012 12:13gpsMaxTag Attached: настройки
12-01-2012 15:54GarlNote Added: 0004895
12-01-2012 16:08GarlNote Edited: 0004895bug_revision_view_page.php?bugnote_id=4895#r2431
12-01-2012 16:18vasketsovNote Added: 0004896
12-01-2012 17:12GarlNote Added: 0004898
12-01-2012 17:43vasketsovNote Added: 0004899
12-01-2012 18:15GarlNote Added: 0004905
13-01-2012 04:10TolikNote Added: 0004907
13-01-2012 04:27GarlNote Added: 0004910
13-01-2012 04:34TolikNote Added: 0004913
13-01-2012 04:40GarlNote Added: 0004915
13-01-2012 04:45TolikNote Added: 0004917
13-01-2012 04:45TolikNote Edited: 0004917bug_revision_view_page.php?bugnote_id=4917#r2438
13-01-2012 04:56GarlNote Added: 0004920
13-01-2012 06:37vasketsovNote Added: 0004922
13-01-2012 06:49TolikNote Added: 0004923
13-01-2012 07:20vdemidovNote Added: 0004924
13-01-2012 07:23vasketsovNote Added: 0004925
13-01-2012 07:25vasketsovNote Edited: 0004925bug_revision_view_page.php?bugnote_id=4925#r2440
13-01-2012 07:26GarlNote Added: 0004926
13-01-2012 07:26TolikNote Added: 0004927
13-01-2012 07:27TolikNote Added: 0004928
13-01-2012 07:30GarlNote Added: 0004929
13-01-2012 07:31vasketsovNote Added: 0004930
13-01-2012 07:32TolikSeveritytweak => feature
13-01-2012 07:32TolikStatusnew => acknowledged
13-01-2012 07:32TolikCategoryБаг => Хотелка
13-01-2012 07:32TolikSummaryАлгоритм формирования вышележащих зумов сбрасывается => Добавить настройку "Алгоритм уменьшения изображений"
13-01-2012 07:33TolikNote Added: 0004931
13-01-2012 07:35vasketsovNote Edited: 0004930bug_revision_view_page.php?bugnote_id=4930#r2442
13-01-2012 07:37TolikNote Edited: 0004931bug_revision_view_page.php?bugnote_id=4931#r2444
13-01-2012 07:45TolikNote Edited: 0004931bug_revision_view_page.php?bugnote_id=4931#r2445
26-01-2012 22:12vdemidovDescription Updatedbug_revision_view_page.php?rev_id=2622#r2622
26-01-2012 22:13vdemidovProduct Version.Nightly => 110418
26-01-2012 22:13vdemidovTarget Version => 24xxxx
01-02-2012 14:06vdemidovStatusacknowledged => confirmed
29-10-2012 12:29vdemidovTarget Version24xxxx => 131111
29-10-2012 12:29vdemidovSummaryДобавить настройку "Алгоритм уменьшения изображений" => Разделить настройку "Алгоритм изменения размера изображений изображений"
29-10-2012 12:30vdemidovNote Added: 0009774
29-10-2012 12:30vdemidovStatusconfirmed => resolved
29-10-2012 12:30vdemidovFixed in Version => 131111
29-10-2012 12:30vdemidovResolutionopen => fixed
29-10-2012 12:30vdemidovAssigned To => vdemidov
29-10-2012 12:39TolikNote Added: 0009775
29-10-2012 12:45TolikNote Added: 0009776
29-10-2012 12:46TolikNote Edited: 0009776bug_revision_view_page.php?rev_id=4780
29-10-2012 12:48TolikNote Edited: 0009776bug_revision_view_page.php?rev_id=4781
31-10-2012 06:59TolikNote Deleted: 0009776
31-10-2012 07:00TolikNote Deleted: 0009775

Notes
(0004877)
vasketsov   
12-01-2012 09:25   
Загадка разрешилась переключением в настройках "Алгоритм растягивания изображений" ))))))))))))))))))))))))))))))))))))))))))))))))))
По какому-то разумению эта опция влияет на формирование верхних зумов, а когда слетела - неизвестно.

Итого необходимо минимум одно из двух:
1. Добавить отдельную опцию для генерации верхних зумов (ну и соответственно обучить сас её юзать).
2. Сохранять (не сбрасывать) алгоритм, как в описании выше.
(0004895)
Garl   
12-01-2012 15:54   
(edited on: 12-01-2012 16:08)
так вроде ж всё логично.
дефолтный алгоритм для всех и для генерации и для показа нижележащих при отсутствии текущих тайлов

единственное что - прописать в WIKI и забыть как страшный сон.

(0004896)
vasketsov   
12-01-2012 16:18   
ну кроме того что растягивание и сжатие сиречь существенно разные с точки зрения логики и настроек процедуры... наличие ровно одной настройки приводит к тому, что её МОЖНО использовать в двух разных процедурах, но не более того. Например в моём случае (снимок nokiamapcreator) как раз растягивать удобнее как Nearest, а сжимать как Box, визуально разница видна невооружённым взглядом.
(0004898)
Garl   
12-01-2012 17:12   
Согласен.
Тогда нужно выносить в конфиг и GUI да брать дефолтные значения оттуда.
(0004899)
vasketsov   
12-01-2012 17:43   
Вопрос остаётся в том, надо ли 2) Сохранять (не сбрасывать) алгоритм. Возможно это было сделано специально, чтобы при повторном открытии окошка значение всегда было дефолтным (из настроек). Ну да думаю когда руки дойдут до этого пункта, в коде видно будет что почём.
(0004905)
Garl   
12-01-2012 18:15   
и тот и тот вариант хорош.
если сбрасывать на дефолтное - тогда разделить эти 2 параметра.
(0004907)
Tolik   
13-01-2012 04:10   
Что-то я запарил: а как формировать вышележащие зумы? В операциях с выделенной областью ведь только формирование меньших зумов?

По поводу GUI, мне кажется, из окна настроек вообще лучше эти алгоритмы выкинуть, в ini запоминать каждый раз при выборе алгоритма в окне формирования.
(0004910)
Garl   
13-01-2012 04:27   
>мне кажется, из окна настроек вообще лучше эти алгоритмы выкинуть
а лезть в ini даже каждый раз когда сносишь конфиг - не айс.
проще спрятать подальше но в GUI. просто некоторые часто юзают эти настройки.
(0004913)
Tolik   
13-01-2012 04:34   
Я не предлагаю править в ini вручную. Наоборот, предлагаю менять дефолтный алгоритм каждый раз, когда выбираешь его при формировании.
Сейчас ведь большинство программ так и делают: ничего не просят сохранить вручную, запоминают настройки сами.
(0004915)
Garl   
13-01-2012 04:40   
а я часто делаю генерацию вышележащих слоёв, и каждый раз выбирать алгоритм - честно влом.
(0004917)
Tolik   
13-01-2012 04:45   
Ну так я и предлагаю каждый раз его сохранять. И для выше, и для ниже - 2 записи в ini.
Так как же всё-таки сгенерировать вышележащий зум?

(0004920)
Garl   
13-01-2012 04:56   
никак
(0004922)
vasketsov   
13-01-2012 06:37   
>Так как же всё-таки сгенерировать вышележащий зум?
В моём понимании вышележащий зум - зум с меньшим номером, погружаемся - зум растёт.
Просто с моей точки зрения слово "слой" некорректно применять к конкретному зуму карты.
Если кого-то ввёл в заблуждение - извиняюсь.
(0004923)
Tolik   
13-01-2012 06:49   
А я себе представляю наоборот :)
Сформулирую тогда первоначальную проблему по-своему:

При формировании меньших зумов выбранный алгоритм не сохраняется, а каждый раз берётся из настройки Resize algorithm - "Алгоритм растягивания изображений".

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

Теперь объясните, что надо сделать:
1. переименовать настройку на "Алгоритм уменьшения изображений"?
2. убрать эту настройку вообще, а сохранять алгоритм при выборе?
3. сделать также функцию формирования бОльших зумов (а для неё другой алгоритм)?
4. что-то ещё?

2 и 3 - это новые фичи
(0004924)
vdemidov   
13-01-2012 07:20   
>1. переименовать настройку на "Алгоритм уменьшения изображений"?
Это с какой радости? Основной смысл настройки, задавать алгоритм именно при растягивании изображений. А то что оно еще и используется как дефолтное в генерации тайлов вышележащих зумов, так это просто побочный эффект.
(0004925)
vasketsov   
13-01-2012 07:23   
(edited on: 13-01-2012 07:25)
Необходимо разделить алгоритмы для растягивания изображений (когда оверзумимся и нет тайлов) и сжатия изображений (когда генерится из 14-го зума к примеру 12-й) в части применяемой опции.
В принципе достаточно иметь опцию в настройках, сохранять в ini-шке (со сбросом к дефолтному значению при открытии окна генерации тайлов как сейчас).
В самих алгоритмах ничего менять не надо.

>Это с какой радости?
+1. Просто добавить уточнение. Переименовывать вообще ни надо ни в английском ни в русском (про другие не знаю).

(0004926)
Garl   
13-01-2012 07:26   
это планета сейчас умеет только растягивать,
а если её научить и сжимать?

вот тогда и пригодится вторая настройка.
(0004927)
Tolik   
13-01-2012 07:26   
Ок, значит, так: эту настройку оставить как есть, а добавить новую "Алгоритм уменьшения изображений"
Ok?
(0004928)
Tolik   
13-01-2012 07:27   
Сжимать-то она умеет - в операциях с выделенной областью.
(0004929)
Garl   
13-01-2012 07:30   
я сжимать имел ввиду по аналогии как сейчас растягивается тайл при остутствии...
(0004930)
vasketsov   
13-01-2012 07:31   
(edited on: 13-01-2012 07:35)
>а если её научить и сжимать?
так вроде ужЕ? просто растягивание налету, а сжатие в файл, с точки зрения алгоритма разницы никакой (я так думаю, может и не прав конечно).

А вообще вроде бы обсуждалось как-то, дофига ресурсов убьёшь чтобы проверить наличие всех тайлов, так что дальше +1 зума технически отстойно будет, а мысла только на +1 зум нырять тоже мало. Да и отсутствие тайла визуально не распознать. Я вот не качаю и не генерю 13-й и 17-й вообще (ну кроме роскосмоса конечно) - ну может и был бы смысл с +1 брать, но это в сущности фигня и не надо.

Толик, всё верно, это необходимый и достаточный минимум, прямо под существующей опцией и засандалить.

(0004931)
Tolik   
13-01-2012 07:33   
(edited on: 13-01-2012 07:45)
Тогда я изменил название.
По поводу добавления уменьшалок и увеличилок надо открывать новые хотелки.

На английском тогда текущую настройку переименовать, например, так:
Upsize algorithm

Новую можно назвать:
Downsize algorithm

(0009774)
vdemidov   
29-10-2012 12:30   
Разделил настройку на несколько разных. Other теперь отвечает только за дефолтный алгоритм генерации тайлов из больших зумов.