Skip to content

Commit 9532c21

Browse files
authored
.Net Notebook: Fixing new skills namespace in memory notebook (#1912)
### Motivation and Context Please help reviewers and future users, providing the following information: 1. Why is this change required? It's a simple bug fix to account for namespace updates 3. What problem does it solve? The memory and embedding notebook still uses the previous skills namespace and thus breaks 5. What scenario does it contribute to? Notebooks 7. If it fixes an open issue, please link to the issue here. ### Description <!-- Describe your changes, the overall approach, the underlying design. These notes will help understanding how your code works. Thanks! --> Just a line change ### Contribution Checklist <!-- Before submitting this PR, please make sure: --> - [X] The code builds clean without any errors or warnings - [X] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md) - [X] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with `dotnet format` - [X] All unit tests pass, and I have added new tests where possible - [X] I didn't break anyone 😄
1 parent 7880407 commit 9532c21

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

samples/notebooks/dotnet/06-memory-and-embeddings.ipynb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,7 @@
213213
},
214214
"outputs": [],
215215
"source": [
216-
"using Microsoft.SemanticKernel.CoreSkills;\n",
216+
"using Microsoft.SemanticKernel.Skills.Core;\n",
217217
"\n",
218218
"// TextMemorySkill provides the \"recall\" function\n",
219219
"kernel.ImportSkill(new TextMemorySkill());"

0 commit comments

Comments
 (0)