search

microsoft / PlanetaryComputerExamples

Examples of using the Planetary Computer
MIT License
369 stars 181 forks source link

Added my tutorial notebook model_training.ipynb #294

Closed jeanahoffmann closed 4 weeks ago

jeanahoffmann commented 4 weeks ago

I added my tutorial notebook that encompasses my intern project work under tutorials/model_training.ipynb. I also added the .json file for the Polygon land areas under tutorials/assets/map.json.

jeanahoffmann commented 4 weeks ago

@microsoft-github-policy-service agree company="Microsoft"

ghidalgo3 commented 4 weeks ago
  1. In the cell directly following I. Object Collection, the whole text is in a ### heading block. Should that be a normal paragraph? I think that's repeated a couple of times wherever you combined notebooks.
  2. In python, the documentation string """text""" goes just after the function signature definition, not before it. It's not like C#/Java, closer to documentation strings.
  3. Right before the start of the second section, can you mention at that point that randomly sampling from the entire globe is good but for specialized models users may want to sample from smaller areas by modifying the json file.
  4. Don't use the language "my project" for this notebook, write "this tutorial" instead.
  5. Where did you get the quote "noisy but semantically diverse image-text datasets [are capable]"?
  6. "The keys included are my choice and should be taken with a grain of salt ;)" more formal tone please
  7. Can you change the link to create an OpenAI Azure resource to point here: https://azure.microsoft.com/en-us/products/ai-services/openai-service
  8. I don't think its necessary to talk about Role Assignments
jeanahoffmann commented 4 weeks ago

Ok I added in all of your suggestions! Thanks so much for taking the time to review the notebook! Let me know if there's anything else I should fix.