What is the logic behind Column/Bar chart sequential order?

ada-discovery / ada-issues

0 stars 0 forks source link

What is the logic behind Column/Bar chart sequential order? #178

Closed gh-osh closed 3 years ago

gh-osh commented 3 years ago

Can anyone please explain why the column order are like this?

For that chart I would expect ..the columns orders will be Visit 0 to Visit 6.. but it is Visit 0, Visit 6, Visit 1, Visit 2, Visit 3. Nor it is sorted on the variables values, neither total count (Ada usually follow sorted order on numeric count, that is also very problematic)

If I change the variable values Visit 0 to only 0 (and so on), the chart becomes like this

:exploding_head:

LBolzani commented 3 years ago

Good afternoon Soumya,

I did an update on a new branch to fix this problem. As soon as possible I'll deploy on the test server then you can check/test the feature.

You can enable the feature when you're going to create a new chart in the Edit view settings, there is a new checkbox called Order by Field

Luca

LBolzani commented 3 years ago

Hello Soumya, you can test the feature at this address https://10.240.16.149/ .

Luca

gh-osh commented 3 years ago

Yes .. I will 🚩🚩

LBolzani commented 3 years ago

@soumyabrataghosh did you have time to check the feature?

sherzinger commented 3 years ago

ping @soumyabrataghosh

gh-osh commented 3 years ago

today .. I will

LBolzani commented 3 years ago

@soumyabrataghosh I need that you verify the issue. Could you give me a feedback?

gh-osh commented 3 years ago

@LBolzani I don't have access to any dataset on https://10.240.16.149/ ... how to check?

sherzinger commented 3 years ago

@soumyabrataghosh that was an unintended side effect of moving the permission system into Keycloak. I added you to the admin role for the test client. Can you please check again?

gh-osh commented 3 years ago

Tonight again.

On Tue, Sep 28, 2021 at 8:07 AM Sascha Herzinger @.***> wrote:

@soumyabrataghosh https://github.com/soumyabrataghosh that was an unintended side effect of moving the permission system into Keycloak. I added you to the admin role for the test client. Can you please check again?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ada-discovery/ada-issues/issues/178#issuecomment-928883481, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD4I3E4K2RLIDJWFJTMWF3UEFLSLANCNFSM5AN6GLXA .

sherzinger commented 3 years ago

@soumyabrataghosh please? :-)

sherzinger commented 3 years ago

Reporter didn't respond. Issue considered solved for now.

gh-osh commented 3 years ago

Agreed.