Skip to content

Fix current_velocity for EDAC #825

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

Merged
merged 5 commits into from
Jun 11, 2025

Conversation

LasNikas
Copy link
Collaborator

@LasNikas LasNikas commented Jun 7, 2025

No description provided.

@LasNikas LasNikas self-assigned this Jun 7, 2025
@LasNikas LasNikas added bug Something isn't working high priority labels Jun 7, 2025
@LasNikas LasNikas requested a review from efaulhaber June 7, 2025 20:32
@LasNikas LasNikas changed the title Fix `current_velocity´ for EDAC Fix current_velocity for EDAC Jun 7, 2025
Copy link

codecov bot commented Jun 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.63%. Comparing base (114a57b) to head (b4ff8f6).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #825      +/-   ##
==========================================
+ Coverage   70.60%   70.63%   +0.03%     
==========================================
  Files         106      106              
  Lines        6769     6771       +2     
==========================================
+ Hits         4779     4783       +4     
+ Misses       1990     1988       -2     
Flag Coverage Δ
unit 70.63% <100.00%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LasNikas
Copy link
Collaborator Author

LasNikas commented Jun 8, 2025

/run-gpu-tests

@efaulhaber
Copy link
Member

If this is critical, why didn't any of the tests fail? Can you please add a test that would catch such an error if it happens again in the future?

@LasNikas
Copy link
Collaborator Author

If this is critical, why didn't any of the tests fail?

idk. Maybe because this is only used by the ParticleShiftingCallback and system_data

LasNikas added 2 commits June 11, 2025 09:56
@LasNikas LasNikas requested a review from svchb June 11, 2025 08:20
efaulhaber
efaulhaber previously approved these changes Jun 11, 2025
@LasNikas LasNikas enabled auto-merge (squash) June 11, 2025 09:42
@LasNikas LasNikas removed the request for review from svchb June 11, 2025 09:42
@LasNikas
Copy link
Collaborator Author

/run-gpu-tests

@LasNikas LasNikas merged commit f8e3631 into trixi-framework:main Jun 11, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working high priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants