История изменений
Исправление invy, (текущая версия) :
- Move face detection code to detect_faces()
- Optimize memory usage in detect_faces()
- Use multi-scale pyramid in detect_faces()
- Refactor and optimize face detection.
Никого не волуют мелкие изменения и технические особенности реализации. Зачем вообще комментировать что куда и зачем и почему перенесено? Это как комментировать в коде:
// create int variable and set to 0
int i = 0;
Один коммит - одно независимое атомарное изменение.
Один коммит - одна небольшая ограниченная тема (1-2 дня работы максимум).
Как ты будешь это разделять это твоим интерактивным rebase?
edit и руками нужное закоммитить.
А особенно удобно бисектить
При нормальной организации работы даже бисектить ничего не надо - все коммиты тематически рассортированы и ограничены. Сломалась подсистема X - смотришь, какие коммиты ее затрагивали.
Исходная версия invy, :
- Move face detection code to detect_faces()
- Optimize memory usage in detect_faces()
- Use multi-scale pyramid in detect_faces()
- Refactor and optimize face detection.
Никого не волуют мелкие изменения и технические особенности реализации.
Один коммит - одно независимое атомарное изменение.
Один коммит - одна небольшая ограниченная тема (1-2 дня работы максимум).
Как ты будешь это разделять это твоим интерактивным rebase?
edit и руками нужное закоммитить.
А особенно удобно бисектить
При нормальной организации работы даже бисектить ничего не надо - все коммиты тематически рассортированы и ограничены. Сломалась подсистема X - смотришь, какие коммиты ее затрагивали.