-
Notifications
You must be signed in to change notification settings - Fork 22
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
Only the first instance of a package shows up on search and lists #124
Comments
That links points to the
That links points to the
It appears on page 20 https://index.ros.org/packages/page/20/released/, released for indigo? Something really funky is going on here.
It does show up on the list, but not as a released package https://index.ros.org/search/?q=rviz2. |
We have to check what rosindex is ingesting and how it is ingesting it. |
It's this one, right? https://github.com/ros2/geometry2/
Agreed 😅
Yeah that's what I meant. All packages show anyway, but I'd expect the following page: https://index.ros.org/packages/page/1/released/#bouncy To:
|
So geometry2 does show in the list for older distros but it's lower down the list on page 3 you can find lunar The sorting is not prioritizing title match over a dependency or other metadata match. Something like angles doesn't have as many other results with it in the metadata. But you can find indigo on page 3 for it. |
Nice catch. It would be nice to prioritize the title. |
For example, the
geometry2
package has an instance for ROS1:https://index.ros.org/r/geometry2/github-ros-geometry2/#melodic
And another for ROS2:
https://index.ros.org/r/geometry2/github-ros2-geometry2/#bouncy
But if I search for
geometry2
, only the melodic package shows up, which makes it look like it hasn't been released for other distros:https://index.ros.org/search/?q=geometry2
This doesn't happen for packages with only one instance across ROS1 and 2, for example:
https://index.ros.org/search/?q=angles
And an even bigger issue is that packages with multiple instances don't even show as released in ROS 2, note how
geometry2
is not in this list:https://index.ros.org/packages/page/1/released/#bouncy
In fact, that list is way too short, so I suspect there may be other issues too. For example, RViz2 has only one instance, and is released on Bouncy, but it doesn't show on the list above.
https://index.ros.org/p/rviz2/github-ros2-rviz/#bouncy
The text was updated successfully, but these errors were encountered: