исключены внутренние зачисления для приобретения ценных бумаг из пополнений;
сохранено распознавание реальных пополнений;
временные выводы под нерассчитанные сделки не считаются выводами;
добавлены проверки классификации.
Проверено на реальном temp/broker.xlsx.
- исключены внутренние зачисления для приобретения ценных бумаг из пополнений;
- сохранено распознавание реальных пополнений;
- временные выводы под нерассчитанные сделки не считаются выводами;
- добавлены проверки классификации.
Независимое ревью завершено. Проверил diff и классификацию на реальном temp/broker.xlsx: внутренние положительные операции с комментарием «Перечисление ДС для приобретения ценных бумаг» исключаются из пополнений; «Вывод ДС под нерассчитанные сделки» исключается из выводов; обычные пополнения/выводы распознаются. Регрессий в затронутом коде не обнаружил, версия 0.15.1 и CHANGELOG обновлены. Блокирующих замечаний нет. Неблокирующее замечание: автоматический тест использует синтетические строки, а не реальный XLSX-файл.
Независимое ревью завершено. Проверил diff и классификацию на реальном temp/broker.xlsx: внутренние положительные операции с комментарием «Перечисление ДС для приобретения ценных бумаг» исключаются из пополнений; «Вывод ДС под нерассчитанные сделки» исключается из выводов; обычные пополнения/выводы распознаются. Регрессий в затронутом коде не обнаружил, версия 0.15.1 и CHANGELOG обновлены. Блокирующих замечаний нет. Неблокирующее замечание: автоматический тест использует синтетические строки, а не реальный XLSX-файл.
Повторное независимое ревью актуального PR (2e1fc59) завершено. Предыдущее замечание проверено: оно остаётся неблокирующим — автоматические проверки классификатора по-прежнему синтетические, реальный XLSX проверен вручную. Новая логика корректно ограничивает сопоставление внешними счетами, использует противоположную сумму, окно 3 дня и one-to-one расходование пары; временные движения без пары исключаются. Проверил SQL-запрос, account_type-фильтрацию и edge cases (pending settlement, income, повторные суммы). Блокирующих регрессий не обнаружил. Неблокирующий риск: жадное сопоставление по сумме/окну без явного идентификатора перевода может ошибочно связать одинаковые операции; желательно покрыть getPortfolioPerformance SQL-интеграционным тестом с несколькими одинаковыми суммами.
Повторное независимое ревью актуального PR (2e1fc59) завершено. Предыдущее замечание проверено: оно остаётся неблокирующим — автоматические проверки классификатора по-прежнему синтетические, реальный XLSX проверен вручную. Новая логика корректно ограничивает сопоставление внешними счетами, использует противоположную сумму, окно 3 дня и one-to-one расходование пары; временные движения без пары исключаются. Проверил SQL-запрос, account_type-фильтрацию и edge cases (pending settlement, income, повторные суммы). Блокирующих регрессий не обнаружил. Неблокирующий риск: жадное сопоставление по сумме/окну без явного идентификатора перевода может ошибочно связать одинаковые операции; желательно покрыть getPortfolioPerformance SQL-интеграционным тестом с несколькими одинаковыми суммами.
admin
merged commit 8202d292cd into main2026-08-28 20:09:19 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Проверено на реальном temp/broker.xlsx.
Независимое ревью завершено. Проверил diff и классификацию на реальном temp/broker.xlsx: внутренние положительные операции с комментарием «Перечисление ДС для приобретения ценных бумаг» исключаются из пополнений; «Вывод ДС под нерассчитанные сделки» исключается из выводов; обычные пополнения/выводы распознаются. Регрессий в затронутом коде не обнаружил, версия 0.15.1 и CHANGELOG обновлены. Блокирующих замечаний нет. Неблокирующее замечание: автоматический тест использует синтетические строки, а не реальный XLSX-файл.
Повторное независимое ревью актуального PR (
2e1fc59) завершено. Предыдущее замечание проверено: оно остаётся неблокирующим — автоматические проверки классификатора по-прежнему синтетические, реальный XLSX проверен вручную. Новая логика корректно ограничивает сопоставление внешними счетами, использует противоположную сумму, окно 3 дня и one-to-one расходование пары; временные движения без пары исключаются. Проверил SQL-запрос, account_type-фильтрацию и edge cases (pending settlement, income, повторные суммы). Блокирующих регрессий не обнаружил. Неблокирующий риск: жадное сопоставление по сумме/окну без явного идентификатора перевода может ошибочно связать одинаковые операции; желательно покрыть getPortfolioPerformance SQL-интеграционным тестом с несколькими одинаковыми суммами.