-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use concrete phase types #71
Conversation
bors r+ |
bors r- |
Canceled. |
bors r+ |
Build failed: |
7bb0ef3
to
0cdc6c4
Compare
Codecov Report
@@ Coverage Diff @@
## main #71 +/- ##
==========================================
- Coverage 96.84% 96.84% -0.01%
==========================================
Files 7 7
Lines 857 855 -2
==========================================
- Hits 830 828 -2
Misses 27 27
Continue to review full report at Codecov.
|
Bump patch version number Remove ExprTools Try CUDA_VERSION 10.2 Only catch phase_type as type parameter Add pTq constructor analysis, fix 0 alloc case
0cdc6c4
to
4709361
Compare
bors r+ |
93: Prep alleviating latency issues r=charleskawczynski a=charleskawczynski This PR takes a couple steps to tackle some latency issues: - [Avoid UnionAll::DataType in struct](52c9ff0), which was shown to improve inference [here](CliMA/Thermodynamics.jl#71), and a smaller MWE [here](https://gist.github.com/charleskawczynski/61efc7b769e14eab7231e686773b9d0e) - [Allow SparseContainers to work with Arrays](c5c53ee) Co-authored-by: Charles Kawczynski <kawczynski.charles@gmail.com>
This should help with allocations, since we make closures over these types, which are Unionall right now, this PR makes them concrete.