Skip to content
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

Names autofix #1502

Closed
wants to merge 5 commits into from
Closed

Names autofix #1502

wants to merge 5 commits into from

Conversation

lucidyan
Copy link

Добавлен функционал унифицирования имен, запрошенный в

Так же, скрипт выводит полезный лог, в котором находит возможные некорректные имена

  • Содержащие некириллические символы

    • Кириллица, написанная латиницей
      image

    • Опечатки
      image

  • Возможные неточности (есть ложные срабатывания)
    image

Лог замен:
image

Поправлено считывание файлов, из-за которого появлялись некорректные символы в именах
https://stackoverflow.com/a/49150749/3338479

Реализовано на основе:

Добавленные зависимости:

  • fuzzyset
  • transliterate

Я знаю, что функционал дублируется #1147 (когда начинал, его не было) - предлагаю ментейнеру протестировать и сравнить результаты.

@hookzof hookzof added attention enhancement New feature or request labels Sep 19, 2019
@fersel
Copy link
Member

fersel commented Sep 19, 2019

Спасибо! Моё мнение: унифицировать имена подписавшихся не нужно, потому что это отражает (либо может отражать) способ самоидентификации человека.

В то же время в ишью, на которые вы ссылаетесь, говорится о необходимости унификации имён фигурантов -- с этим я согласен.

@lucidyan
Copy link
Author

lucidyan commented Sep 19, 2019

@fersel Ознакомьтесь с другими списками подписавшихся или даже с другим списком от IT-сообщества - они все имеют четкую структуру. Все-таки это открытое обращение, нужно соблюдать какие-то формальности. И уж точно это не место для каких-то самоидентификаций.

Так что я полностью согласен с #553 (comment)

Issues, которые я упоминул, в комментариях затрагивают и тему самого списка (хоть в названиях и участвует слово "фигуранты").

@lucidyan
Copy link
Author

@hookzof Примите, пожалуйста, решение о принятии, либо отклонении пулл-реквеста

@fersel
Copy link
Member

fersel commented Sep 20, 2019

@lucidyan спасибо за проявленный и интерес и помощь, но мы решили, что не будем унифицировать имена (подписантов).

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

[Далее ответ в личном качестве] Я не знаком со всеми списками, но ни список учителей, ни "открытая часть" второго списка от IT-сообщества не имеют определённой структуры в подписях. В последнем случае люди пишут свои имена так же произвольно, как и здесь.

В то же время, согласен, есть списки с четкой структурой, и они выглядят красиво, но я считаю это важной частью либо небольших списков, либо "закрытой части" списка (как в тексте этого списка, до комментариев), потому что это -- "лицо" письма. У нас, по моему мнению, другая ситуация.

Более того, смысл "открытого письма" не в том, чтобы собрать упорядоченную базу, а в оказании (политического) давления; сила определяется количеством и реальностью этих подписей, форматирование для этого неважно.

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

@fersel fersel closed this Sep 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
attention enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants