Filter range does not filter if min == max · Issue #477 · gephi/gephi · GitHub
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

Filter range does not filter if min == max #477

Closed
sheymann opened this issue Nov 18, 2011 · 8 comments
Closed

Filter range does not filter if min == max #477

sheymann opened this issue Nov 18, 2011 · 8 comments
Labels
Milestone

Comments

@sheymann
Copy link
Member

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.

@ghost ghost assigned mbastian Nov 18, 2011
@mbastian
Copy link
Member

Use a Equal Filter, not a Range Filter to do that.

@suthers
Copy link

suthers commented Feb 4, 2012

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.

@mbastian
Copy link
Member

mbastian commented Feb 4, 2012

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.

@suthers
Copy link

suthers commented Feb 4, 2012

Fair enough - you're doing a lot and I appreciate it. I just have
to advocate for the software working as expected being more
important than the user working as expected.

@sheymann sheymann reopened this Feb 25, 2012
@sheymann
Copy link
Member Author

We should not close this bug, because we may find someone who will take the time to fix it.

@tarasklas
Copy link
Contributor

I fixed that bug, however this looks a bit weird.
Seems that Range everywhere is inclusive, but it processed as an error.
Mathieu, could you please take a look at tarasklas@fa44b46?

@mbastian
Copy link
Member

mbastian commented Mar 2, 2012

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.

@mbastian
Copy link
Member

mbastian commented Mar 2, 2012

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.

@mbastian mbastian closed this as completed Mar 2, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants