Repository navigation
feature(ai): A02 — Piyasa Veri Kümesi Okuyucusu ve Ortak Özellik (Feature) Üretim Hattı - #46
Merged
Merged
Conversation
…elerini ve birim testlerini eklendi
… formatina donustur
…mmit-level gemini appsec review
…me ve piyasa veri okuyucusunu eklendi
🛡️ Otomatik Kod ve Güvenlik İncelemesi
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Firatasi
commented
Oct 24, 2026
Firatasi
left a comment
Firatasi
left a comment
Owner
Author
There was a problem hiding this comment.
🔍 Code Review Değerlendirmesi: ONAYLANDI (LGTM)
Özellik üretim hattı (pipeline.py), çoklu zaman dilimi birleştirmesi ve veri kümesi okuyucusu incelendi:
- Train/Serve Parity ve Determinizm: Hem eğitim hem de inference aynı saf
build_featuresfonksiyonunu kullanıyor;SPECsıralaması,SCHEMA_HASHveuuid5tabanlıFeatureSnapshot.idtam determinizm sağlamış. - Zaman Sızıntısı (Lookahead) Koruması:
finalized,close_time <= as_ofveavailable_at <= as_ofüçlü kontrolü ile kapanmamış veya geç yayınlanmış mumların modele sızması tamamen engellenmiş. - Finansal Hassasiyet: ATR ve referans fiyatın
Decimalile hesaplanıp string olarak B09NumericPolicykatmanına aktarılması sözleşme kurallarına birebir uyumlu.
Testler ve kabul kriterleri eksiksiz karşılandığı için develop branch'ine birleştirilmeye uygundur.
🛡️ Otomatik Kod ve Güvenlik İncelemesi
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
📌 Özet
Bu PR, hem çevrimdışı eğitimde (offline training) hem de canlı tahminde (online inference) aynı saf fonksiyonları kullanan özellik üretim hattını (
app/features/pipeline.py), çoklu zaman dilimi birleştiricisini (app/features/multiframe.py) ve B06 immutable veri kümesi okuyucusunu (app/infrastructure/market_dataset.py) devreye alır.🛠️ Yapılan Değişiklikler
app/features/pipeline.py):SPECveSCHEMA_HASH(sha256:...) tanımlandı.finalized=True,close_time <= as_ofveavailable_at <= as_ofşartlarını sağlayan son 21 mumu işleyenbuild_featuresfonksiyonu yazıldı.< 21mum ->INSUFFICIENT_HISTORY), zaman boşluğu (!= 60sn ->CANDLE_GAP) ve bayat veri (> max_age_seconds->STALE_CANDLES) kalite ayrımları eklendi.INVALID_CANDLEfırlatan doğrulama ve Backend (B09) içinDecimaltabanlıatrPrice/referencePricestring çıktıları (decimal-atr-sma14-v1) eklendi.asOfve şema özetinden deterministikuuid5FeatureSnapshot.idüretimi sağlandı.app/features/multiframe.py):max(child.available_at) <= as_ofkoşulu sağlanmadan üst zaman dilimi barı üretmeyen birleştirme (resample/join) kuralı eklendi.app/infrastructure/market_dataset.py):dataset_id/snapshot_id/instrument_idbağını doğrulayan adaptör yazıldı.tests/test_core.py):INSUFFICIENT_DATA) ve 21 mum (READY) sınır testleri, gelecekteki fiyat değiştiğinde geçmişas_ofvektörünün sabit kaldığını kanıtlayan sızıntı (lookahead) testi ve train/serve parity kontrolleri eklendi.🧪 Test ve Doğrulama Kanıtları
.\.venv\Scripts\python.exe -m pytest tests -q-> Tüm feature, sınır, boşluk ve parity testleri başarıyla geçti.tanından sonraki mumların fiyatları değiştirildiğindetanına aitFeatureSnapshotvektörünün veiddeğerinin değişmediği doğrulandı.available_at > as_ofolan barların özellik hesabına dahil edilmediği teyit edildi.Bu PR özellik mühendisliğini, zaman sızıntısı kontrollerini ve veri kümesi okuma sınırını kapsar. Etiketleme (labeling), zaman sıralı eğitim/doğrulama bölünmesi (walk-forward / purge / embargo split) bir sonraki görevde (
A03) ele alınacaktır.