Skip to content

Conversation

@elong0527
Copy link
Collaborator

The issue #87 has been fixed.

Example:

devtools::load_all()

adsl <- forestly::forestly_adsl_3grp[1:100,]
adae <- forestly::forestly_adae_3grp[1:100,]
forestly::meta_forestly(
  dataset_adsl = adsl,
  dataset_adae = adae
) |>
  forestly::prepare_ae_forestly()|>
  forestly::format_ae_forestly(display = c("diff", "fig_diff")) |>
  forestly::ae_forestly()
image
adsl <- forestly::forestly_adsl_3grp[1:100,]
adae <- forestly::forestly_adae_3grp[1:100,]
forestly::meta_forestly(
  dataset_adsl = adsl,
  dataset_adae = adae
) |>
  forestly::prepare_ae_forestly()|>
  forestly::format_ae_forestly(display = c("n", "diff", "fig_diff")) |>
  forestly::ae_forestly()
image
adsl <- forestly::forestly_adsl_3grp[1:100,]
adae <- forestly::forestly_adae_3grp[1:100,]
forestly::meta_forestly(
  dataset_adsl = adsl,
  dataset_adae = adae
) |>
  forestly::prepare_ae_forestly()|>
  forestly::format_ae_forestly(display = c("prop", "diff", "fig_diff")) |>
  forestly::ae_forestly()
image

elong0527 and others added 4 commits August 18, 2025 19:41
🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Reversed y-axis ordering for non-reference groups in prop plots
- Reversed y-axis ordering for diff plots
- Ensures consistent visual ordering between table and interactive plots

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Added Plotly.js integration information
- Documented sparkline_point_js() function
- Added brew and plotly dependencies
- Included notes about JavaScript templates and debugging

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Added display_n and display_prop flags that are set based on the display parameter.
Modified column definitions to include show attribute that respects these flags.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@elong0527 elong0527 requested a review from fukuhiro2023 August 19, 2025 00:38
@wangben718 wangben718 merged commit 4018d2b into main Aug 20, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants