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

fix: minor load time patch + remove UnrolledUtilities #75

Merged
merged 2 commits into from
Sep 11, 2024
Merged

Conversation

avik-pal
Copy link
Member

UU broke Enzyme + Lux CUDA support (not sure if Enzyme is to blame or UU but the problem came from an UU update). I will remove uses of UU for now to make things happy.

Copy link

codecov bot commented Sep 11, 2024

Codecov Report

Attention: Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Project coverage is 39.60%. Comparing base (7d2776e) to head (7361845).

Files with missing lines Patch % Lines
src/public.jl 66.66% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (7d2776e) and HEAD (7361845). Click for more details.

HEAD has 2 uploads less than BASE
Flag BASE (7d2776e) HEAD (7361845)
6 4
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #75       +/-   ##
===========================================
- Coverage   82.38%   39.60%   -42.79%     
===========================================
  Files          17       18        +1     
  Lines         352      351        -1     
===========================================
- Hits          290      139      -151     
- Misses         62      212      +150     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

src/internal.jl Outdated Show resolved Hide resolved
@avik-pal avik-pal merged commit 448337d into main Sep 11, 2024
14 of 22 checks passed
@avik-pal avik-pal deleted the ap/remove_uu branch September 11, 2024 16:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant