Fix Starter Code in README

[aaravnavani]

In this PR, I made 2 changes:

1. Remove {gr.xml_prefix_prompt} and ${gr.json_suffix_prompt_v2_wo_none} from the prompt section of the sample RAIL Spec.

2. Change openai.Completion.create to openai.completions.create based on the recent change in the OpenAI API.

[zsimjee]

Thanks for the fix! Did removing the prompt suffixes help get results more consistently? Is that why they were removed?

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (main@8833c05). Click here to learn what that means.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #538   +/-   ##
=======================================
  Coverage        ?   82.78%           
=======================================
  Files           ?       93           
  Lines           ?     5751           
  Branches        ?        0           
=======================================
  Hits            ?     4761           
  Misses          ?      990           
  Partials        ?        0           

Flag	Coverage Δ
unittests	82.78% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[aaravnavani]

@zsimjee When I included the prompt suffixes, I got this response from the LLM:

validated_output {'bank_run': {'explanation': 'stringgggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggg', 'follow_up_url': 'https://example.com'}}

When I removed the prompt suffixes, the response is as expected:

validated_output {'bank_run': {'explanation': "A bank run is when a large number of customers withdraw their money from a bank at the same time, usually due to fears of the bank's financial stability. This can lead to a bank's collapse and cause a ripple effect on the economy.", 'follow_up_url': 'https://www.investopedia.com/terms/b/bankrun.asp'}}

I'm not really sure why that happens, as the {gr.xml_prefix_prompt} and the {gr.json_suffix_prompt_v2_wo_none} are properly set in constants.xml.

[zsimjee]

That’s hilarious!! Davinci 003 used to be our default model and it worked really well with instruction following. I think this might be a model-to-model issue. Sounds good though!

[zsimjee]

We'll merge this one, and I have this as a follow up - #540. I think we could do better than this example, we just need to find a better usage that shows how guardrails is useful.

[zsimjee]

Closing this in favor of #549