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

Qualification ZIP file - 2 or 3 files? #39

Closed
osrf-migration opened this issue Dec 22, 2018 · 12 comments
Closed

Qualification ZIP file - 2 or 3 files? #39

osrf-migration opened this issue Dec 22, 2018 · 12 comments
Labels
bug Something isn't working minor

Comments

@osrf-migration
Copy link

Original report (archived issue) by Martin Dlouhy (Bitbucket: robotikacz).


The https://osrf-migration.github.io/subt-gh-pages/#!/osrf/subt/wiki/tutorials/qual states that qualification ZIP file should contain 2 files, while SubT_Qualification_Guide.pdf says: "A submission will consist of a single zip (.zip) file that contains three files: simulation.log, subt.log, and narrative.md." ... so which one is correct?

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


Thanks. I've update the subt wiki to indicate that three files are required.

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


  • changed state from "new" to "resolved"

Updated wiki to match the qualification guide.

@osrf-migration
Copy link
Author

Original comment by Martin Dlouhy (Bitbucket: robotikacz).


  • changed priority from "blocker" to "minor"

@osrf-migration
Copy link
Author

Original comment by Martin Dlouhy (Bitbucket: robotikacz).


Thank you Nathan for update - could you maybe please also update the command in "3. Create the final zip file", i.e. to add on the command line "narrative.md"?

For me it is still not clear if we should submit improvements and what is the current threshold for qualification? Is there any "score board" with other teams? I did not realize that it is manual work so maybe multiple submissions are discouraged?
thanks a lot
Martin

@osrf-migration
Copy link
Author

Original comment by Martin Dlouhy (Bitbucket: robotikacz).


  • changed state from "resolved" to "open"

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


The following text has been added to the wiki:

There are sixteen artifacts in the tunnel qualification world. In order to pass qualifications, you will need to find at least half of these artifacts. For each successful artifact found, you can receive a minimum score of 3 and a maximum score of 9. The more accurate, in Euclidean distance, your artifact report is the higher the score. A minimum overall score of 24 is required, which is 8 artifacts each with a value of 3. Keep in mind that a score of 24 is not the only requirement, you must find at least 8 artifacts.

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


A leaderboard that will display team scores is under development. We are not discouraging multiple submissions from teams, but it would be good to refrain from inundating us with numerous submissions.

@osrf-migration
Copy link
Author

Original comment by Martin Dlouhy (Bitbucket: robotikacz).


Thank you Nathan for update. Is there a change in scoring since 21st Dec 2018? At that time the point for artifact was multiplied by up to 3 times for artifact precision and the same how fast you report it. Our first attempt was in 20 minutes and it was 1 point only. The second attempt (the same artifact) was 4 points:

#!c++

465 1000000 new_artifact_reported
465 1000000 distance_bonus_x2
465 1000000 time_bonus_x2
465 1000000 modified_score 4
465 1000000 new_total_score 4 

Note, that draft "SubT_Challenge_Competition_Rules_Tunnel_Circuit.pdf" section 11.1 talks only about 1 point for 5 meter precision.

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


Our original scoring algorithm predated the competition rules. Pull request #107 will bring the scoring algorithm inline with the competition rules. Once merged, I'll update the wiki page to match.

@osrf-migration
Copy link
Author

Original comment by Martin Dlouhy (Bitbucket: robotikacz).


OK, thank you

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


  • changed state from "open" to "resolved"

We have merge the pull request, and updated the wiki. Feel free to re-open if the problem persists.

@osrf-migration
Copy link
Author

Original comment by Alfredo Bencomo (Bitbucket: bencomo).


  • changed state from "resolved" to "closed"

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working minor
Projects
None yet
Development

No branches or pull requests

1 participant