Decrease compile time.
avhz opened this issue · comments
Compile time increased quite substantially after adding Polars as a dependency.
Should look into reducing the compile time a bit if possible.
Have made polars crate part of an opt-in feature.
we can remove https://github.com/avhz/RustQuant/blob/634735dd5d860af26f5e31091aca44ee69ea2430/Cargo.toml#L45C16-L45C16
and edit data = ["dep:polars/default"]
Line 56 in 634735d
The way I have it currently reduces the number of dependencies compiled from 400+ to around 270-280.
Are you saying we should remove optional = true and change polars to a default feature ? This would increase the dependencies compiled to 400+ again wouldn't it ?
Ohh My bad we can keep it as true and data = [] ( empty) , the data feature is disabled by default, meaning Polars will not be included in the compilation process unless explicitly enabled using the --features data flag or specifying features = ["data"] in the Cargo.toml file.
But polars is already not included unless data is specified as a feature.
Why do you want to have data = [] instead of data = ["dep:polars"] ?