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

Added New Features #17

Merged
merged 7 commits into from
Jun 28, 2024
Merged

Added New Features #17

merged 7 commits into from
Jun 28, 2024

Conversation

gha3mi
Copy link
Owner

@gha3mi gha3mi commented Jun 28, 2024

  • Added generic get_nc() method to nurbs_surface and nurbs_volume derived types.
  • Added det, inv, dyad and gauss_leg procedures in the forcad_utils module.
  • Added procedure set1a to the nurbs_curve derived type.
  • Added procedure set4 to nurbs_curve, nurbs_area and nurbs_volume derived types.
  • Added optional input variable elem to derivative_scalar procedures.
  • Added ansatz procedures to compute shape functions, derivatives of shape functions and (dV, dA, dL).
  • Added cmp_length() to compute the length of a NURBS curve.
  • Added cmp_area() to compute the area of a NURBS surface.
  • Added cmp_volume() to compute the volume of a NURBS volume.
  • Added examples for cmp_length(), cmp_area(), and cmp_volume().

- Remove unused variables in fdm_curve.f90, fdm_surface.f90 and fdm_volume.f90.

Signed-off-by: Seyed Ali Ghasemi <info@gha3mi.com>
Signed-off-by: Seyed Ali Ghasemi <info@gha3mi.com>
- Added `det`, `inv`, `dyad` and `gauss_leg` procedures in the `forcad_utils` module.
- Added procedure `set1a` to the `nurbs_curve` derived type.
- Added procedure `set4` to the `nurbs_curve` NURBS derived type.
- Added optional input variable `elem` to `derivative_scalar` procedures.
- Added `ansatz` procedures to compute shape functions, derivatives of shape functions and (dV, dA, dL).
- Added `cmp_length()` to compute the length of a NURBS curve.
- Added `cmp_area()` to compute the area of a NURBS surface.
- Added `cmp_volume()` to compute the volume of a NURBS volume.
- Added examples for `cmp_length()`, `cmp_area()`, and `cmp_volume()`.

Signed-off-by: Seyed Ali Ghasemi <info@gha3mi.com>
Signed-off-by: Seyed Ali Ghasemi <info@gha3mi.com>
Signed-off-by: Seyed Ali Ghasemi <info@gha3mi.com>
Signed-off-by: Seyed Ali Ghasemi <info@gha3mi.com>
@gha3mi gha3mi added the enhancement New feature or request label Jun 28, 2024
@gha3mi gha3mi self-assigned this Jun 28, 2024
Signed-off-by: Seyed Ali Ghasemi <info@gha3mi.com>
@codecov-commenter
Copy link

codecov-commenter commented Jun 28, 2024

Codecov Report

Attention: Patch coverage is 0.74627% with 399 lines in your changes missing coverage. Please review.

Project coverage is 23.84%. Comparing base (d7f1f08) to head (a48d7df).

Current head a48d7df differs from pull request most recent head 7455eff

Please upload reports for the commit 7455eff to get more accurate results.

Files Patch % Lines
src/forcad_nurbs_volume.f90 0.00% 112 Missing ⚠️
src/forcad_nurbs_surface.f90 0.00% 96 Missing ⚠️
src/forcad_nurbs_curve.f90 0.00% 86 Missing ⚠️
src/forcad_utils.f90 0.00% 70 Missing ⚠️
example/cmp_volume.f90 0.00% 15 Missing ⚠️
example/cmp_area.f90 0.00% 11 Missing ⚠️
example/cmp_length.f90 0.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
- Coverage   25.73%   23.84%   -1.90%     
==========================================
  Files          36       39       +3     
  Lines        4177     4505     +328     
  Branches      346      377      +31     
==========================================
- Hits         1075     1074       -1     
- Misses       3033     3362     +329     
  Partials       69       69              

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

@gha3mi gha3mi merged commit d9a6d11 into main Jun 28, 2024
12 checks passed
@gha3mi gha3mi deleted the dev branch June 28, 2024 14:52
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.

2 participants