Skip to content

Conversation

@mohdibntarek
Copy link
Member

I had to give up on Zygote-over-Zygote for this. I used Zygote over FiniteDifferences instead and hacked around a few things to make it work. It works now.

@mohdibntarek
Copy link
Member Author

Needs TopOpt master which I will release as soon CI passes on master.

@codecov-commenter
Copy link

Codecov Report

Merging #18 (a221f11) into master (95db7da) will decrease coverage by 7.03%.
The diff coverage is 76.19%.

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
- Coverage   94.33%   87.30%   -7.04%     
==========================================
  Files           1        1              
  Lines          53       63      +10     
==========================================
+ Hits           50       55       +5     
- Misses          3        8       +5     
Impacted Files Coverage Δ
src/ReliabilityOptimization.jl 87.30% <76.19%> (-7.04%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mohdibntarek mohdibntarek merged commit b237603 into master Dec 12, 2022
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.

4 participants