Open3D-ML icon indicating copy to clipboard operation
Open3D-ML copied to clipboard

Alex/open3d ml tutorials

Open rozeappletree opened this issue 3 years ago • 1 comments

Re-write condensed tutorials based on suggestions.

Changes were made to 3 tutorial notebooks

  • docs/tutorial/notebook/01_config_files.ipynb
  • docs/tutorial/notebook/02_datasets.ipynb
  • docs/tutorial/notebook/03_train_model.ipynb

Suggested changes were:

  1. The notebooks must look similar to already existing open3d tutorial notebooks (all notebooks)
  2. Run all the notebooks files and check if they work locally (all notebooks)
  3. Working directory must be docs/tutorial/notebook/ for all data, model checkpoint logs etc. (all notebooks)
  4. Remove garbage-in garbage-out dataset graphic. (01_config_files.ipynb)
  5. Use consistent vector notations (01_config_files.ipynb)
  6. Change to right-handed coordinate system (02_datasets.ipynb)
  7. Change colour of axes in the coordinate system (02_datasets.ipynb)

Please note:

  1. Some cells of 02_datasets.ipynb will not run because the notebook does not have access to smaller version of semanticKITTI dataset. ~~@sanskar107 can you share a downloadable link for that dataset that I can wget in the the notebook?~~ Will use shared link https://cdn.discordapp.com/attachments/903644960211992607/946792875423838288/semkitti.zip.
  2. I will pass all the text information through gramarly before merging once structure and content gets green signal.
  3. I will remove Inference_on_a_custom_data.ipynb, reading_a_config_file.ipynb, reading_a_dataset.ipynb, train_ss_model_using_pytorch.ipynb and train_ss_model_using_tensorflow.ipynb before merging because they are essentially duplicates. Keeping them here for reference temporarily.

This change is Reviewable

rozeappletree avatar Jun 16 '22 06:06 rozeappletree

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB