Skip to content

Conversation

@chr-hertel
Copy link
Member

Q A
Bug fix? no
New feature? yes
Docs?
Issues
License MIT

Bringing in the first non-chatbot example - sitting on top of Hugging Face.
The frontend & resampler implementation is rather bad, but will have another look if time permits - and i guess the idea is clear.

Screencast.from.2025-11-10.00-08-47.mp4

@chr-hertel chr-hertel requested a review from Nyholm as a code owner November 9, 2025 23:37
@chr-hertel chr-hertel added the Demo Issues & PRs about the demo application label Nov 9, 2025
@chr-hertel chr-hertel force-pushed the huggingface-smart-crop branch 3 times, most recently from 0f955ca to 8ae6015 Compare November 10, 2025 00:04
OskarStark added a commit that referenced this pull request Nov 10, 2025
This PR was merged into the main branch.

Discussion
----------

[Demo] Remove `Bot` suffix from link name

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Docs?         | no
| Issues        | Refs #851
| License       | MIT

Commits
-------

2d78017 [Demo] Remove `Bot` suffix from link name
<div class="card-body">
<h5 class="card-title">Video Bot</h5>
<p class="card-text">Simple demonstration of vision capabilities of GPT in combination with your webcam.</p>
<a href="{{ path('video') }}" class="btn btn-outline-dark d-block">Try Video Bot</a>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<a href="{{ path('video') }}" class="btn btn-outline-dark d-block">Try Video Bot</a>
<a href="{{ path('video') }}" class="btn btn-outline-dark d-block">Try Video</a>

Should we remove the Bot suffix in this context too?

@chr-hertel chr-hertel force-pushed the huggingface-smart-crop branch from 8ae6015 to 37d065f Compare November 10, 2025 08:30
Copy link
Member Author

Choose a reason for hiding this comment

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

FTR, this implementation is bad and only works well for some cases.

@chr-hertel chr-hertel force-pushed the huggingface-smart-crop branch from 37d065f to a1bb014 Compare November 10, 2025 08:34
@OskarStark
Copy link
Contributor

Thanks Christopher for working on this feature, this is much appreciated.

@OskarStark OskarStark merged commit 5f498f5 into symfony:main Nov 10, 2025
15 checks passed
@chr-hertel chr-hertel deleted the huggingface-smart-crop branch November 10, 2025 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Demo Issues & PRs about the demo application Feature New feature Status: Reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants