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

Burgers' equation resolved by FEM #26

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

yuehhua
Copy link
Collaborator

@yuehhua yuehhua commented Nov 24, 2021

Resolves #25

@codecov-commenter
Copy link

codecov-commenter commented Nov 24, 2021

Codecov Report

Merging #26 (61e9ca4) into master (ad6b55e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #26   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines           50        50           
=========================================
  Hits            50        50           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ad6b55e...61e9ca4. Read the comment docs.

@yuehhua
Copy link
Collaborator Author

yuehhua commented Nov 25, 2021

Some benchmark results.

BenchmarkTools.Trial: 4 samples with 1 evaluation.
 Range (min  max):  1.240 s    1.312 s  ┊ GC (min  max): 0.23%  0.13%
 Time  (median):     1.252 s              ┊ GC (median):    0.18%
 Time  (mean ± σ):   1.264 s ± 32.685 ms  ┊ GC (mean ± σ):  0.21% ± 0.11%

  █      █   █                                            █  
  █▁▁▁▁▁▁█▁▁▁█▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁█ ▁
  1.24 s         Histogram: frequency by time        1.31 s <

 Memory estimate: 173.70 MiB, allocs estimate: 38409.

@foldfelis foldfelis changed the base branch from master to main July 7, 2022 13:50
update README and format
@codecov
Copy link

codecov bot commented Jul 17, 2022

Codecov Report

Merging #26 (5e289dd) into main (63f0c58) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #26   +/-   ##
=======================================
  Coverage   95.27%   95.27%           
=======================================
  Files          10       10           
  Lines         148      148           
=======================================
  Hits          141      141           
  Misses          7        7           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@@ -27,3 +27,4 @@ end
 l、゙ ~ヽ
 じしf_, )ノ
=#
# Want some fish?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Want some fish?

using Test

@testset "Burgers_FEM.jl" begin
# Write your tests here.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any test for this?

@KirillZubov
Copy link
Member

would be nice to have a FEM full-time evolution solver for all examples

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.

Benchmarks
4 participants