Skip to content
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

Import needs_primal from Reactant.jl #2021

Merged
merged 8 commits into from
Oct 30, 2024
Merged

Conversation

mofeing
Copy link
Contributor

@mofeing mofeing commented Oct 28, 2024

needs_primal(Mode) returns true if the mode needs to compute the primal.

CC @wsmoses

@codecov-commenter
Copy link

codecov-commenter commented Oct 28, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.49%. Comparing base (037dfed) to head (053f627).
Report is 190 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2021      +/-   ##
==========================================
+ Coverage   67.50%   70.49%   +2.98%     
==========================================
  Files          31       42      +11     
  Lines       12668    15247    +2579     
==========================================
+ Hits         8552    10748    +2196     
- Misses       4116     4499     +383     

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

Copy link
Member

@wsmoses wsmoses left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm but can you bump the version toml for EnzymeCore? [and add compat for Enzyme.jl itself]

test/runtests.jl Outdated Show resolved Hide resolved
@@ -35,7 +35,7 @@ EnzymeStaticArraysExt = "StaticArrays"
BFloat16s = "0.2, 0.3, 0.4, 0.5"
CEnum = "0.4, 0.5"
ChainRulesCore = "1"
EnzymeCore = "0.8.4"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can the compat be set to either 0.8.4 or 0.8.5 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you mean to support both versions? we can put the ~ specifier in the version

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yup so "0.8.4, 0.8.5"

@wsmoses wsmoses merged commit 26ca6fe into EnzymeAD:main Oct 30, 2024
11 of 27 checks passed
@mofeing mofeing deleted the needs_primal branch October 30, 2024 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants