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

Clay embeddings on Sentinel-1 data #315

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

anindabitm
Copy link

Added notebook to demonstrate use of Clay embeddings on Sentinel-1 imagery for earthquake detection

anindabitm and others added 2 commits August 1, 2024 16:38
Added notebook to demonstrate use of Clay embeddings on Sentinel-1 imagery for earthquake detection
@srmsoumya
Copy link
Collaborator

Thank you, @anindabitm, for the well-formatted and clean notebook.

Before we proceed with merging the pull request, here are a few additions I recommend:

  • Please include a brief description of the dataset and the problem statement to give context to anyone reviewing this work.
  • It would be beneficial to elaborate on our approach using the CLAY model. Specifically, explain how we're utilizing CLAY embeddings for analyzing pre/post-earthquake scenarios and the rationale behind building a classifier on top of these embeddings.
  • Could you switch to using the actual validation and test sets? This change will help in providing a more accurate evaluation of the embeddings' effectiveness and their performance relative to the leaderboard.
  • Lastly, some insights into the time required to generate these embeddings and to train a robust model using them would be valuable for understanding the computational efficiency and feasibility of our approach.

Looking forward to the updates, and thank you for all your hard work.

@anindabitm
Copy link
Author

Thanks @srmsoumya for your inputs. I have updated the notebook. Request you to please review

@lauracchen
Copy link
Member

@yellowcap is this ready to be merged?

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