procyon-rs / vega_lite_4.rs

rust api for vega-lite v4
Apache License 2.0
26 stars 7 forks source link

Compile Times #14

Open theduke opened 3 years ago

theduke commented 3 years ago

Thanks for this crate!

Sadly due to the large number of types, the crate really bloats compile times a lot.

You could look into using miniserde instead.

As my benchmark referenced in this pull request show, the compile time improvements can be significant.

ThomAub commented 3 years ago

Hello,

Thank you for your interest in the crate. We will look into using miniserde.

What are the current use case for you in using Vega_lite_4.rs ? We are also looking at other ways to reduce compile time and feed back could be appreciated.