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 'Allow Optim.Options to pass to BAT.find_mode' #439

Closed
wants to merge 1 commit into from
Closed

Update 'Allow Optim.Options to pass to BAT.find_mode' #439

wants to merge 1 commit into from

Conversation

Micki-D
Copy link
Collaborator

@Micki-D Micki-D commented May 2, 2024

This updates PR #423 to the current main branch.

@Micki-D
Copy link
Collaborator Author

Micki-D commented May 2, 2024

I created a new PR, since I thought this would be cleaner than changing PR #423. Let me know if I should do it the other way.

Copy link

codecov bot commented May 2, 2024

Codecov Report

Attention: Patch coverage is 76.31579% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 47.41%. Comparing base (67ac2bf) to head (dc9112e).

Files Patch % Lines
ext/BATOptimizationExt.jl 69.23% 8 Missing ⚠️
ext/BATOptimExt.jl 91.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #439      +/-   ##
==========================================
+ Coverage   47.26%   47.41%   +0.15%     
==========================================
  Files         111      112       +1     
  Lines        5385     5418      +33     
==========================================
+ Hits         2545     2569      +24     
- Misses       2840     2849       +9     

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

@oschulz
Copy link
Member

oschulz commented May 2, 2024

I created a new PR,

That's fine!

@Micki-D
Copy link
Collaborator Author

Micki-D commented May 4, 2024

I am confused about the Julia 1.6-ubuntu-latest tests failing.
In the compilation of the environment, it gives a warning that the exact package versions could not be loaded, and indeed, when comparing the compiled package versions to the other test runs that pass, the failing one has different versions.
Most are lower versions, but there are some higher versions than in the passing test.

Could there be some issue with the job setup?

@oschulz
Copy link
Member

oschulz commented May 20, 2024

@Micki-D can you try to run tests manually on v1.6?

@Micki-D Micki-D closed this by deleting the head repository May 26, 2024
@oschulz
Copy link
Member

oschulz commented May 26, 2024

Why did you close this one @Micki-D ?

@Micki-D
Copy link
Collaborator Author

Micki-D commented May 27, 2024

Ah beans. I did not mean to do so. I'll reopen this @oschulz .

@Micki-D Micki-D mentioned this pull request Jun 3, 2024
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.

2 participants