Skip to content

Add FileCompression skill - #82

Merged
Lee Miller (lemillermicrosoft) merged 10 commits into
microsoft:mainfrom
glahaye:FileCompressionMerged
Mar 30, 2023
Merged

Add FileCompression skill#82
Lee Miller (lemillermicrosoft) merged 10 commits into
microsoft:mainfrom
glahaye:FileCompressionMerged

Conversation

@glahaye

@glahaye Gil LaHaye (glahaye) commented Mar 15, 2023

Copy link
Copy Markdown
Contributor

Motivation and Context
We want to be able to compress and decompress file.

Description
Add the FileCompressionSkill in dotnet samples folder

Contribution Checklist

Comment thread dotnet/src/SemanticKernel.Skills/Skills.FileCompression/FileCompressionSkill.cs Outdated
Comment thread dotnet/src/SemanticKernel.Skills/Skills.Web/WebFileDownloadSkill.cs
@awharrison-28

Copy link
Copy Markdown
Contributor

Can we put this project under the samples/skills folder?

@glahaye

Copy link
Copy Markdown
Contributor Author

Can we put this project under the samples/skills folder?

What determines whether a skill goes in samples/skills or src/SemanticKernel.Skills?

Shawn Callegari (@shawncal) told me one big difference is that src/SemanticKernel.Skills skills go out in Nugets and therefore need more support down the line and as such, when things are still in flux (like they are now), going with samples/skills makes more sense.

Going with your suggestion, Abby Harrison (@awharrison-28)

@adrianwyatt Adrian Bonar (adrianwyatt) added the PR: feedback to address Waiting for PR owner to address comments/questions label Mar 20, 2023
@glahaye Gil LaHaye (glahaye) added PR: ready for review All feedback addressed, ready for reviews and removed PR: feedback to address Waiting for PR owner to address comments/questions labels Mar 21, 2023
@lemillermicrosoft

Copy link
Copy Markdown
Member

Is this needed for memory-sample branch work?

@glahaye

Copy link
Copy Markdown
Contributor Author

Is this needed for memory-sample branch work?

It's a "nice-to-have" for the memory sample. Not needed now.

Comment thread samples/dotnet/FileCompression/FileCompressionSkill.cs Outdated
Comment thread samples/dotnet/FileCompression/FileCompressionSkill.cs Outdated
Comment thread samples/dotnet/FileCompression/FileCompressionSkill.cs Outdated
@adrianwyatt Adrian Bonar (adrianwyatt) added PR: feedback to address Waiting for PR owner to address comments/questions and removed PR: ready for review All feedback addressed, ready for reviews labels Mar 22, 2023
@glahaye Gil LaHaye (glahaye) removed the PR: feedback to address Waiting for PR owner to address comments/questions label Mar 27, 2023
@glahaye Gil LaHaye (glahaye) added the PR: ready for review All feedback addressed, ready for reviews label Mar 27, 2023
@adrianwyatt
Adrian Bonar (adrianwyatt) enabled auto-merge (squash) March 27, 2023 23:29

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pre-approval pending merge/rebase.

Comment thread samples/dotnet/FileCompression/FileCompression.csproj
auto-merge was automatically disabled March 30, 2023 19:35

Head branch was pushed to by a user without write access

@lemillermicrosoft
Lee Miller (lemillermicrosoft) merged commit ad77101 into microsoft:main Mar 30, 2023
@glahaye
Gil LaHaye (glahaye) deleted the FileCompressionMerged branch March 31, 2023 16:39
Desmond Howard (dehoward) pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
**Motivation and Context**
We want to be able to compress and decompress file.

**Description**
Add the FileCompressionSkill in dotnet samples folder
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
### Motivation and Context

<!-- Thank you for your contribution to the copilot-chat repo!
Please help reviewers and future users, providing the following
information:
  1. Why is this change required?
  2. What problem does it solve?
  3. What scenario does it contribute to?
  4. If it fixes an open issue, please link to the issue here.
-->
We would like to show in the webapp some of the AI service
options/configurations we select for our webapi.

### Description
1. Add an OptionsController to allow the webapp to pull (non-secretive)
configurations from the backend.
2. Webapp to retrieve memories storage option from the webapi and show
it in the documents tab.

![image](https://github.com/microsoft/chat-copilot/assets/12570346/2c7c6b93-3550-400d-aa33-7798dc66751b)

<!-- Describe your changes, the overall approach, the underlying design.
These notes will help understanding how your code works. Thanks! -->

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [ ] The code builds clean without any errors or warnings
- [ ] The PR follows the [Contribution
Guidelines](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [ ] All unit tests pass, and I have added new tests where possible
- [ ] I didn't break anyone 😄
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: ready for review All feedback addressed, ready for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants