Moving all scripts to git #957
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4 tasks
shahdyousefak
requested review from
jeffbl
and removed request for
jaydeepsingh25
jeffbl
requested review from
JRegimbal and
jeffbl
and removed request for
jeffbl
Member
|
I added a README.md and added/modified the documentation in each of the scripts @shahdyousefak I didn't review your abstraction into env files, but I assume we'll quickly see if all that works once deployed, and these are not showstoppers since they are only support scripts, so I'm not too worried. @shahdyousefak Is the take that we get this merged, then update unicorn and pegasus manually to switch over to these? (update cron path, links in @JRegimbal assigining to you for a brief look since you created many of these scripts. The bar for these is lower than for code meant for production. |
JRegimbal
approved these changes
Feb 17, 2025
…and the orchestrator (#945) * adding healthchecks to preprocessors * Adding healthchecks to orchestartor, fixing casing * correcting orchestrator, nominatim healthchecks. Adding services healthchecks * healthcheck for tat service * fixing casing * Adding healthchecks to all the handlers * fixing pep8 * fixing failed tests * fixing context for dockerfile - tat service * harmonizing healthchecks (1) * harmonizing healthchecks (2) * Remove mistakenly committed file * standardize healthcheck output * adding || break so that the script exits the loop if nc fails * adding a break to supercollider-alpine healthcheck * investigating build failure
* initial checkin of text-followup preprocessor * switched prompt to JSON outout, about to update parsing and verification * first mostly working version; need to remove debug lines * first mostly working version; need to remove debug lines * update prompt, pii debug logging, error handling, build, docker-compose * workflow add for text-followup * flake8 * Moving the healthchecks out of test-docker-compose.yml * Add "Built with Llama" to README * Add Built with Llama to README * formatting in README * docker-compose.yml memcached restart: unless-stopped * Update schemas to resolve #940. * Add Built with Llama to README * formatting in README * readme update for prompt override example * add text-followup preprocessor to build.yml * add PII logging to test-docker-compose * reorg Dockerfile, adding healthcheck --------- Co-authored-by: Shahd Yousef <[email protected]> Co-authored-by: Juliette Regimbal <[email protected]>
This is likely to occur when dealing with non-photo data, as is required to resolve #939.
#955) * Introduce prompt override via env variable to content-categoriser and graphic-caption preprocessors; minor edits to resolve deprecations * pep8
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please ensure you've followed the checklist and provide all the required information before requesting a review.
If you do not have everything applicable to your PR, it will not be reviewed!
If you don't know what something is or if it applies to you, ask!
Don't delete below this line.
Required Information
Coding/Commit Requirements
New Component Checklist (mandatory for new microservices)
docker-compose.ymlandbuild.yml..github/workflows.README.mdfile that describes what the component does and what it depends on (other microservices, ML models, etc.).OR