Skip to content

Get sdl helpers#1333

Open
OluwatobiOni wants to merge 5 commits into
developmentfrom
get-sdl-helpers
Open

Get sdl helpers#1333
OluwatobiOni wants to merge 5 commits into
developmentfrom
get-sdl-helpers

Conversation

@OluwatobiOni

Copy link
Copy Markdown
Collaborator

TODOs

LOCAL MODE (get_sdl_processed_data)

  • Verify all functions where BYOC_MODE was added in advance
  • Ensure underlying functions accept BYOC_MODE argument
  • Confirm paths still resolve correctly
  • Remove temporary comments once validated

RAW TABLE FUNCTION (get_sdl_raw_table_name)

  • Replace all dummy table names with actual Denodo tables

BOTH FUNCTIONS

  • Ensure dataset IDs are identical across both functions

@OluwatobiOni OluwatobiOni requested a review from Jennit07 June 1, 2026 14:23
@Jennit07

Jennit07 commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Hi @OluwatobiOni this looks good - i need to test in R again. However the concept should work! I was also thinking that we could use the helper functions with those datasets we have already refactored in BYOC Mode e.g. get_sdl_raw_names("acute") to read the table names. We could update all of them on this branch!

@lizihao-anu lizihao-anu self-requested a review June 16, 2026 09:09
Comment thread R/get_sdl_helpers.R
Comment on lines +127 to +130
"sc_all_alarms_telecare" = "sdl_sc_all_alarms_telecare_processed",
"sc_all_care_homes" = "sdl_sc_all_care_homes_processed",
"sc_all_home_care" = "sdl_sc_all_home_care_processed",
"sc_all_self_directed_support" = "sdl_sc_all_self_directed_support_processed",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better use sc_all_at, sc_all_ch, etc, for consistency?

@lizihao-anu

Copy link
Copy Markdown
Collaborator

Can merge it first and keep building on this function in each individual PR.

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