Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Yet more precompiles #551

Merged
merged 1 commit into from
Jan 1, 2021
Merged

Yet more precompiles #551

merged 1 commit into from
Jan 1, 2021

Conversation

timholy
Copy link
Member

@timholy timholy commented Dec 31, 2020

These were identified similarly to #549

These were identified similarly to #549
@timholy
Copy link
Member Author

timholy commented Dec 31, 2020

These forthcoming tools in SnoopCompile are awesome. Time for ProfileView to produce a flamegraph window (as in, "time to first flamegraph"):

  • master: 2.63s
  • upcoming release: 0.94s

There will be a lot of updated packages before that becomes widely available (including a new GtkObservables package).

@codecov
Copy link

codecov bot commented Dec 31, 2020

Codecov Report

Merging #551 (a202b6d) into master (30d49ec) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #551   +/-   ##
=======================================
  Coverage   47.50%   47.51%           
=======================================
  Files          32       32           
  Lines        2368     2376    +8     
=======================================
+ Hits         1125     1129    +4     
- Misses       1243     1247    +4     
Impacted Files Coverage Δ
src/GLib/signals.jl 73.97% <100.00%> (-0.12%) ⬇️
src/precompile.jl 100.00% <100.00%> (ø)
src/events.jl 44.18% <0.00%> (-4.66%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 30d49ec...a202b6d. Read the comment docs.

@tknopp tknopp merged commit 3333628 into master Jan 1, 2021
@timholy timholy deleted the teh/more_precompiles branch January 1, 2021 21:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants