-
Notifications
You must be signed in to change notification settings - Fork 0
Comparing changes
Open a pull request
base repository: queelius/algebraic_hashing
base: v2.0.0
head repository: queelius/algebraic_hashing
compare: v2.0.1
- 5 commits
- 73 files changed
- 2 contributors
Commits on Sep 28, 2025
-
Clean up repository and improve organization
- Removed temporary build artifacts and binary files - Moved internal documentation to docs/ directory - Updated .gitignore with comprehensive exclusion rules - Added previously untracked configuration files (.clang-format, .clang-tidy, etc.) - Organized project structure for better maintainability Project is now clean and ready for distribution. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for 134e771 - Browse repository at this point
Copy the full SHA 134e771View commit details
Commits on Oct 1, 2025
-
Remove old paper directory and research files
Clean up legacy documentation and research materials that are no longer needed. The whitepaper has been consolidated in the docs/ directory. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for 3169f05 - Browse repository at this point
Copy the full SHA 3169f05View commit details
Commits on Oct 2, 2025
-
Major whitepaper revision with empirical validation
Academic Review Fixes: - Corrected mathematical errors (removed false group structure claim) - Fixed avalanche preservation proof with empirical validation - Added comprehensive Related Work section with proper citations - Added FKS and universal hashing citations in main text Implementation Improvements: - Added seed parameter to fnv_hash for meaningful composition - Fixed include paths in proracle.hpp and hash_value.hpp - Fixed hash_value bugs (sizeof(T) -> sizeof(uint8_t), etc.) Benchmarks & Validation: - Implemented comprehensive benchmark suite (throughput, quality, PHF) - Replaced fabricated data with real measurements - All 26 unit tests passing Paper Updates: - Updated abstract with honest performance claims - Replaced benchmark tables with actual data - Aligned code listings with real implementation - Performance: FNV 209 Mhash/s, FNV⊕FNV 120.5 Mhash/s - Quality: Avalanche 0.48 → 0.41 (near ideal 0.5) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for 4f67ad2 - Browse repository at this point
Copy the full SHA 4f67ad2View commit details -
Fix LaTeX compilation: add pifont package and headheight
- Added \usepackage{pifont} for \ding symbols (checkmarks/crosses) - Fixed headheight warning (12pt -> 14pt) - PDF now compiles cleanly with only minor float warnings 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>Configuration menu - View commit details
-
Copy full SHA for 3d1ade7 - Browse repository at this point
Copy the full SHA 3d1ade7View commit details
Commits on Nov 15, 2025
-
Major improvements: Fix critical bug, add comprehensive testing, and …
…enhance test coverage This commit addresses all critical issues identified in TDD review and significantly improves the quality and reliability of the AlgebraicHashing library. **Critical Bug Fixes:** - Fix stack-use-after-return in get_arithmetic_bytes() for numeric types - Changed return type from span to array to prevent dangling pointers - Bug was causing undefined behavior when hashing integers and floats - All tests now pass with sanitizers enabled **New Test Infrastructure (549+ lines):** - Add regression tests for numeric hashing consistency - Add comprehensive thread safety test suite (5 tests, 8-32 threads) - Concurrent hashing determinism - Shared hasher instance safety - Concurrent composition operations - Hash value operations thread safety - Data race detection tests - Add official FNV-1a test vectors for validation - 64-bit FNV-1a test vectors (11 test cases) - 32-bit FNV-1a test vectors (7 test cases) - Binary data and long string tests - All vectors verified against reference implementation **Coverage Reporting:** - Add CMake coverage target with lcov/genhtml integration - Verified 99.5% line coverage, 100% function coverage - HTML coverage reports generated at build/coverage_html/ - Enable with: cmake -DENABLE_COVERAGE=ON **Statistics Thread Safety:** - Document that statistics are NOT thread-safe (educational use only) - Add ALGEBRAIC_HASHING_DISABLE_STATISTICS compile option - Hash functions themselves remain fully thread-safe - Clear warnings in documentation and code comments **Documentation Updates:** - Update CLAUDE.md with comprehensive build instructions - Correct coverage claims (from "99.4%" to "comprehensive" until verified) - Add thread safety documentation - Update README, CHANGELOG, and release notes **Test Results:** - All 34+ tests passing (100% pass rate) - Thread safety verified with concurrent stress tests - FNV implementation validated against official specification - Sanitizers (ASan, UBSan) clean 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for e8160a8 - Browse repository at this point
Copy the full SHA e8160a8View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v2.0.0...v2.0.1