Features for working with synthetitc data in sandbox - #13
Conversation
Accept the verified AMS 'Deleted <event-id>' response only when it matches the requested ID. Preserve fail-closed handling for ambiguous responses and document the connector/caller responsibility boundary.
…etic Data Pipeline - Created quickstart guide detailing the operational pattern, prerequisites, and local workflow for the planned incremental pipeline. - Developed a comprehensive research document outlining key decisions and rationale for the incremental synthetic data pipeline. - Established a feature specification document defining user scenarios, functional requirements, and success criteria for the pipeline.
…e-coders can use it effectively too
|
In the other branch we discussed whether a more detailed confirmation with a summary was desirable. I had already implemented it in this to-be-merged branch. I propose to leave it in, experience it and later decided to either keep or drop it. Feel free to close this comment and -if the rest looks good- proceed to merge. |
|
|
||
| - Version 1 targets explicitly mapped event forms in a verified Smartabase sandbox and excludes Athlete Profile. | ||
| - The desired synthetic timeline is append-only. Consumers can select a rolling 90-day view without scheduled deletion of older events. | ||
| - A non-interactive schedule prepares plans but does not perform live AMS mutation under the current constitution and risk model. |
There was a problem hiding this comment.
This part seems in contradiction with the --auto mode. --auto mode was designed to accept certain risks when approved by a human. If we do not explicitly mention the exception for --auto, this might cause some issues in the actual usage of the API, I think. What is your thought on this @Rens88
|
|
||
| Rules: | ||
|
|
||
| - Requires a real interactive terminal. Piped input, Databricks job parameters, |
There was a problem hiding this comment.
I believe we should avoid the mentioning of specific tools such as databricks and the way databricks works. Tomorrow it could be Fabric, Snowflake etc. I think we should keep the API as clean as possible. I feel that maybe we are complicating things too much. Do we need sandbox, databricks job parameter checks, if we already built in the different modes. Maybe we should discuss the design choises to make sure our ideas are fully aligned? @nielsd01 @Rens88
TeamNL1
left a comment
There was a problem hiding this comment.
I left a two notes in the reviews of the adjusted files. I also scheduled a meeting about the direction and designchoises of the API to make sure we al have the same idea about how the product should work. I believe it would be good to discuss our development strategy for the next phases.
PS: I was able to extract data with the API from the AMS sandbox and load it directly into the Azure Fabric platform, which shows the basics of the API are working!🥳
The combined commits were necessary for me to get the code working with my ams synthetic data use case. It involves deletion, uploading and adding a new day of synthetic data.
Have a critical look at what part belongs in the ams-api repo and which parts may be better stored in the synthetic-data repo.
Written by Codex:
Summary
Safety
Daily generation and scheduled execution remain read-only with respect to AMS. Live publication is a separate interactive sandbox-only workflow that requires an exact immutable plan, a fresh TTY-based typed confirmation, bounded event inserts, single-use authorization, and immediate read-only reconciliation.
Unexpected, incomplete, ambiguous, partial, or unknown states fail closed. No profile upserts, updates, overwrites, deletions, automatic mutation retries, unattended confirmations, or production operations are introduced.
Validation
git diff --checkpassedGovernance
Applicable aims: B2, B4, B6–B8, C1–C3, C5–C8, E1, E4, E6–E9, F3, F5–F8.
Highest applicable risk: High.