demand_forecasting: enable time dimension + change-tracking note in predictive READMEs#76
Open
pkouki wants to merge 3 commits into
Open
demand_forecasting: enable time dimension + change-tracking note in predictive READMEs#76pkouki wants to merge 3 commits into
pkouki wants to merge 3 commits into
Conversation
…to predictive READMEs - demand_forecasting.py: enable the temporal GNN path — set time_col on the PropertyTransformer, make train/val/test relationships arity-3 (carry Sale.date), and set has_time_column=True. Fixes the "expect arity=3 for training/validation sets with has_time_column=True" error. - Add a note after `rai init` across the predictive template READMEs (demand_forecasting, fraud-detection, smoker_status_prediction, subscriber_retention, telco_network_recovery) telling users to add `data.ensure_change_tracking: true` to raiconfig.yaml. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
The docs preview for this pull request has been deployed to Vercel!
|
Mirrors the demand_forecasting README — tells users to create FRAUD_DETECTION.EXPERIMENTS and grant permissions to the RAI Native App before running the local demo. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… READMEs - demand_forecasting: drop "worker is not ready to accept jobs" entry - subscriber_retention: drop "Train job stays in QUEUED" entry Both describe transient infra states rather than user-actionable template issues. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Related fixes to the predictive templates, surfaced while running
demand_forecasting.pyend-to-end:demand_forecasting.py— enable the time dimension. The script previously sethas_time_column=Falsewith a stale "SDK limitation" workaround comment, treatingSale.dateas a plain feature. Enabling time properly requires three coordinated changes (otherwise you hitValueError: expect arity=3 for training/validation sets with has_time_column=True):time_col=[Sale.date]on thePropertyTransformer.{Sale} at {Any:timestamp} has {Any:value}) and bindingTrainTable.date/ValTable.date/TestTable.date.has_time_column=Trueon theGNN.Add a
rai initfollow-up note to the predictive template READMEs telling users to add the following to theirraiconfig.yaml:Applied to:
demand_forecasting,fraud-detection,smoker_status_prediction,subscriber_retention,telco_network_recovery.fraud-detectionREADME — add experiment-schema setup DDL. Mirrors thedemand_forecastingREADME: a note under Prerequisites > Access with the SQL to createFRAUD_DETECTION.EXPERIMENTSand grant permissions to the RAI Native App before running the local demo.Remove transient infra troubleshooting entries. These described temporary server-side states rather than user-actionable template issues:
demand_forecasting— "worker is not ready to accept jobs - please retry the job later"subscriber_retention— "Train job stays in QUEUED with no progress"Notes
I reviewed all predictive scripts across these templates for the same time-dimension issue.
demand_forecasting.pywas the only script with the arity mismatch. The other temporal scripts (fraud_detection*,retail_planning*) already use arity-3 relationships correctly, and the non-temporal ones (subscriber_retention,telco_network_recovery,smoker_status_prediction*) correctly leavehas_time_column=False(static node tasks with non-temporal splits).Test plan
python demand_forecasting.pyruns end-to-end withhas_time_column=True(no arity error)python fraud_detection_local.pyruns after the new schema DDL🤖 Generated with Claude Code