fix: Playlists not being updated#5
Merged
trackstacker merged 3 commits intotrackstacker:mainfrom Feb 2, 2025
Merged
Conversation
I find it helpful to have a devcontainer set up and configured for the repo for quick edits and fixes. This adds basic dependabot python devcontainer and installs pip requirements.txt. Also auto adds a dependabot rule to check for updates.
Contributor
Author
|
I've manually run several times to confirm the playlist is updating (vs. new one being created). I'll let it run for a few hours to confirm the playlist is updating on schedule too. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
This PR is a quick attempt to fix the problem of new playlists being created each refresh vs. updating a playlist.
It fixes #4
To make it easier to spin up and isolate changes I also added a simple devcontainer config, which is configured for python development as well as running docker.
Updating Playlist
I saw that you had started to improve the LLM prompt in the v2 version. I copied the key elements of the v2 prompt into the v1 code to ensure the title always starts with "Meloday for".
Minor adjustments to the example prompts as not all attributes are available in v1 code (e.g. previous playlist details).
Devcontainer
I created a pretty standard devcontainer. This allows you to simply fire up the repo in codespace for small adjustments in a repeatable and declarative environment. This can also be run locally as a devcontainer without codepaces.
The devcontainer is set up with:
docker-compose up -din the devcontainer.vscode also auto applied a dependabot schedule to check for updates to the devcontainer.
Minor note about codespcaes.
You can define the three key environment variables as codespaces secrets in your settings. You can then define the scope of the secret i.e. which repo can see it. This helps to avoid injecting secrets into a git commit.