Skip to content

Conversation

@DanielGMorenaFhecor
Copy link
Collaborator

Please, find here all the code implementations for the following specified chapters:

  • Compressive strength (Ch. 14.6.1.1)
  • Tensile stress (Ch. 14.6.1.2)
  • Modulus of elasticity (Ch. 14.6.1.4)
  • Stress-strain relations (Ch. 14.6.1.5)
  • Time effects (Ch. 14.6.1.6)
  • Temperature effects (Ch. 14.6.1.7)

The code is already up to review 👍

mortenengen and others added 30 commits December 13, 2022 21:01
@DanielGMorenaFhecor DanielGMorenaFhecor added enhancement New feature or request MC2020 fib Model Code 2020 labels Aug 27, 2024
@DanielGMorenaFhecor DanielGMorenaFhecor self-assigned this Aug 27, 2024
@DanielGMorenaFhecor DanielGMorenaFhecor linked an issue Aug 27, 2024 that may be closed by this pull request
6 tasks
return 0.20


def sigma_c(
Copy link
Member

Choose a reason for hiding this comment

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

This function should be considered removed in favour of one of our builtin constitutive laws. @talledodiego can you advice on this?


# Dictionary for normal concrete grades and
# their respective fck and fck,cube values
_concrete_strengths = {
Copy link
Member

Choose a reason for hiding this comment

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

I think this is a clever solution when the data is given in a table in the code, but it might be a short-coming when we start implementing methods on the material level, and the characteristic cylinder strength is the expected input and not the strength class. @talledodiego any thoughts on this?



# Dictionary for concrete properties based on grade
concrete_properties = {
Copy link
Member

Choose a reason for hiding this comment

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

Same as above, @talledodiego, any thoughts?

Copy link
Member

@mortenengen mortenengen left a comment

Choose a reason for hiding this comment

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

What a massive PR @DanielGMorenaFhecor! Thanks for the contribution 💪

I have merged dev, fixed docstrings, added the functions to the API docs, left a couple of comments, and asked @talledodiego for a review of compliance with the design code.

@@ -0,0 +1,1363 @@
"""Tests for fib MC2020 Chapter 14.6.1."""
Copy link
Member

Choose a reason for hiding this comment

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

There are several lines that are not covered by tests. Can you please add tests to reach 100% coverage of the contribution?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request MC2020 fib Model Code 2020

Projects

Status: Under review 👀

Development

Successfully merging this pull request may close these issues.

MC2020: Material properties of concrete

2 participants