Simon Schölly

Results 188 comments of 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>...