Skip to content

BYOC_ID: 45598 - refactor gp-ooh_top-level#1282

Draft
OluwatobiOni wants to merge 10 commits into
developmentfrom
refactor_gp_ooh_top_level
Draft

BYOC_ID: 45598 - refactor gp-ooh_top-level#1282
OluwatobiOni wants to merge 10 commits into
developmentfrom
refactor_gp_ooh_top_level

Conversation

@OluwatobiOni

@OluwatobiOni OluwatobiOni commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

Opening as a draft PR.

TO-DOs:

  • refactoring still needed for process_extract_gpooh when ooh-costs and ooh cup marker datasets are added to denodo.
  • Placeholder table name to be updated in process_costs_gpooh
  • Confirm if process_costs_gpooh needs pivoting as we are discussing it beong already pivoted in denodo

@OluwatobiOni OluwatobiOni requested review from Jennit07 and lizihao-anu and removed request for lizihao-anu April 7, 2026 17:40
@Jennit07 Jennit07 added On hold Waiting for something / someone outside of our control BYOC dependency missing labels Apr 13, 2026
@Jennit07

Copy link
Copy Markdown
Collaborator

Update: On Hold due to dependencies missing from UAT

@lizihao-anu

Copy link
Copy Markdown
Collaborator

Looks ok. Needs testing when data is available.

@lizihao-anu lizihao-anu changed the title refactor gp-ooh_top-level BYOC_ID: 45598 - refactor gp-ooh_top-level May 11, 2026
@lizihao-anu lizihao-anu linked an issue May 11, 2026 that may be closed by this pull request
7 tasks
@lizihao-anu

Copy link
Copy Markdown
Collaborator

Tobi, could you also update the dummy_targets.R script?

@OluwatobiOni

Copy link
Copy Markdown
Collaborator Author

Tobi, could you also update the dummy_targets.R script?

Thanks, I'll update the branch and also update the dummy_targets.R script @lizihao-anu

@lizihao-anu lizihao-anu left a comment

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.

Looks good. Waiting for test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor GP Out of Hours (OOH)

3 participants