search

OSIPI / OSIPI_CAPLEX

TF4.2
https://osipi.github.io/OSIPI_CAPLEX/
0 stars 7 forks source link

Notations for Indicator Kinetic Models were corrected and Description added for DPM, THM, AATHM #76

Closed Aditish51 closed 4 months ago

Aditish51 commented 6 months ago

Fixes #73

netlify[bot] commented 6 months ago

Deploy Preview for caplex ready!

Name Link
Latest commit 99c90f85394dd5a0ca8e3b58548a297dff2f8947
Latest deploy log https://app.netlify.com/sites/caplex/deploys/66190310ae2c7d00086a34c0
Deploy Preview https://deploy-preview-76--caplex.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

MRdep commented 6 months ago

Hi @Aditish51, this looks great. A few small changes required before I merge. The case of the terms is quite important, and in some cases Xax and Ve, Vp are written with uppercase instead of lowercase. Also in the AATHM, there is no whitespace between fort> Vp/Fp.

Also for two compartment exchange model, the standard abbreviation is 2CXM.

Aditish51 commented 6 months ago

@MRdep I apologize for the errors . I will correct them asap. Thank you

Aditish51 commented 6 months ago

Hello, @MRdep The changes to the uppercase letters, AAThM and naming 2CXM have been made.

Thank you.