Skip to content

Conversation

@anna-charlotte
Copy link
Contributor

@anna-charlotte anna-charlotte commented Dec 9, 2022

Goals:

  • add MeshUrl
  • add tests for MeshUrl
  • add PointCloudUrl
  • add tests for PointCloudUrl
  • add predefined Mesh Document
  • add predefined PointCloud Document
  • check and update documentation, if required. See guide

@anna-charlotte anna-charlotte changed the base branch from main to feat-rewrite-v2 December 9, 2022 13:39
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
anna-charlotte added 11 commits December 9, 2022 15:16
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
anna-charlotte added 3 commits December 12, 2022 14:38
anna-charlotte added 3 commits December 12, 2022 16:46
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
anna-charlotte added 11 commits December 13, 2022 09:33
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
@JohannesMessner JohannesMessner mentioned this pull request Dec 13, 2022
47 tasks
Co-authored-by: samsja <[email protected]>
Signed-off-by: Charlotte Gerhaher <[email protected]>
Copy link
Member

@JohannesMessner JohannesMessner left a comment

Choose a reason for hiding this comment

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

Just one thing, can you adjust the class docstrings for MeshUrl, PointCloudUrl, and Url3d? Right now they all say the same thing, as a user which one should I use when? Or is it self-explanatory for someone that (unlike me) knows what they are doing with 3D stuff?

anna-charlotte added 2 commits December 14, 2022 10:46
@anna-charlotte
Copy link
Contributor Author

Ok, added a little bit more information to the docstring

@github-actions
Copy link

📝 Docs are deployed on https://ft-feat-add-3d-docs-v2--jina-docs.netlify.app 🎉

@samsja samsja merged commit dedd18e into feat-rewrite-v2 Dec 14, 2022
@samsja samsja deleted the feat-add-3d-docs-v2 branch December 14, 2022 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add 3d predefined documents in v2

4 participants