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

Type tao #40

Merged
merged 3 commits into from
Jul 14, 2023
Merged

Type tao #40

merged 3 commits into from
Jul 14, 2023

Conversation

fsxbhyy
Copy link
Collaborator

@fsxbhyy fsxbhyy commented May 5, 2023

Make Cheb grid available for arbitrary type of Float.

@fsxbhyy fsxbhyy added the enhancement New feature or request label May 5, 2023
@fsxbhyy fsxbhyy requested a review from iintSjds May 5, 2023 02:00
@fsxbhyy fsxbhyy self-assigned this May 5, 2023
@codecov-commenter
Copy link

codecov-commenter commented May 5, 2023

Codecov Report

Merging #40 (7e35116) into dev (6a3d99a) will increase coverage by 11.45%.
The diff coverage is 53.57%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@             Coverage Diff             @@
##              dev      #40       +/-   ##
===========================================
+ Coverage   51.03%   62.48%   +11.45%     
===========================================
  Files          10        9        -1     
  Lines        1354     1141      -213     
===========================================
+ Hits          691      713       +22     
+ Misses        663      428      -235     
Impacted Files Coverage Δ
src/CompositeGrids.jl 100.00% <ø> (ø)
src/grid/chebyshev.jl 72.63% <53.57%> (-7.50%) ⬇️

... and 2 files with indirect coverage changes

@@ -186,39 +244,48 @@ function chebdiff(n, x, f, invmat)
return sum(intw .* f)
end

struct BaryCheb1D{N}
struct BaryCheb1D{N,T}
Copy link
Member

Choose a reason for hiding this comment

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

Shall we add a restriction like T<:AbstractFloat to aviod wrong inputs like Int?

@@ -401,7 +401,7 @@ the grid has 2 segments, 2*(M+1)*N+1 points.
seg = 2
size = (M+1)*N*seg+1
bounds = @SVector[0.0,β]
return UniLogs{Float64,size,seg}(bounds,minterval,M,N)
Copy link
Member

Choose a reason for hiding this comment

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

Why is this file changed? Perhaps this file should be removed.

@kunyuan kunyuan mentioned this pull request Jul 13, 2023
@fsxbhyy fsxbhyy merged commit 67f3cd3 into dev Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants