docs: simplify UpdateElementStyle function calls in diagrams #52
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ment - Updated the dfda-cost-benefit-analysis.md file to ensure consistent formatting of monetary values and improve clarity throughout the document. - Adjusted various sections to maintain uniformity in the presentation of cost estimates, enhancing readability and comprehension for stakeholders. - These changes aim to provide a clearer understanding of the financial implications associated with the dFDA initiative.
- Introduced a new markdown document detailing the systemic issues preventing patient access to Alzheimer's clinical trials, highlighting the disconnect between available treatments and trial participation. - The document outlines key barriers such as geographic limitations, strict eligibility criteria, and bureaucratic inefficiencies, supported by quantitative data on patient access. - Emphasizes the need for a decentralized approach to clinical trials to improve patient enrollment and expedite treatment availability, aligning with the dFDA initiative's goals.
fixed latex equations
…entation - Updated the recovery-trial.md file to enhance the formatting of hyperlinks for cost per patient and other key statistics, ensuring a more cohesive presentation of information. - Clarified cost efficiency comparisons by providing authoritative sources for cost estimates, emphasizing the trial's dramatic cost-effectiveness relative to traditional clinical trials. - These changes aim to improve readability and support the dFDA initiative's goal of transparent and accessible clinical trial data.
fix: Improve hyperlink formatting and clarity in recovery trial
…ments and remove outdated sections
…e and efficiency benchmarks
docs: enhance referral rewards system documentation
$VICTORY tokens fundraising strategy documentation
…icipation subsidy with algorithmic determination of subsidies
…nhanced citations and formatting for better readability
…nhanced citations and formatting for better readability
Contributor
Reviewer's GuideThis PR streamlines documentation by standardizing link and citation formatting, refining markdown tables in home.md, and simplifying UpdateElementStyle calls to positional arguments in diagram definitions. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Voting is closed Vote Summary: Acceptance Criteria:
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Hey @mikepsinn - I've reviewed your changes - here's some feedback:
- Ensure that all new reference-style link IDs in home.md are unique and correctly defined to avoid broken links.
- Consider moving the large citations block into its own file (e.g., CITATIONS.md) to keep home.md more focused and maintainable.
- Double-check that the simplified UpdateElementStyle positional arguments in docs/diagrams/README.md match the diagram tool’s expected signature so diagrams render correctly.
Here's what I looked at during the review
- 🟡 General issues: 2 issues found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
home.md
Outdated
| > | ||
| > -- <cite>[Deloitte via Fierce Biotech (2024)](https://www.fiercebiotech.com/biotech/drug-development-cost-pharma-22b-asset-2024-plus-how-glp-1s-impact-roi-deloitte)</cite> | ||
| 3. **Claim:** 21,000 to 120,000 more people are being dying each decade due to suboptimal regulatory processes. |
Contributor
There was a problem hiding this comment.
issue (typo): Grammatical error in claim text.
Change "are being dying" to "are dying" or "die" for correct grammar.
|
|
||
| 1. **Claim:** ~\$43,000 cost per participant in traditional clinical trials. | ||
| > "Median per‑patient cost for a phase‑3 drug trial in 2024 exceeded **\$43,000**" | ||
| > |
Contributor
There was a problem hiding this comment.
issue: Missing quote for citation 3.
Please add the missing quote content for citation 3 to complete the blockquote.
…md for clarity and accuracy
…md for clarity and accuracy
|
Voting is closed Vote Summary: Acceptance Criteria:
|
… improved sourcing, including revised citations and new sourcing standards
… improved sourcing, including revised citations and new sourcing standards
|
Voting is closed Vote Summary: Acceptance Criteria:
|
…fining cost estimates and outlining broader initiative expenses
…fining cost estimates and outlining broader initiative expenses
|
Voting is closed Vote Summary: Acceptance Criteria:
|
…s and add new open-ecosystem-and-bounty-model.md to outline strategy for sustainable development
…s and add new open-ecosystem-and-bounty-model.md to outline strategy for sustainable development
|
Voting is closed Vote Summary: Acceptance Criteria:
|
|
Voting is closed Vote Summary: Acceptance Criteria:
|
…aths and chemical additives
|
Current Voting Result Vote Summary: Acceptance Criteria:
|
mikepsinn
added a commit
to mikepsinn/dfda-wiki
that referenced
this pull request
Jun 24, 2025
docs: simplify UpdateElementStyle function calls in diagrams
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary by Sourcery
Refactor documentation to replace inline links with reference-style citations and tables in home.md and simplify diagram styling calls in docs/diagrams/README.md.
Enhancements:
Documentation: