Skip to content

Commit 15162bd

Browse files
HamadaSalhabHamadaSalhab
authored
Merge pull request #1324 from julep-ai/x/cookook-fixes
fix(cookbooks): Fix regressed issues in cookbooks
2 parents 12b7283 + 29c7eeb commit 15162bd

File tree

4 files changed

+626
-698
lines changed

4 files changed

+626
-698
lines changed

‎cookbooks/01-website-crawler.ipynb

Lines changed: 49 additions & 124 deletions
Large diffs are not rendered by default.

‎cookbooks/02-sarcastic-news-headline-generator.ipynb

Lines changed: 69 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@
109109
},
110110
{
111111
"cell_type": "code",
112-
"execution_count": 30,
112+
"execution_count": 1,
113113
"metadata": {},
114114
"outputs": [],
115115
"source": [
@@ -130,7 +130,7 @@
130130
},
131131
{
132132
"cell_type": "code",
133-
"execution_count": 31,
133+
"execution_count": 2,
134134
"metadata": {},
135135
"outputs": [],
136136
"source": [
@@ -152,14 +152,14 @@
152152
},
153153
{
154154
"cell_type": "code",
155-
"execution_count": 32,
155+
"execution_count": 3,
156156
"metadata": {},
157157
"outputs": [],
158158
"source": [
159159
"from julep import Client\n",
160160
"import os\n",
161161
"\n",
162-
"JULEP_API_KEY = \"YOUR_JULEP_API_KEY\"\n",
162+
"JULEP_API_KEY = os.environ['JULEP_API_KEY']\n",
163163
"\n",
164164
"# Create a client\n",
165165
"client = Client(api_key=JULEP_API_KEY, environment=\"production\")"
@@ -178,7 +178,7 @@
178178
},
179179
{
180180
"cell_type": "code",
181-
"execution_count": 33,
181+
"execution_count": 4,
182182
"metadata": {},
183183
"outputs": [],
184184
"source": [
@@ -210,7 +210,7 @@
210210
},
211211
{
212212
"cell_type": "code",
213-
"execution_count": 34,
213+
"execution_count": 5,
214214
"metadata": {},
215215
"outputs": [],
216216
"source": [
@@ -246,7 +246,7 @@
246246
" integration:\n",
247247
" provider: brave\n",
248248
" setup:\n",
249-
" api_key: \"{BRAVE_API_KEY}\"\n",
249+
" brave_api_key: \"{BRAVE_API_KEY}\"\n",
250250
"\n",
251251
"########################################################\n",
252252
"####################### MAIN WORKFLOW ##################\n",
@@ -299,7 +299,7 @@
299299
},
300300
{
301301
"cell_type": "code",
302-
"execution_count": 35,
302+
"execution_count": 6,
303303
"metadata": {},
304304
"outputs": [],
305305
"source": [
@@ -324,7 +324,7 @@
324324
},
325325
{
326326
"cell_type": "code",
327-
"execution_count": 36,
327+
"execution_count": 7,
328328
"metadata": {},
329329
"outputs": [],
330330
"source": [
@@ -360,72 +360,81 @@
360360
},
361361
{
362362
"cell_type": "code",
363-
"execution_count": 51,
363+
"execution_count": 8,
364364
"metadata": {},
365365
"outputs": [
366366
{
367367
"name": "stdout",
368368
"output_type": "stream",
369369
"text": [
370-
"Based on the recent news about Tottenham Hotspur, let me craft a sarcastic headline for you:\n",
370+
"\"Genius Billionaire Desperately Attempts Comedy, Proves Money Can't Buy Sense of Humor\"\n",
371371
"\n",
372-
"\"BREAKING: Tottenham Fans Shocked to Discover Their Team's Trophy Cabinet Still Has Same Amount of Dust as Last Season\"\n",
372+
"\"Breaking: World's Richest Man Still Can't Figure Out How to Make People Genuinely Like Him\"\n",
373373
"\n",
374-
"Alternative sarcastic headlines:\n",
374+
"\"Shocking Report: Man Who Named Child 'X Æ A-12' May Not Be Most Stable CEO\"\n",
375375
"\n",
376-
"\"EXCLUSIVE: Spurs Players Perfect the Art of 'Almost Winning' in Revolutionary Training Session\"\n",
376+
"\"Former Trump Officials Suggest Drug Tests for Musk; Musk Agrees, Proving He Might Actually Need One\"\n",
377377
"\n",
378-
"\"REPORT: Scientific Study Confirms Supporting Tottenham Linked to Increased Heart Medication Sales\"\n",
378+
"\"Billionaire Space Enthusiast Too Busy Being 'Funny' on Twitter to Notice Everyone's Cringing\"\n",
379379
"\n",
380-
"\"SHOCKING: Tottenham's GPS Data Reveals Players Expertly Avoiding Trophy-Winning Positions\"\n",
380+
"\"Studies Show Being Worth $180 Billion Still Won't Make Your Memes Any Better\"\n",
381381
"\n",
382-
"These headlines play on Tottenham's notorious reputation for coming close but not quite winning major trophies, despite being one of England's biggest clubs. They maintain their status as every neutral's favorite \"almost there\" team!\n",
382+
"\"Man Who Could End World Hunger Chooses to Post Dad Jokes on Social Media Instead\"\n",
383383
"\n",
384-
"Would you like me to generate more sarcastic headlines about any specific aspect of Tottenham Hotspur?\n"
384+
"\"Breaking: Local Attention-Seeking Billionaire Still Seeking Attention, More at 11\"\n"
385385
]
386386
}
387387
],
388388
"source": [
389-
"# Get execution details\n",
389+
"import time\n",
390+
"\n",
391+
"execution = client.executions.get(execution.id)\n",
392+
"\n",
393+
"while execution.status != \"succeeded\":\n",
394+
" time.sleep(5)\n",
395+
" execution = client.executions.get(execution.id)\n",
396+
" print(\"Execution status: \", execution.status)\n",
397+
" print(\"-\"*50)\n",
398+
"\n",
390399
"execution = client.executions.get(execution.id)\n",
391-
"# Print the output\n",
400+
"\n",
392401
"print(execution.output)"
393402
]
394403
},
395404
{
396405
"cell_type": "code",
397-
"execution_count": 48,
406+
"execution_count": 9,
398407
"metadata": {},
399408
"outputs": [
400409
{
401410
"name": "stdout",
402411
"output_type": "stream",
403412
"text": [
404413
"Transition type: init\n",
405-
"Transition output: {'topic': 'Tottenham Hotspur'}\n",
414+
"Transition output: {'topic': 'Elon Musk'}\n",
406415
"--------------------------------------------------\n",
407416
"Transition type: step\n",
408-
"Transition output: {'result': [{'link': 'https://www.tottenhamhotspur.com/news/', 'title': 'News | Tottenham Hotspur', 'snippet': 'Latest official <strong>news</strong> and video from <strong>Tottenham</strong> <strong>Hotspur</strong>'}, {'link': 'https://www.football.london/tottenham-hotspur-fc/', 'title': 'Tottenham Hotspur FC - latest news, pictures, video comment - Football.london', 'snippet': 'The latest <strong>Tottenham</strong> <strong>news</strong>, transfers, fixtures and more. Including Live blogs, pictures, video, podcasts, polls and indepth analysis from our dedicated Spurs writers.'}, {'link': 'https://www.nytimes.com/athletic/football/team/tottenham-hotspur/', 'title': 'Tottenham Hotspur - News, Schedule, Scores, Roster, and Stats - The Athletic', 'snippet': 'Breaking <strong>Tottenham</strong> <strong>Hotspur</strong> <strong>news</strong> and in-depth analysis from the best newsroom in sports. Follow your favorite clubs. Get the latest injury updates, player <strong>news</strong> and more from around the league.'}]}\n",
417+
"Transition output: {'result': [{'link': 'https://9gag.com/tag/elon-musk', 'title': 'Best Elon Musk Memes and Images - 9GAG', 'snippet': 'Enjoy the best of new <strong>funny</strong> cursed meme images about <strong>Elon</strong> <strong>Musk</strong>, Twitter, $8 verified badge, Tesla, the Boring Company and more on 9GAG. Never run out of hilarious cursed memes to share.'}, {'link': 'https://www.youtube.com/watch?v=ukxf9AqLsPY', 'title': 'Elon FINALLY Gets the HUMILIATION He Deserves in Front of MILLIONS - YouTube', 'snippet': 'Comedian Bill Burr roasts <strong>Elon</strong> <strong>Musk</strong>, MAGA melts down over SNL, and protesters hit Trump where it hurts—his billionaire bestie&#x27;s wallet. Gabe Sanchez reports ...'}, {'link': 'https://gizmodo.com/elon-musk-is-annoying-unfunny-and-should-probably-take-a-drug-test-trump-officials-reportedly-say-2000587948', 'title': 'Elon Musk Is Annoying, Unfunny, and Should Probably Take a Drug Test, Trump Officials Reportedly Say', 'snippet': 'Administration officials joked to one another about subjecting <strong>Musk</strong> to mandatory drug testing, which <strong>Musk</strong> himself has said would be a “great idea” for federal employees. It gets worse. “I have been in the same room with <strong>Elon</strong>, and he always tries to be <strong>funny</strong>.'}]}\n",
409418
"--------------------------------------------------\n",
410419
"Transition type: step\n",
411-
"Transition output: {'search_results': [{'title': 'News | Tottenham Hotspur', 'snippet': 'Latest official <strong>news</strong> and video from <strong>Tottenham</strong> <strong>Hotspur</strong>'}, {'title': 'Tottenham Hotspur FC - latest news, pictures, video comment - Football.london', 'snippet': 'The latest <strong>Tottenham</strong> <strong>news</strong>, transfers, fixtures and more. Including Live blogs, pictures, video, podcasts, polls and indepth analysis from our dedicated Spurs writers.'}, {'title': 'Tottenham Hotspur - News, Schedule, Scores, Roster, and Stats - The Athletic', 'snippet': 'Breaking <strong>Tottenham</strong> <strong>Hotspur</strong> <strong>news</strong> and in-depth analysis from the best newsroom in sports. Follow your favorite clubs. Get the latest injury updates, player <strong>news</strong> and more from around the league.'}]}\n",
420+
"Transition output: {'search_results': [{'title': 'Best Elon Musk Memes and Images - 9GAG', 'snippet': 'Enjoy the best of new <strong>funny</strong> cursed meme images about <strong>Elon</strong> <strong>Musk</strong>, Twitter, $8 verified badge, Tesla, the Boring Company and more on 9GAG. Never run out of hilarious cursed memes to share.'}, {'title': 'Elon FINALLY Gets the HUMILIATION He Deserves in Front of MILLIONS - YouTube', 'snippet': 'Comedian Bill Burr roasts <strong>Elon</strong> <strong>Musk</strong>, MAGA melts down over SNL, and protesters hit Trump where it hurts—his billionaire bestie&#x27;s wallet. Gabe Sanchez reports ...'}, {'title': 'Elon Musk Is Annoying, Unfunny, and Should Probably Take a Drug Test, Trump Officials Reportedly Say', 'snippet': 'Administration officials joked to one another about subjecting <strong>Musk</strong> to mandatory drug testing, which <strong>Musk</strong> himself has said would be a “great idea” for federal employees. It gets worse. “I have been in the same room with <strong>Elon</strong>, and he always tries to be <strong>funny</strong>.'}]}\n",
412421
"--------------------------------------------------\n",
413422
"Transition type: finish\n",
414-
"Transition output: Based on the recent news about Tottenham Hotspur, let me craft a sarcastic headline for you:\n",
423+
"Transition output: \"Genius Billionaire Desperately Attempts Comedy, Proves Money Can't Buy Sense of Humor\"\n",
415424
"\n",
416-
"\"BREAKING: Tottenham Fans Shocked to Discover Their Team's Trophy Cabinet Still Has Same Amount of Dust as Last Season\"\n",
425+
"\"Breaking: World's Richest Man Still Can't Figure Out How to Make People Genuinely Like Him\"\n",
417426
"\n",
418-
"Alternative sarcastic headlines:\n",
427+
"\"Shocking Report: Man Who Named Child 'X Æ A-12' May Not Be Most Stable CEO\"\n",
419428
"\n",
420-
"\"EXCLUSIVE: Spurs Players Perfect the Art of 'Almost Winning' in Revolutionary Training Session\"\n",
429+
"\"Former Trump Officials Suggest Drug Tests for Musk; Musk Agrees, Proving He Might Actually Need One\"\n",
421430
"\n",
422-
"\"REPORT: Scientific Study Confirms Supporting Tottenham Linked to Increased Heart Medication Sales\"\n",
431+
"\"Billionaire Space Enthusiast Too Busy Being 'Funny' on Twitter to Notice Everyone's Cringing\"\n",
423432
"\n",
424-
"\"SHOCKING: Tottenham's GPS Data Reveals Players Expertly Avoiding Trophy-Winning Positions\"\n",
433+
"\"Studies Show Being Worth $180 Billion Still Won't Make Your Memes Any Better\"\n",
425434
"\n",
426-
"These headlines play on Tottenham's notorious reputation for coming close but not quite winning major trophies, despite being one of England's biggest clubs. They maintain their status as every neutral's favorite \"almost there\" team!\n",
435+
"\"Man Who Could End World Hunger Chooses to Post Dad Jokes on Social Media Instead\"\n",
427436
"\n",
428-
"Would you like me to generate more sarcastic headlines about any specific aspect of Tottenham Hotspur?\n",
437+
"\"Breaking: Local Attention-Seeking Billionaire Still Seeking Attention, More at 11\"\n",
429438
"--------------------------------------------------\n"
430439
]
431440
}
@@ -457,7 +466,7 @@
457466
},
458467
{
459468
"cell_type": "code",
460-
"execution_count": 39,
469+
"execution_count": 10,
461470
"metadata": {},
462471
"outputs": [],
463472
"source": [
@@ -471,33 +480,42 @@
471480
},
472481
{
473482
"cell_type": "code",
474-
"execution_count": 49,
483+
"execution_count": 11,
475484
"metadata": {},
476485
"outputs": [
477486
{
478487
"name": "stdout",
479488
"output_type": "stream",
480489
"text": [
481-
"Based on the recent news about Tottenham Hotspur, let me craft a sarcastic headline for you:\n",
482-
"\n",
483-
"\"BREAKING: Tottenham Fans Shocked to Discover Their Team's Trophy Cabinet Still Has Same Amount of Dust as Last Season\"\n",
484-
"\n",
485-
"Alternative sarcastic headlines:\n",
490+
"Execution status: succeeded\n",
491+
"--------------------------------------------------\n",
492+
"\"In Shocking Display of Stockholm Syndrome, Former Arsenal Executive Defects to Tottenham\"\n",
486493
"\n",
487-
"\"EXCLUSIVE: Spurs Players Perfect the Art of 'Almost Winning' in Revolutionary Training Session\"\n",
494+
"\"Desperate Spurs Hire Ex-Arsenal CEO, Proving There's No Rock Bottom They Won't Explore\"\n",
488495
"\n",
489-
"\"REPORT: Scientific Study Confirms Supporting Tottenham Linked to Increased Heart Medication Sales\"\n",
496+
"\"Local Man Voluntarily Leaves Success to Join Eternal Trophy Drought: A London Tale\"\n",
490497
"\n",
491-
"\"SHOCKING: Tottenham's GPS Data Reveals Players Expertly Avoiding Trophy-Winning Positions\"\n",
498+
"\"Tottenham Finally Wins Something: The Heart of Arsenal's Former CEO\"\n",
492499
"\n",
493-
"These headlines play on Tottenham's notorious reputation for coming close but not quite winning major trophies, despite being one of England's biggest clubs. They maintain their status as every neutral's favorite \"almost there\" team!\n",
500+
"\"North London Rivalry Takes Bizarre Turn as Spurs Resort to Executive Poaching Instead of Trophy Hunting\"\n",
494501
"\n",
495-
"Would you like me to generate more sarcastic headlines about any specific aspect of Tottenham Hotspur?\n"
502+
"Pick any of these sarcastic headlines that capture the irony of Tottenham hiring their rival Arsenal's former CEO, while playfully jabbing at Spurs' notorious trophy drought and the historic rivalry between the two clubs! 😏\n"
496503
]
497504
}
498505
],
499506
"source": [
507+
"import time\n",
508+
"\n",
509+
"execution = client.executions.get(execution.id)\n",
510+
"\n",
511+
"while execution.status != \"succeeded\":\n",
512+
" time.sleep(5)\n",
513+
" execution = client.executions.get(execution.id)\n",
514+
" print(\"Execution status: \", execution.status)\n",
515+
" print(\"-\"*50)\n",
516+
"\n",
500517
"execution = client.executions.get(execution.id)\n",
518+
"\n",
501519
"print(execution.output)"
502520
]
503521
},
@@ -512,21 +530,21 @@
512530
},
513531
{
514532
"cell_type": "code",
515-
"execution_count": 50,
533+
"execution_count": 12,
516534
"metadata": {},
517535
"outputs": [
518536
{
519537
"data": {
520538
"text/plain": [
521-
"{'search_results': [{'title': 'News | Tottenham Hotspur',\n",
522-
" 'snippet': 'Latest official <strong>news</strong> and video from <strong>Tottenham</strong> <strong>Hotspur</strong>'},\n",
523-
" {'title': 'Tottenham Hotspur FC - latest news, pictures, video comment - Football.london',\n",
539+
"{'search_results': [{'title': 'Tottenham Hotspur FC - latest news, pictures, video comment - Football.london',\n",
524540
" 'snippet': 'The latest <strong>Tottenham</strong> <strong>news</strong>, transfers, fixtures and more. Including Live blogs, pictures, video, podcasts, polls and indepth analysis from our dedicated Spurs writers.'},\n",
525-
" {'title': 'Tottenham Hotspur - News, Schedule, Scores, Roster, and Stats - The Athletic',\n",
526-
" 'snippet': 'Breaking <strong>Tottenham</strong> <strong>Hotspur</strong> <strong>news</strong> and in-depth analysis from the best newsroom in sports. Follow your favorite clubs. Get the latest injury updates, player <strong>news</strong> and more from around the league.'}]}"
541+
" {'title': 'Spurs Media Watch, News and Stories',\n",
542+
" 'snippet': 'These stories have been specially selected from today&#x27;s media. They do not necessarily represent the views or position of <strong>Tottenham</strong> <strong>Hotspur</strong> Football Club. For total Spurs <strong>news</strong> coverage, visit NewsNow.co.uk, the UK&#x27;s #1 football <strong>news</strong> aggregator.'},\n",
543+
" {'title': 'Tottenham Hotspur FC - Transfer news, results, fixtures, video and audio',\n",
544+
" 'snippet': 'Having left Arsenal at the end of the 2023-24 campaign, Venkatesham has bided his time before returning to football. <strong>Tottenham confirmed Venkatesham&#x27;s appointment as their next CEO on Friday</strong>, in news that arrived with an element of surprise given his previous ties with Spurs&#x27; rivals.'}]}"
527545
]
528546
},
529-
"execution_count": 50,
547+
"execution_count": 12,
530548
"metadata": {},
531549
"output_type": "execute_result"
532550
}

0 commit comments

Comments
 (0)