Skip to content

Add sample data instructions#451

Merged
ebrahimebrahim merged 2 commits intoOpenwaterHealth:mainfrom
ebrahimebrahim:add-sample-data-instructions
Apr 28, 2026
Merged

Add sample data instructions#451
ebrahimebrahim merged 2 commits intoOpenwaterHealth:mainfrom
ebrahimebrahim:add-sample-data-instructions

Conversation

@ebrahimebrahim
Copy link
Copy Markdown
Collaborator

Replacing DVC by the openlifu-sample-database as a source of sample data to direct new users to.

I will push tags to openlifu-sample-database that we can reference in the git clone command in the README here. Right now there is just one tag openlifu-v0.20.0 indicating which version of the sample data should be used with openlifu v0.20.0.

Once this is merged, I will updated our release notes for our latest releases to point to this new section in the readme, rather than pointing to dvc data.

For review: Just look at the updated readme and see that the formatting and instructions are sensible.

…lth#337)

Removes the stale TransducerArray skip. Since openlifu-python commit
cbd521b introduced TransducerArray with its own to_file, and
Database.write_transducer is type-agnostic and just calls
transducer.to_file(...), the skip is no longer needed.
…ns (OpenwaterHealth#338)

The sample database is now in its own repository
(openlifu-sample-database) with some files tracked with Git LFS.
The DVC-based flow, which was intended for developers during the
early days of the project, is not the way for general users to get sample data.
@ebrahimebrahim ebrahimebrahim force-pushed the add-sample-data-instructions branch from f964d9c to bab0fd5 Compare April 28, 2026 14:58
Copy link
Copy Markdown
Contributor

@sadhana-r sadhana-r left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@ebrahimebrahim ebrahimebrahim merged commit cd03383 into OpenwaterHealth:main Apr 28, 2026
10 checks passed
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