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

Refactoring constructor of parallel cubed sphere #41

Merged

Conversation

amartinhuertas
Copy link
Collaborator

@amartinhuertas amartinhuertas commented Oct 25, 2024

No description provided.

cell-wise coordinates in the reference space of each panel
Added a new driver for the DarcyCubedSphere that uses conforming
refinement of a patch generated by NestedMeshes py code
Modified RefinedPatchDarcyCubedSphereTests.jl such that it now solves
the Darcy problem on uniformly refined meshes
@amartinhuertas amartinhuertas marked this pull request as draft October 25, 2024 21:43
Moved parsing and previous algorithm to src/
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

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

Project coverage is 0.00%. Comparing base (cf19338) to head (5125cfb).
Report is 17 commits behind head on master.

Files with missing lines Patch % Lines
src/ParseCubedSphereDiscreteModels.jl 0.00% 158 Missing ⚠️
src/mpi/CubedSphereDistributedDiscreteModels.jl 0.00% 20 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #41    +/-   ##
=======================================
  Coverage    0.00%   0.00%            
=======================================
  Files          18      19     +1     
  Lines        1277    1592   +315     
=======================================
- Misses       1277    1592   +315     

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

@amartinhuertas amartinhuertas marked this pull request as ready for review November 8, 2024 07:48
@amartinhuertas amartinhuertas merged commit 6603998 into master Nov 8, 2024
3 of 4 checks passed
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