Skip to content
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

[bounty] move as much as possible code from pre_build.js to build.rs #720

Closed
louis030195 opened this issue Nov 20, 2024 · 7 comments · Fixed by #729
Closed

[bounty] move as much as possible code from pre_build.js to build.rs #720

louis030195 opened this issue Nov 20, 2024 · 7 comments · Fixed by #729
Labels

Comments

@louis030195
Copy link
Collaborator

atm we have lot of pre build thing in pre_build.js script used for the app but we want to make it easy to use the CLI

installation of screenpipe as a CLI or app should be dead simple in less than 2 minutes from install to run and work on most OSes

for example onnxruntime dll stuff for windows should be in build.rs of screenpipe audio so can use CLI without app and more rust native

/bounty 100

will add bonus if you do more than the onnxruntime thing

@louis030195 louis030195 added the enhancement New feature or request label Nov 20, 2024
Copy link

linear bot commented Nov 20, 2024

Copy link

algora-pbc bot commented Nov 20, 2024

💎 $100 bounty • Screenpi.pe

Steps to solve:

  1. Start working: Comment /attempt #720 with your implementation plan
  2. Submit work: Create a pull request including /claim #720 in the PR body to claim the bounty
  3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to mediar-ai/screenpipe!

Add a bountyShare on socials

Attempt Started (GMT+0) Solution
🟢 @imperivox Nov 21, 2024, 5:27:06 PM WIP
🟢 @Neptune650 #729

@Neptune650
Copy link
Contributor

@louis030195 Do you suggest that pre_build.js should only have stuff used in the app? How do you feel about moving the ollama sidecar too then

@louis030195
Copy link
Collaborator Author

yes

we should have the LLM sidecar in core codebase too but that's some heavy work

@imperivox
Copy link

imperivox commented Nov 21, 2024

/attempt #720

Copy link

algora-pbc bot commented Nov 21, 2024

💡 @Neptune650 submitted a pull request that claims the bounty. You can visit your bounty board to reward.

Copy link

algora-pbc bot commented Nov 21, 2024

🎉🎈 @Neptune650 has been awarded $100! 🎈🎊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants