Conversation
Updated layout to skip agentops files.
added additional skips for agentops files
Contributor
There was a problem hiding this comment.
I think this is a good first step. Next step would be to add a variable in the template JSON schema, and merge the overlapping files that right now we just added a suffix too. But this can be done in a follow-up PR to update this branch
Noticed we don't have any CICD files today? Also, currently tests are failing, which is fine for now, but we will need to fix before merging to main branch
template/update_layout.tmpl
Outdated
|
|
||
| # Skip all Agentops files in the template | ||
| {{ skip (printf `%s/%s/%s` $root_dir $project_name_alphanumeric_underscore `data_preparation`) }} | ||
| {{ skip (printf `%s/%s/%s` $root_dir $project_name_alphanumeric_underscore `data_preparation/data_ingestion`) }} |
Contributor
There was a problem hiding this comment.
Skipping the directory will skip all of its files, no need to repeat
Collaborator
Author
There was a problem hiding this comment.
modified update_layout to skip the directories where needed.
…pdated databricks template
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
3 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.
AgentOps initial integration into MLOps Stacks
Overview
This PR adds in the AgentOps stacks files to the project template and incorporates copies of existing files appended with _agentops for comparison as we continue to shape up the LLMOps flavor.
I've also added the skip commands for all files in the template folders that pertain to AgentOps automatically.
Note: there are some files above the template in this PR that are _agentops versions of existing files that will need to be rectified. They are there currently for reference.