Skip to content

Metadata/Information for Items #20

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Dec 20, 2024
Merged

Metadata/Information for Items #20

merged 16 commits into from
Dec 20, 2024

Conversation

0PandaDEV
Copy link
Owner

@0PandaDEV 0PandaDEV commented Dec 16, 2024

To review:

  • Correct Scheme
  • Code language detection accuracy

Test builds are here https://github.com/0PandaDEV/Qopy/actions/runs/12353882273

@0PandaDEV 0PandaDEV added the Feature A feature request label Dec 16, 2024
@0PandaDEV 0PandaDEV self-assigned this Dec 16, 2024
@0PandaDEV 0PandaDEV linked an issue Dec 16, 2024 that may be closed by this pull request
32 tasks
@0PandaDEV 0PandaDEV changed the title [Issue] Metadate/Information for Items Metadate/Information for Items Dec 16, 2024
@0PandaDEV 0PandaDEV changed the title Metadate/Information for Items Metadata/Information for Items Dec 18, 2024
@0PandaDEV
Copy link
Owner Author

@wada3n Would be able to test the two review points from this PR so I can make improvements. It's currently known that the programming language detection is not good at all, I may have to temporarily remove this and train a new AI model for this instead of relying on another crate.

@wada3n
Copy link

wada3n commented Dec 20, 2024

Hi @0PandaDEV,

I've tested it, and it works as expected for: files, images, colors, links and text. However, I believe that the programming language detection feature might be a significant burden, given the vast number of programming languages. Additionally, this feature could shift the application's focus towards being a Snippet manager rather than its primary purpose. It might also take a considerable amount of time to develop features that are not directly related to the core functionality of the app.

Thank you for your efforts!

@0PandaDEV 0PandaDEV merged commit 56fab79 into main Dec 20, 2024
1 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature A feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Metadata/Information for items
2 participants