Skip to content

docs: Add warehouse bird's eye view image and improve laser mapping documentation#3373

Closed
Aaravanand00 wants to merge 2 commits intoJdeRobot:gh-pagesfrom
Aaravanand00:gh-pages
Closed

docs: Add warehouse bird's eye view image and improve laser mapping documentation#3373
Aaravanand00 wants to merge 2 commits intoJdeRobot:gh-pagesfrom
Aaravanand00:gh-pages

Conversation

@Aaravanand00
Copy link

  • Added warehouse_birdeye.png showing top-down view of warehouse environment
  • Improved Goal section with clearer objectives
  • Added 'What you must do' section with practical checklist
  • Expanded Theory section with mapping concepts and occupancy grid details
  • Added 'When you are done' validation checklist

Fixes #3050

…ocumentation

- Added warehouse_birdeye.png showing top-down view of warehouse environment
- Improved Goal section with clearer objectives
- Added 'What you must do' section with practical checklist
- Expanded Theory section with mapping concepts and occupancy grid details
- Added 'When you are done' validation checklist

Fixes JdeRobot#3050
@javizqh javizqh requested a review from jmplaza January 7, 2026 19:07
@javizqh javizqh added the gh-pages Github Pages label Jan 15, 2026
…ry improvements with warehouse bird's eye view and checklists
@Aaravanand00
Copy link
Author

Hi @jmplaza sir
conflicts was resolved if any changes was required I'm happy to work on it

@javizqh javizqh requested a review from aquintan4 March 12, 2026 18:09
@aquintan4
Copy link
Collaborator

The changes seem mostly cosmetic (spacing, minor markdown edits) and do not significantly improve the documentation.

The newly added image also appears to be AI-generated and contains visual inconsistencies (broken grid, unusual shelves), which may not accurately represent the environment used in the exercise.

Additionally, a paragraph explaining that the exercise assumes the robot is already localized has been removed, which is an important conceptual detail for students.

For these reasons, I’m not sure the PR provides a meaningful improvement to the documentation.

@aquintan4 aquintan4 closed this Mar 12, 2026
@Aaravanand00
Copy link
Author

Hi @aquintan4,

I would like to clarify that the image I added was not AI-generated. I created it myself while trying to illustrate a bird’s-eye view of the warehouse environment. However, I understand your concern about the visual inconsistencies and I will be more careful in the future to ensure that any images accurately reflect the real simulation environment used in the exercise.

Also, thank you for pointing out the removed paragraph about the robot already being localized. I now realize that it is an important conceptual detail for students, and I appreciate you highlighting that.

@javizqh, if possible, could you please guide me on which issue or area of the repository would be a good place for me to contribute next? I would really appreciate your guidance so that I can work on something more relevant and useful for the project. I’m happy to learn and will do my best to contribute meaningfully under your guidance.

@JdeRobot JdeRobot locked as resolved and limited conversation to collaborators Mar 13, 2026
@jmplaza
Copy link
Member

jmplaza commented Mar 14, 2026

@javizqh, if possible, could you please guide me on which issue or area of the repository would be a good place for me to contribute next? I would really appreciate your guidance so that I can work on something more relevant and useful for the project. I’m happy to learn and will do my best to contribute meaningfully under your guidance.

Hi @Aaravanand00,

Check the open issues, they are the most useful points to contribute :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

gh-pages Github Pages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants