Skip to content

doc: Add information about the test dataset#556

Merged
ethanjli merged 4 commits intomasterfrom
docs/test-dataset-link
Apr 4, 2025
Merged

doc: Add information about the test dataset#556
ethanjli merged 4 commits intomasterfrom
docs/test-dataset-link

Conversation

@ethanjli
Copy link
Collaborator

@ethanjli ethanjli commented Apr 2, 2025

In the 2025-04-02 software meeting we decided to have a publicly-accessible dataset hosted on Google Drive for people to download as a way of testing the segmenter. @chevreuill3000 uploaded a dataset to https://drive.google.com/drive/folders/1g6OPaUIhYkU2FPqtIK4AW6U4FYmhFxuw?usp=sharing , and PlanktoScope/pallet-segmenter@3be18e3 added usage instructions for it to the pallet-segmenter repo (as a way of testing that repo). This PR makes a similar(-ish) copy of those instructions for the tips-and-tricks document, because I wasn't sure where else to put information about that test dataset.

@sonnyp: I'm adding you as a reviewer in case you can easily think of a more appropriate place for this information, or in case you want to test out the dataset using these instructions; otherwise, I guess we can just merge this PR?

@ethanjli ethanjli requested a review from sonnyp April 2, 2025 22:26
@ethanjli ethanjli marked this pull request as ready for review April 2, 2025 22:26
Copy link
Collaborator

@sonnyp sonnyp left a comment

Choose a reason for hiding this comment

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

LGTM

I would suggest trying to make the text "breathe". Try to be more concise and straight to the point. Logical blocks should be separated by newlines

image


This page is for developers, what can be expressed with shell doesn't next to be explained textually. Consider providing commands to do what is explained?

ssh ...
cd ...
wget ...
unzip ...

@ethanjli ethanjli added this pull request to the merge queue Apr 4, 2025
@ethanjli
Copy link
Collaborator Author

ethanjli commented Apr 4, 2025

Thanks for the guidance, it was very helpful for my edits! I'll merge this PR now. Of course, feel free to edit it as you see fit after merging.

Merged via the queue into master with commit 9133a97 Apr 4, 2025
3 checks passed
@ethanjli ethanjli deleted the docs/test-dataset-link branch April 4, 2025 23:32
Copy link
Collaborator

@sonnyp sonnyp left a comment

Choose a reason for hiding this comment

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

Read greats 👍

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