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

Network creation query without CPVs results in a network without edges and nodes #517

Open
nightsh opened this issue Mar 30, 2020 · 7 comments
Assignees
Labels

Comments

@nightsh
Copy link
Member

nightsh commented Mar 30, 2020

Example network: https://defense.tenders.exposed/network/395ec875-ec97-41f3-afe2-083d5dccb1c9

Steps to reproduce:

  • go to query builder
  • select at least a country
  • select no actor
  • select no CPV, click on the "proceed anyway" warning
  • at the final step, submit the request
  • loading carousel appears, but the nework that comes back from the backend is invalid, hence nothing is rendered
@nightsh nightsh added the bug label Mar 30, 2020
@nightsh nightsh changed the title Network creation queries without CPVs results in a network without edges and nodes Network creation queries without CPVs result in a network without edges and nodes Mar 30, 2020
@nightsh nightsh changed the title Network creation queries without CPVs result in a network without edges and nodes Network creation query without CPVs results in a network without edges and nodes Mar 30, 2020
@georgiana-b
Copy link
Member

Hey @nightsh! Thanks for pointing out.

This is not a bug in Elvis actually. The original data just happens to lack the name of the bidders sometimes.
https://drive.google.com/file/d/1TNYwKIbPUwHowEwv_DFOVXfCiRnOmC_C/view?usp=sharing

It's one of those cases where it would really help to see the network query 😄Then it would become pretty clear that it has anything to do with the CPVs at all. This is how the network query looks in the db. It's perfectly fine but you're getting those nameless bidders because you chose Greece.

image

@zufanka We actually talked about this in #428. The idea was to show the nameless bidders/buyers as grey nodes. I see that the issue was closed a few days ago. Does that mean we're dropping that idea?

@georgiana-b georgiana-b assigned zufanka and unassigned georgiana-b Apr 3, 2020
@zufanka
Copy link
Member

zufanka commented Apr 7, 2020

That issue was about them not appearing at all. The issue was closed because they now do appear as a node with no name.

@georgiana-b
Copy link
Member

How do they appear though?

@georgiana-b
Copy link
Member

I don't understand how we decided to display the tenders who are missing the bidders

@zufanka
Copy link
Member

zufanka commented Apr 7, 2020

I am confused: #428 was about "bidders missing a name", not tenders that are missing bidders?

@georgiana-b
Copy link
Member

I am confused: #428 was about "bidders missing a name", not tenders that are missing bidders?

@zufanka This are the same thing in the data: a tender record with an empty cell in the bidder name. Of course there must've been a bidder if the tender was awarded but we don't know it.

I suggest we close this issue in favour of #524

@zufanka
Copy link
Member

zufanka commented Apr 30, 2020

@georgiana-b I think this something else. If you create a network but select no CPV codes at all, it does not select all CPV codes, instead it selects nothing at all. See https://defense.tenders.exposed/network/d0649fbb-f298-4d26-83ac-5abc592a6978/ for example

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants