Simon Schölly
Simon Schölly
Thanks for this. Unfortunately I have to deal with a broken laptop at the moment so it will take a few days until I get to review it.
> Thanks for writing this up. I am strongly in favor. > > 1. One minor UX consistency nit that might need to be spelled out in more detail. What...
2. We don't have `AbstractAlgorithm` anywhere in our code right now as far as I know? The idea was floating around for a while I think. And yes, this might...
3. I don't like having long names all the time - so perhaps even `Execution` might be a better name (but that is not a strong opinion` - there are...
I don't want to be too negative here - but I am not sure if this is something we want right now. My reasons for this: - It will make...
> > Expression: result ⊜ true > > Evaluated: ⟪result: 😭 FAILED: /Users/runner/work/Graphs.jl/Graphs.jl/Project.toml > > The file `/Users/runner/work/Graphs.jl/Graphs.jl/Project.toml` is not in canonical format. > > Looks like 1.6 / 1.9...
I think we are due for bumping the min version of this package anyway so I created a PR: #432 - let's see if someone objects.
> > * It is unclear how much this will increase loading time. > > Wouldn't this strictly decrease loading time? And decrease compilation time as well? It is a...
@Krastanov Thanks, these are some very solid benchmarks. @rafaqz > We should also move Inflate.jl to an extension, it pulls in binary dependencies that most users wont use. [Inflate.jl](https://github.com/GunnarFarneback/Inflate.jl) is...
On the subject of loading time - this issue #244 reminded me that there is a `@time_import` macro to see which package uses how much time when loading. ```julia julia>...