You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Закрываю по итогам сравнения трёх независимых решений issue #2. За базу взят #10.
Это был контрольный прогон — третье независимое решение той же задачи, чтобы отделить разброс между запусками от систематики. Он свою роль выполнил: дефолт CollectionStatus.IN_PROGRESS здесь выбран в безопасную сторону, в отличие от #8 с его COMPLETE. Значит слабость #8 была неудачей одного прогона, а не свойством подхода.
Почему всё же не эта ветка.status остаётся хранимым полем — проверил на ветке, status=COMPLETE с нулём экспортов конструируется явно. В #10 это состояние непредставимо: поле вычисляется из exports.
Плюс: нет missing_views вообще — файл сообщает in_progress, но не говорит, чего именно не хватает. Issue просила «статус или список недостающих видов», формально выполнено, практически менее полезно. И нет проверки наличия <view>.parquet на диске при резюме: удалённый вручную файл будет молча пропущен. Тестов 4 против 27.
model_validator на уникальность view в exports — этого не было ни в одном другом решении. Проверил, работает. Защищает не от своего кода (там дубли и так невозможны), а от уже испорченного манифеста на диске, который читается при резюме. Защита на чтении в дополнение к защите на записи.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #2
Summary
--resume-runvalidation to append only missing views for the matching phrase and regionTests
uv run pytest(73 passed)uv run ruff check .Risk