Skip to content

Соболев Александр#8

Open
Soblev515 wants to merge 4 commits intorailbotan:mainfrom
Soblev515:Соболев-Александр

Hidden character warning

The head ref may contain hidden characters: "\u0421\u043e\u0431\u043e\u043b\u0435\u0432-\u0410\u043b\u0435\u043a\u0441\u0430\u043d\u0434\u0440"
Open

Соболев Александр#8
Soblev515 wants to merge 4 commits intorailbotan:mainfrom
Soblev515:Соболев-Александр

Conversation

@Soblev515
Copy link

Исправлены четыре ошибки, которые заставляли фильтр работать не так, как нужно:

  • Переполнением беззнакового целого numpy.uint8.
  • Изменены именами переменных.
  • Исправлены подсчёты компонентов серого цвета
  • Исправлены неверно работающие граничные условия

Исправлены четыре ошибки, которые заставляли фильтр работать не так, как нужно:
- Переполнением беззнакового целого numpy.uint8.
- Изменены именами переменных.
- Исправлены подсчёты компонентов серого цвета
- Исправлены неверно работающие граничные условия
@Soblev515 Soblev515 changed the title Этап 1 Соболев Александр Nov 14, 2021
-PEP8.
-Именование переменных.
-Возможность управлять размерами мозайки (сейчас — только 10x10).
-Возможность управлять градациями серого (сейчас — с шагом 50). Лучше сделать просто в виде задания количества шагов. Например: 4 градации, 6 градаций.
-Выделение функций(методов).
-По возможности убраны ручные циклы, заменены на матричные преобразования.
- Исправлена ошибка с делением на 3(суммы серого).
Переписана в консольная утилита, которой на вход подаются имена исходного изображения и результата
average = self.__GetAverage(y, x)
self.arr[y: y + self.mosaic_size, x: x + self.mosaic_size] = int(average // self.grayscale) * self.grayscale

def __SaveResult(self, newIMG):
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

self.__SaveResult(newIMG)

img = Image.open(input("Введите название изображение: "))
filtr = Filter(img, 10, 50)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@railbotan
Copy link
Owner

5

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.

2 participants