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

Update ad_utils.jl #2313

Merged
merged 1 commit into from
Sep 2, 2024
Merged

Update ad_utils.jl #2313

merged 1 commit into from
Sep 2, 2024

Conversation

yebai
Copy link
Member

@yebai yebai commented Sep 2, 2024

No description provided.

@yebai yebai merged commit 963269b into mhauru/tapir-tests Sep 2, 2024
19 of 25 checks passed
@yebai yebai deleted the yebai-patch-1 branch September 2, 2024 18:09
@yebai yebai mentioned this pull request Sep 2, 2024
Copy link

codecov bot commented Sep 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (2f93d75) to head (f4b44c9).
Report is 2 commits behind head on mhauru/tapir-tests.

❗ There is a different number of reports uploaded between BASE (2f93d75) and HEAD (f4b44c9). Click for more details.

HEAD has 14 uploads less than BASE
Flag BASE (2f93d75) HEAD (f4b44c9)
54 40
Additional details and impacted files
@@                  Coverage Diff                   @@
##           mhauru/tapir-tests   #2313       +/-   ##
======================================================
- Coverage               86.17%   0.00%   -86.18%     
======================================================
  Files                      24      22        -2     
  Lines                    1599    1534       -65     
======================================================
- Hits                     1378       0     -1378     
- Misses                    221    1534     +1313     

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

yebai added a commit that referenced this pull request Sep 3, 2024
* Test with Tapir

* Relax Tapir version bound

* Relax Tapir version bounds more

* Add test/test_utils/ad_utils.jl

* Change how Tapir is installed for tests

* Typo fix

* Turn Tapir's safe mode off

* Use standard AutoReverseDiff constructor

Co-authored-by: Hong Ge <[email protected]>

* Revert back to previous AutoReverseDiff constructor

* modify `setvarinfo`

* fix test error

* fix more error

* fix error

* fix error

* Exclude Tapir from AdvancedHMC tests

Co-authored-by: Will Tebbutt <[email protected]>

* Update ad_utils.jl (#2313)

* Update test/test_utils/ad_utils.jl

* Move code around in ad_utils.jl

* Add a todo note

---------

Co-authored-by: Hong Ge <[email protected]>
Co-authored-by: Xianda Sun <[email protected]>
Co-authored-by: Xianda Sun <[email protected]>
Co-authored-by: Will Tebbutt <[email protected]>
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.

1 participant