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

comments #1

Open
robotics-dev-0101 opened this issue Feb 8, 2020 · 2 comments
Open

comments #1

robotics-dev-0101 opened this issue Feb 8, 2020 · 2 comments

Comments

@robotics-dev-0101
Copy link

Hi,

Thanks a lot for sharing this code! it is very well written and the results of the SLAM when the code is executed are amazing :)

I was wondering if you could comment your code because now I am reading it and some things are not clear directly, so I am trying to go through the code line by line to understand what why you've done it this way. The general idea is very clear however.

It would be great if you can comment these functions:

In ScanMatcher_OGBased.py
frameSearchSpace()
matchScan()
searchToMatch(), her you are looping only over theta, should you loop inside this loop over x and y to find the best combination in the search space?
some variables are not clear like: rv and rrv, could you please clarify what these are?

In generateProbSearchSpace() it is not clear why you do this: probSP[probSP > 0.5 * probMin] = 0, I didn't what you are doing in generateProbSearchSpace()

In OccupancyGrid.py
expandOccupancyGridHelper(self, position, axis):

Thanks again for this amazing piece of code!

@deveshdatwani
Copy link

@robotics-dev-0101 I am going to start reading the code. Did you figure it out finally?

@shan83kalai
Copy link

@xiaofeng419 Thanks lot sharing code. could please explain how or from where did you get intel.gfs file in the raw directory?

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

No branches or pull requests

3 participants