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

CST tutorial #151

Merged
merged 7 commits into from
Aug 22, 2022
Merged

CST tutorial #151

merged 7 commits into from
Aug 22, 2022

Conversation

eytanadler
Copy link
Contributor

Purpose

Adds a tutorial showing the basic usage of DVGeometryCST.

Expected time until merged

Now please

Type of change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (non-backwards-compatible fix or feature)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Documentation update
  • Maintenance update
  • Other (please describe)

@eytanadler eytanadler requested review from ewu63 and hajdik August 10, 2022 20:27
@eytanadler eytanadler requested a review from a team as a code owner August 10, 2022 20:27
@eytanadler eytanadler requested a review from joanibal August 10, 2022 20:27
@codecov
Copy link

codecov bot commented Aug 10, 2022

Codecov Report

Merging #151 (286d6c8) into main (8453c44) will decrease coverage by 10.47%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #151       +/-   ##
===========================================
- Coverage   63.74%   53.27%   -10.48%     
===========================================
  Files          47       47               
  Lines       11689    11689               
===========================================
- Hits         7451     6227     -1224     
- Misses       4238     5462     +1224     
Impacted Files Coverage Δ
pygeo/parameterization/DVGeoMulti.py 0.40% <0.00%> (-89.42%) ⬇️
pygeo/constraints/areaConstraint.py 50.50% <0.00%> (-25.42%) ⬇️
pygeo/constraints/DVCon.py 68.04% <0.00%> (-3.69%) ⬇️
pygeo/pyBlock.py 45.37% <0.00%> (-1.65%) ⬇️
pygeo/parameterization/DVGeo.py 64.23% <0.00%> (-0.50%) ⬇️
pygeo/topology.py 54.92% <0.00%> (-0.23%) ⬇️
pygeo/__init__.py 91.30% <0.00%> (+8.69%) ⬆️
pygeo/parameterization/__init__.py 88.88% <0.00%> (+11.11%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@eytanadler eytanadler requested a review from sseraj August 10, 2022 21:10
hajdik
hajdik previously approved these changes Aug 10, 2022
Copy link
Contributor

@hajdik hajdik left a comment

Choose a reason for hiding this comment

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

LGTM

@eytanadler eytanadler removed the request for review from sseraj August 10, 2022 21:22
@ewu63 ewu63 removed their request for review August 13, 2022 14:41
@eytanadler eytanadler requested review from A-CGray and ArshSaja August 19, 2022 14:49
@A-CGray A-CGray requested a review from hajdik August 22, 2022 17:49
@hajdik hajdik merged commit 7212c82 into mdolab:main Aug 22, 2022
@eytanadler eytanadler deleted the cst_tutorial branch August 22, 2022 18:32
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.

3 participants