Skip to content

Chore/adding dvc and add assets/ - #1

Merged
rumunius merged 3 commits into
mainfrom
chore/adding_dvc
Aug 5, 2026
Merged

Chore/adding dvc and add assets/#1
rumunius merged 3 commits into
mainfrom
chore/adding_dvc

Conversation

@rumunius

@rumunius rumunius commented Aug 5, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI lite review requested due to automatic review settings August 5, 2026 09:42
@rumunius
rumunius merged commit b5ae1ce into main Aug 5, 2026
1 check passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces DVC tracking for an assets/ directory and adds initial DVC configuration/ignore files to support managing large or generated artifacts outside of Git.

Changes:

  • Added assets.dvc to track the assets/ directory with DVC.
  • Added DVC ignore files (.dvcignore, .dvc/.gitignore) and initialized .dvc/config.
  • Updated .gitignore with an additional assets ignore rule.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
assets.dvc Adds DVC metadata to track the assets/ directory.
.gitignore Adds an additional ignore entry for assets (currently redundant with an existing rule).
.dvcignore Adds a baseline DVC ignore file (comments only).
.dvc/config Adds DVC remote configuration (currently includes a non-portable local path and a likely incorrect section header).
.dvc/.gitignore Adds standard DVC internal ignores (config.local, tmp, cache).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .dvc/config
Comment on lines +1 to +4
[core]
remote = hmm-storage
['remote "hmm-storage"']
url = E:/DVC/HMM
@rumunius
rumunius deleted the chore/adding_dvc branch August 5, 2026 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants