Opened 5 лет ago

Last modified 4 года ago

#160 new улучшение

Замечания к работе функции сброса к заводским настройкам

Сообщил: Art_M Владелец:
Приоритет: фигня Этап разработки: 2-я очередь
Ключевые слова: Копия: alx, andrei, san

Описание (последним изменил Art_M)

Прошивка 692

В технологических настройках добавлен пункт сброс к заводским настройкам. Не смотря на то, что этой функции не было в ТЗ, данная функция полезна и ее нужно довести до ума. А пока на данный момент есть замечания:

  1. Кроме технологических настроек сбрасываются все настройки, включая настройки ЧРП, которые включают в себя и тип ЧРП (сбрасываются на Dnafoss), способ управления (сбрасываются на RS-485), ток дивгателя (сбрасывается на 0), сбрасывается и точка доступа APN. Возможно сбросится и датчик температуры. Здесь нужно подумать нужно ли сбрасывать такие важные уставки.
  1. Требуется подтверждения необходимости сброса, т.к. тычок в сброс может быть выполнен случайно, и шансов у оператора исправить ситуацию уже не будет.
  1. Значения по умолчанию должны быть пересмотрены мной (Макаровым А.П.), т.к. в процессе получения реального опыта пересмотрены некоторые значения.

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

Мне это видится следующим образом:
"Общие настройки" между пунктами "ТипPDP" (кстати, а что это?) и "Рестарт smarthdc" ввести пункт "Сохр/вост/сброс конфигурации", внутри которого "Сохранение конфигурации", "Восстановление конфигурации", "Сброс конфигурации".

  1. При вызове "Сохранение конфигурации", выбор "Накопитель USB" или "Память контроллера".

1а. При нажатии "Накопитель USB", то выполняется, то что сейчас выполняется при сохранении конфига на флэшку.
1б. При нажатии "Память контроллера", то выполняется сохранение в какой то определенный файл.

  1. При вызове "Восстановление конфигурации", выбор "Накопитель USB" или "Память контроллера".

2а. При нажатии "Накопитель USB", то выполняется, то что сейчас выполняется при восстановлении конфига с флэшки.
2б. При нажатии "Память контроллера", то выполняется подхват уставок из хранящегося файла.

  1. При вызове "Сброс" произвести сброс уставок к заводским. Естественно требуется подтверждения необходимости сброса, при чем курсор должен быть в положении "нет".

История изменений (8)

comment:1 by Art_M, 5 лет ago

Описание: изменено (отличие)

comment:2 by andrei, 5 лет ago

Копия: alx andrei san added

in reply to:  description ; comment:3 by alx, 5 лет ago

Replying to Art_M:

Прошивка 692

Что такое "Прошивка 692"?

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

Это все и сейчас можно - есть функция сохранения, функция сброса, функции изменения настроек и функция восстановления. Что с чем предлагается объединить? Сохранение со сбросом? Но тогда нельзя будет выполнить просто сохранение без сброса. Можно, конечно, сразу после сохранения выполнить восстановление, но это же лишние действия... Не понимаю, в чем же тут улучшение...

Мне это видится следующим образом:
"Общие настройки" между пунктами "ТипPDP" (кстати, а что это?)

Packet Data Protocol (англ.) — пакетный протокол, используемый в беспроводных сетях GPRS/HSDPA. Подробнее можно прочитать, например, здесь.

1б. При нажатии "Память контроллера", то выполняется сохранение в какой то определенный файл.

??? Сейчас у нас в точности это и происходит автоматически, без выбора какого-либо специального функта меню при любом изменении настроек (кроме тех, для которых в ТЗ явно указано, что их сохранять не надо). Зачем же тогда дополнительный пункт? Опять не вижу, каким образом это улучшит станцию.

2б. При нажатии "Память контроллера", то выполняется подхват уставок из хранящегося файла.

Этот "подхват" выполняется автоматически при старте smarthdcd.

in reply to:  3 comment:4 by andrei, 5 лет ago

Replying to Art_M:

Прошивка 692

В технологических настройках добавлен пункт сброс к заводским настройкам. Не смотря на то, что этой функции не было в ТЗ, данная функция полезна и ее нужно довести до ума. А пока на данный момент есть замечания:

  1. Кроме технологических настроек сбрасываются все настройки, включая настройки ЧРП, которые включают в себя и тип ЧРП (сбрасываются на Dnafoss), способ управления (сбрасываются на RS-485), ток дивгателя (сбрасывается на 0), сбрасывается и точка доступа APN. Возможно сбросится и датчик температуры. Здесь нужно подумать нужно ли сбрасывать такие важные уставки.

В ТЗ есть таблица, содержащая значения всех настроек/уставок. В таблице есть столбец "По умолчанию". Разве после сброса устанавливаются значения не из этого столбца?
Если не из этого, то наверно нужно поправить программу.
Если из этого, но это плохо в эксплуатации, то нужно переписать таблицу.

Replying to alx:

Replying to Art_M:

Прошивка 692

Что такое "Прошивка 692"?

Имеется в виду версия smartHDC.

in reply to:  3 ; comment:5 by Art_M, 5 лет ago

Replying to alx:

Что такое "Прошивка 692"?

Версия "smarthdc 1.0-r692"

Это все и сейчас можно - есть функция сохранения, функция сброса, функции изменения настроек и функция восстановления. Что с чем предлагается объединить? Сохранение со сбросом? Но тогда нельзя быдет выполнить просто сохранение без сброса. Можно, конечно, сразу после сохранения выполнить восстановление, но это же лишние действия... Не понимаю, в чем же тут улучшение...

Я имею ввиду объединить функции в одну логичную менюшку, а не про последовательное выполнение. Что бы не искать сброс где-то в "технологических настройках", а восстановление в "общих настройках"

??? Сейчас у нас в точности это и происходит автоматически, без выбора какого-либо специального функта меню при любом изменении настроек (кроме тех, для которых в ТЗ явно указано, что их сохранять не надо). Зачем же тогда дополнительный пункт? Опять не вижу, каким образом это улучшит станцию.
Этот "подхват" выполняется автоматически при старте smarthdcd.

Я имею ввиду сохранение дополнительного резервного файла конфигурации в памяти самого контроллера, который можно восстановить в любой момент. К примеру, у оператора нет с собой флэшки, а нужно провести какие-нибудь опытные исследовательские работы на действующем приводе, у которого возникла какая-то неисправность. Для выяснения причин отказа методом "от противного", следует перенастроить привод на иную методику работы, для этого требуется скорректировать к примеру 5-6 значений уставок. После выяснения и устранения причины неисправности нужно вернуть все к режиму, как было при приезде оператора. Что бы не вспоминать все уставки в которые вносились изменения нужно иметь возможность быстро восстановиться из резервного файла настроек хранящегося в собственной памяти контроллера.

Replying to andrei:

В ТЗ есть таблица, содержащая значения всех настроек/уставок. В таблице есть столбец "По умолчанию". Разве после сброса устанавливаются значения не из этого столбца?
Если не из этого, то наверно нужно поправить программу.
Если из этого, но это плохо в эксплуатации, то нужно переписать таблицу.

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

in reply to:  5 comment:6 by andrei, 5 лет ago

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

Тогда вопросов нет, я думал что может ты хочешь сделать другой набор значений по-умолчанию, специально для функции сброса.

in reply to:  5 comment:7 by alx, 5 лет ago

Replying to Art_M:

Я имею ввиду объединить функции в одну логичную менюшку,

Спасибо, понял. Но я не вижу здесь улучшения.

С одной стороны, эти функции, действиетльно, сходного типа (все они оперируют настройками), и есть логика в том, чтобы поместить их в одно подменю с функцией "Записать настройки в ЧРП".

Но, с другой стороны, функции "Сохранить конфиг" и "Восстановить конфиг" работают с портативным носителем. И если перенести их в некое другое подменю, получится, что часть функций по работе с портативным носителем находятся в подменю "Портативный носитель", а другая часть - совсем в другом подменю, что логику группировки нарушает.

Получается, что, как ни группируй, какая-то логика будет нарушаться...

Я имею ввиду сохранение дополнительного резервного файла конфигурации в памяти самого контроллера,

Согласен. Это, наверное, может быть полезным.

comment:8 by san, 5 лет ago

Приоритет: среднийфигня
Note: See TracTickets for help on using tickets.