-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Filter range does not filter if min == max #477
Comments
Use a Equal Filter, not a Range Filter to do that. |
I was also about to post this bug as not fixed. Apparently your reply is it's not a bug, but the interface should provide better feedback.Also, why not make it work? What's so hard? Then we can avoid others thinking it is a bug to be reported. |
Sure it could be improved but we need to prioritize and there are other UI improvements we could do in the software that could have a higher impact. But this is open to contributions. The code is in the FiltersPluginUI module, feel free to submit a patch. |
Fair enough - you're doing a lot and I appreciate it. I just have |
We should not close this bug, because we may find someone who will take the time to fix it. |
I fixed that bug, however this looks a bit weird. |
I'm working on refactoring this area. The code in the Range class and the way we update min/max is not good and I think we should simplify this. My idea is to put RangeFilter in the Filters SPI and manage ranges from the controller so we have the right graph at the right time. |
Hey this should be fixed in the latest push. Thanks megaterik for taking the time to dig into that code :-) The Range class is quite simpler now and the FilterProcessor is now responsible for updating the range. It's better this way. It also fix some problems when the Toolkit was using the API. |
I want to filter the network to see only nodes having degree = 1. It's now impossible since the last update.
I do a Degree Range, then set the range from 1 to 1. The network is not filtered.
Setting from 0 to 1 is not possible on my network because every node is connected to at least another one.
The text was updated successfully, but these errors were encountered: