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

Update wording for Palt footnote #327

Open
jwildfire opened this issue Oct 28, 2020 · 2 comments
Open

Update wording for Palt footnote #327

jwildfire opened this issue Oct 28, 2020 · 2 comments
Labels
good first issue Good for newcomers

Comments

@jwildfire
Copy link
Contributor

"PALT shows promise in predicting the percentage hepatocyte loss on the basis of the maximum value and the AUC of serum ALT observed during a DILI event" -> "PALT shows promise in predicting the percentage hepatocyte loss on the basis of the maximum value and the AUC of serum ALT observed."

Also remove the link to the paper and refer to our clinical workflow instead.

@jwildfire jwildfire added the good first issue Good for newcomers label Oct 28, 2020
@MayaGans
Copy link

MayaGans commented Oct 29, 2020

and refer to out clinical workflow instead

I see I should remove "For more details see <a href = "https://www.ncbi.nlm.nih.gov/pubmed/30303523">A Rapid Method to Estimate Hepatocyte Loss Due to Drug-Induced Liver Injury</a> by Chung et al.", would you mind sending me the link to the clinical workflow so I can site that instead?

Also I see this text/footnote lives in both

  • src/callbacks/onPreprocess/flattenData/calculatePalt.js
  • hepexplorer.js

Do I change both?

@jwildfire
Copy link
Contributor Author

jwildfire commented Oct 29, 2020

Various pdf versions of workflow live here; direct link to latest versions is here: https://github.com/SafetyGraphics/SafetyGraphics.github.io/raw/master/guide/HepExplorerWorkflow_v1_2.pdf

As a sidenote: I would like to create a markdown or html version of the workflow at some point, but it's a pretty big document, so not sure if/when that will happen.

As for the code update - you should just need to update the reference in the /src folder and then call npm run build to regenerate hepexplorer.js. Bundle is done with rollup and formatting is with prettier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants