paulremo / NRPMHandbook

Repository to store and track the development on the ADS/Matrisk project for developing a jupyter book of the NRPM handbook.
Other
1 stars 1 forks source link

Miscellaneous chapter review #16

Closed paulremo closed 2 years ago

paulremo commented 2 years ago

This is @paulremo's review of the miscellaneous chapter:

I suggest the following changes/improvements (to be discussed):

This is @MatthiasSchubert's review of the miscellaneous chapter:

MatthiasSchubert commented 2 years ago

Moved comments

csxADS commented 2 years ago

First changes commit d1931c7ae11e758b52e06a8bae640753217462c5

csxADS commented 2 years ago

Statistical data in commit 9ffd2fc05d027515745387b769437df2655b048c

csxADS commented 2 years ago

Added bullet points commit d289f67beb47c8285ab29bc286377ee3c26ae182

csxADS commented 2 years ago

@paulremo Issue with equations using label => numbering go over equation when it's longer than the "normal" width

paulremo commented 2 years ago

Can you give me an example?

csxADS commented 2 years ago

https://nrpmhandbook.reliability.space/system/handbook/reliability_prediction/system_level_modelling_wear_out.html#equation-equation-7-103

csxADS commented 2 years ago

That's why it's not responsive on mobile version

paulremo commented 2 years ago

https://nrpmhandbook.reliability.space/system/handbook/reliability_prediction/system_level_modelling_wear_out.html#equation-equation-7-103

This equation is too long and can be rewritten to something more compact easily. Just change:

R_{k}\left( t \right) = e^{\left\lbrack - \left( \lambda_{k}t + \left( \frac{T_{\text{Ck} - 1} + t}{\alpha_{k}} \right)^{\beta_{k}} \right) \right\rbrack} \bullet e^{\left\lbrack - \left( \sum_{i = 1}^{i = k - 1}{\lambda_{i}T_{i}} \right) \right\rbrack} \bullet e^{\left\lbrack - \left( \sum_{i = 1}^{i = k - 1}\left( \frac{T_{\text{Ci}}}{\alpha_{i}} \right)^{\beta_{i}} \right) \right\rbrack}

to

R_{k}\left( t \right) = e^{\left\lbrack - \left( \lambda_{k}t + \left( \frac{T_{\text{Ck} - 1} + t}{\alpha_{k}} \right)^{\beta_{k}} \right)  - \left( \sum_{i = 1}^{i = k - 1}{\lambda_{i}T_{i}} \right) - \left( \sum_{i = 1}^{i = k - 1}\left( \frac{T_{\text{Ci}}}{\alpha_{i}} \right)^{\beta_{i}} \right) \right\rbrack}

The product of exponentials is equal to the exponential of the sum of their arguments. In this case, I would actually not use the e^{x} notation, but opt for \exp(x) because the argument (x in this case) becomes very hard to read otherwise.

csxADS commented 2 years ago

Ok for this one but other equations are affected, and when we check the mobile version, almost all equations are displayed the wrong way. I don't think we can modify each formula

csxADS commented 2 years ago

@paulremo For information \dot gives image

which is not the same as \bullet

csxADS commented 2 years ago

I found that I should use \cdot

csxADS commented 2 years ago

@paulremo issue with url in bibtex library => displayed if we add "\usepackage{url}" to Bibtex style but I don't know where to find it

csxADS commented 2 years ago

Almost everything done, miss the table 5.3 but not sure it's possible

paulremo commented 2 years ago

The issue pertains to the table on this page.

csxADS commented 2 years ago

DONE