fix парсинга Праздников Святым
A pull request for this has already been merged.
- #34 by @maxim1770 — merged
- Dominant language
- Python
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
Баги
-
Много повторных данных не собирается из указано по умолчанию HolidayCategoryTitle == 'День памяти'
Т.к Дней Памяти Святого может быть больше одного в году, или просто не смог найти верный HolidayCategoryTitle из full_title
Увидел этот баг, что не спрасились данные с 'переходящее' датой, т.к holiday.slug уже был задан
Посмотреть при каких данных встречаются эти баги, и исправить, чтобы было больше данных
Или добавить эти данные вручную, если это будет легче- Из-за этого не собран Праздник Иоа́нна Златоуста
- Другой Праздник Иоа́нна Златоуста так же не собран из-за
class="holiday_link"на теге <a>
-
Перенесение из Иерусалима в Константинополь мощей первомученика Стефа́на, архидиакона (ок. 428).
2024-08-15
Данные не собираются, т. собирается только 'Перенесение мощей'
Исправить это, чтобы тоже собиралось это, но помнить про то, что есть еще 'Второе перенесение мощей', которое должно собираться отдельно. -
Ап. от 70-ти первомч. и архидиа́кона Стефа́на (34);
2024-01-09
Дата указана 34, собрана, после обработки 34+8=42
Так и не понял, возможно, что тут на сайте дата указана правильно
В житие на сайте написано Так архидиакон Стефан стал первым мучеником за Христа в 34 году после Р.Х.
Нужно думать, если что исправлять данные у себя (я про 42)) -
Два пробела в бд (уже исправил, пробелы убираются в общем prepare _clean_holiday_title())
Прп. Ио́сифа Многоболезненного, Печерского, в Дальних пещерах (XIV);
2024-04-17 -
fix(models): убрать unique из holiday.title т.к у двух Святых оказалось одно имя на сайте azbyka
https://azbyka.ru/days/sv-evpsihij-kesarijskij-kappadokijskij
https://azbyka.ru/days/sv-evpsihij-kesarijskij-kappadokijskij-drugoj
slug - разные, а title абсолютно одинаковые
УЖЕ СДЕЛАЛ, НО ВОЗМОЖНО ЭТО НЕВЕРНО, И СТОИТ Update КАКОЙ ТО ИЗ holiday_title, а unique оставить как было =True
Заметки
-
Национальность
Данные о национальности указаны не много где: 26 в бд, 65 на сайт c (saint-href)
Не много данных, поэтому наверное создавать поле для этого не стоит, А МОЖЕТ И СТОИТ -
'переходящее' даты
Так же данны немного, на все сайт c (saint-href) всего 11, в бд 6
Но в бд было бы больше, если бы не баг с holiday.slug уже был задан
Некоторые переходящие Праздники не помечены: -
Может быть данных о переходящих датах и больше:
Неделя 5-я Великого поста. Прп. Марии Египетской.
2024-04-21
Тут так же переходящая, но это мы не учли в 11, т.к нет слова 'переходящее' -
'.' - никогда не берется в full_title т.к записана не в тегах <a>
Пишется после окончания группы <p>
И что интересно после Первого Праздника дня - первый праздник самый главный и большой в этот день
Возможно иногда логические группы праздников разделяются и ';' в одном <p>
Но скорее всего просто обозначение конца каждого full_title, часто (707 в бд) ';' попадает в full_title, и ее потом нужно убирать для holiday_title
Точку с конца full_title нельзя удалять т.к это окончание сокращения dignity or face_sanctity
БАГ: НО ЕСТЬ ИСКЛЮЧЕНИЕ мч. Илиодо́ра Магидского (Памфилийского) (ок. 273).; И В БД: 'мч. Илиодо́ра Магидского (Памфилийского) .' ИСПРАВИТЬ ВРУЧНУЮ ЧЕРЕЗ Update -
Интересная ситуация
Преставление (662), перенесение мощей прп. Максима Исповедника;
Прп. Максима Исповедника (662);
Подробнее читать житие на сайте, по поиску 'перенесение'
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No source files or tests are named in the issue. Review the merged pull request #34 first, then reproduce the listed parsing and database cases; done means the affected holidays are collected correctly without reintroducing date, title, or uniqueness errors.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- postgresql, python, sqlalchemy
- Domain
- backend, databases
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 15/100