Skip to content

fix error on logger_plot#185

Closed
marinlauber wants to merge 1 commit intoWaterLily-jl:masterfrom
marinlauber:update-logger
Closed

fix error on logger_plot#185
marinlauber wants to merge 1 commit intoWaterLily-jl:masterfrom
marinlauber:update-logger

Conversation

@marinlauber
Copy link
Copy Markdown
Member

Fix a mistake in the logger plotting script and add a case for logging with BiotSavartBCs, where the number of BiotSavartBCs evaluations nᵇ is also shown.

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 13, 2024

Codecov Report

Attention: Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
ext/WaterLilyPlotsExt.jl 0.00% 8 Missing ⚠️
Files with missing lines Coverage Δ
ext/WaterLilyPlotsExt.jl 0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes

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

@marinlauber marinlauber changed the title add BiotSavart compatilibilty and fix error fix error on logger_plot Feb 12, 2025
@marinlauber
Copy link
Copy Markdown
Member Author

This is breaking due to the codec; it's a plotting helper that is not in runtest.jl.

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