fix(clippy): allow module_name_repetitions and integer_division in ML crates

module_name_repetitions: PpoConfig in ppo module is conventional ML naming.
Renaming breaks every import across the workspace.

integer_division: Basis point calculations, batch size math, combinatorial
formulas — truncation is intentional. Float conversion would introduce bugs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
jgrusewski
2026-03-10 11:27:01 +01:00
parent 7ef92983f9
commit 278fd3673f
13 changed files with 30 additions and 0 deletions

View File

@@ -5,6 +5,8 @@
// Workspace lints deny unwrap/expect/indexing at warn level; override to allow in tests only.
#![cfg_attr(test, allow(clippy::unwrap_used, clippy::expect_used, clippy::indexing_slicing))]
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
pub use ml_core::MLError;

View File

@@ -1,3 +1,6 @@
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
//! # Unified Model Weight Persistence System
//!
//! Comprehensive checkpoint system for all AI models

View File

@@ -1,5 +1,7 @@
#![deny(clippy::unwrap_used, clippy::expect_used)]
#![cfg_attr(test, allow(clippy::unwrap_used, clippy::expect_used))]
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
//! # ML Data Validation
//!
//! Data quality validation, cross-validation, and statistical correction

View File

@@ -4,6 +4,9 @@
//! with all 6 components: Double Q-learning, Dueling Networks, Prioritized Experience Replay,
//! Multi-step Learning, Distributional RL (C51), and Noisy Networks.
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
// Re-export shared modules from ml-core for convenience
pub use ml_core::action_space;
pub use ml_core::order_router;

View File

@@ -3,6 +3,8 @@
//! Core feature extractors: technical indicators, price patterns, volume analysis,
//! microstructure proxies, OFI (Order Flow Imbalance), normalization, and more.
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
#![deny(
clippy::unwrap_used,
clippy::expect_used,

View File

@@ -14,6 +14,8 @@
#![deny(clippy::unwrap_used, clippy::expect_used)]
#![cfg_attr(test, allow(clippy::unwrap_used, clippy::expect_used))]
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
#![allow(clippy::doc_markdown)]
#![allow(clippy::float_arithmetic)]
#![allow(clippy::indexing_slicing)]

View File

@@ -30,6 +30,8 @@
#![deny(clippy::unwrap_used, clippy::expect_used)]
#![cfg_attr(test, allow(clippy::unwrap_used, clippy::expect_used))]
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
// Overrides of workspace deny → allow (labeling code uses these patterns)
#![allow(clippy::shadow_reuse, clippy::shadow_same, clippy::shadow_unrelated)]
#![allow(clippy::non_ascii_literal, clippy::str_to_string)]

View File

@@ -5,6 +5,8 @@
#![deny(clippy::unwrap_used, clippy::expect_used)]
#![cfg_attr(test, allow(clippy::unwrap_used, clippy::expect_used))]
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
// Workspace lints deny `panic` and `indexing_slicing` — allow where needed
#![allow(clippy::panic, clippy::indexing_slicing)]

View File

@@ -9,6 +9,9 @@
//! - Reward normalization for numerical stability
//! - Transaction costs and position limits for risk management
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
// Re-export shared modules from ml-core for convenience
pub use ml_core::cuda_compat;

View File

@@ -1,5 +1,7 @@
#![deny(clippy::unwrap_used, clippy::expect_used)]
#![cfg_attr(test, allow(clippy::unwrap_used, clippy::expect_used))]
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
#![allow(dead_code)]
#![allow(missing_docs)]
#![allow(missing_debug_implementations)]

View File

@@ -4,6 +4,9 @@
//! Each model directory contains the core network, layers, and self-contained training logic.
//! Checkpointable impls live in the `checkpoint` module. `UnifiedTrainable` adapters live in `ml`.
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
// Re-export shared types from ml-core
pub use ml_core::cuda_compat;
pub use ml_core::mixed_precision;

View File

@@ -5,6 +5,8 @@
#![deny(clippy::unwrap_used, clippy::expect_used)]
#![cfg_attr(test, allow(clippy::unwrap_used, clippy::expect_used))]
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
// Workspace denies that need local overrides for universe code patterns
#![allow(clippy::indexing_slicing)] // Matrix indexing in correlation/volatility code
#![allow(clippy::shadow_reuse)] // Common in scoring/ranking code

View File

@@ -1,5 +1,7 @@
#![deny(clippy::unwrap_used, clippy::expect_used)]
#![cfg_attr(test, allow(clippy::unwrap_used, clippy::expect_used))]
#![allow(clippy::module_name_repetitions)]
#![allow(clippy::integer_division)]
#![allow(clippy::float_arithmetic)]
#![allow(clippy::doc_markdown)]
#![allow(clippy::missing_const_for_fn)]