Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove clay-v0-tutorials & add visualize embeddings for v1 #256

Merged
merged 5 commits into from
May 28, 2024

Conversation

srmsoumya
Copy link
Collaborator

  • Remove deprecated tutorial notebooks for clay-v0
  • Add a tutorial notebook to show how embedding space looks like for Clay Encoder (v1)
  • Show examples for 2 sensors, i.e NAIP & Sentinel-2

@lauracchen
Copy link
Member

hey @srmsoumya is there any reason to keep but "archive" the older tutorial notebooks? I doubt there's a reason someone wants to use v0 over v1 at this point, but maybe in the future we should think about keeping deprecated notebooks so people can still make use of the older model versions if they want to?

Copy link
Member

@yellowcap yellowcap 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 🐧 do you think we can make the linters happy before merging?

@yellowcap
Copy link
Member

hey @srmsoumya is there any reason to keep but "archive" the older tutorial notebooks?

@lauracchen the notebooks stay in the repo, so that is definitively a way to archive them. We were also thinking of re-organising the content once we have merged the main pieces and we see what structure would make sense. Let's consider linking to the old notebooks at that point.

@yellowcap yellowcap merged commit a61f9fa into main May 28, 2024
2 of 3 checks passed
@yellowcap yellowcap deleted the docs/tutorial branch May 28, 2024 07:40
@yellowcap
Copy link
Member

Merging to have the updates notebooks out asap. Let's figure out the linters afterwards.

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.

3 participants