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

Add correction factors for pressure-based Green function #90

Merged
merged 4 commits into from
Jan 18, 2024

Conversation

RShaw026
Copy link
Contributor

Description and Context

In PR #87, The pressure-based Green function was added. In this PR, the correction factors corresponding to the pressure-based Green function are added.
These values are calculated by using MIRCO to solve a contact between a linear elastic half-plane and a rigid flat-indenter.

Related Issues and Pull Requests

How Has This Been Tested?

ctest is passing

Checklist

  • My commit messages mention the appropriate GitHub issue numbers and are longer than one line where appropriate.
  • I have added/updated documentation where necessary.

Additional Information

Interested Parties / Possible Reviewers

@isteinbrecher @mayrmt

@RShaw026 RShaw026 self-assigned this Jan 18, 2024
isteinbrecher
isteinbrecher previously approved these changes Jan 18, 2024
// The following pressure based constants are calculated by solving a flat indentor problem using
// the pressure based Green function described in Pohrt and Li (2014).
// http://dx.doi.org/10.1134/s1029959914040109
std::vector<double> alpha_con_pressure{0.961389237917602, 0.924715342432435, 0.899837531880697,

Choose a reason for hiding this comment

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

What does alpha_con stand for?

Also, you can make these vectors const.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This variable name has not been changed since the beginning. The Greek letter alpha is used in Section 3.3 of https://doi.org/10.1007/s00466-019-01791-3 to denote this variable. These are called "shape factors". Once we have the shape factors, the Elastic compliance correction can be calculated as done in line 89 of the mirco_set_parameters.cpp. I will rename this variable for a better readability of the code.

Copy link
Member

@mayrmt mayrmt left a comment

Choose a reason for hiding this comment

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

@RShaw026 These factors are constant, so you could make them constant in code as well.

Am I right, that each vector entry corresponds to a certain resolution? Where is it documented, which resolutions these vectors cover?

// The following pressure based constants are calculated by solving a flat indentor problem using
// the pressure based Green function described in Pohrt and Li (2014).
// http://dx.doi.org/10.1134/s1029959914040109
std::vector<double> alpha_con_pressure{0.961389237917602, 0.924715342432435, 0.899837531880697,
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
std::vector<double> alpha_con_pressure{0.961389237917602, 0.924715342432435, 0.899837531880697,
const std::vector<double> alpha_con_pressure{0.961389237917602, 0.924715342432435, 0.899837531880697,

0.882669916668780};
// The following force based constants are taken from Table 1 of Bonari et al. (2020).
// https://doi.org/10.1007/s00466-019-01791-3
std::vector<double> alpha_con_force{0.778958541513360, 0.805513388666376, 0.826126871395416,
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
std::vector<double> alpha_con_force{0.778958541513360, 0.805513388666376, 0.826126871395416,
const std::vector<double> alpha_con_force{0.778958541513360, 0.805513388666376, 0.826126871395416,

@RShaw026
Copy link
Contributor Author

@isteinbrecher @mayrmt FYI Since MIRCO currently consumes a lot of memory and time to solve high-resolution problems with large contact area percentages (in this case: flat indenters), our MATLAB-based bem-code is used to calculate these factors.

@RShaw026
Copy link
Contributor Author

@RShaw026 These factors are constant, so you could make them constant in code as well.

Am I right, that each vector entry corresponds to a certain resolution? Where is it documented, which resolutions these vectors cover?

Yes, you are right. It is not mentioned here, but currently, MIRCO works only from the resolution of 1 to 8. (2^n+1 times 2^n+1 nodes grid)

isteinbrecher
isteinbrecher previously approved these changes Jan 18, 2024
// These are the shape factors to calculate the elastic compliance correction of the micro-scale
// contact constitutive law for various resolutions.
// NOTE: Currently MIRCO works for resouluion of 1 to 8. The following vectors store the shape
// factors for resolution of 1 to 8.

Choose a reason for hiding this comment

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

Alternatively you could use a map here, something like

const std::map<int, double> shape_factors_presure{{1,0.961389237917602},{2,0.924715342432435},...};

Then you can access the data with shape_factors_pressure[Resolution] instead of shape_factors_pressure[Resolution - 1], i.e., you map a resolution value to a pressure.

Copy link
Member

Choose a reason for hiding this comment

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

I like the map proposal. It makes the assignment of resolution to the value very explicit.

@RShaw026 RShaw026 merged commit 8a8ae9c into master Jan 18, 2024
2 checks passed
@RShaw026 RShaw026 deleted the add-new-corr-factors branch January 18, 2024 16:47
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