Skip to content
This repository has been archived by the owner on Jun 4, 2022. It is now read-only.

Problem with "Liguria" map #2

Closed
ferradario opened this issue Mar 25, 2020 · 8 comments
Closed

Problem with "Liguria" map #2

ferradario opened this issue Mar 25, 2020 · 8 comments
Assignees

Comments

@ferradario
Copy link

image

When trying to visualize the "Liguria" data, the maps shows Tuscany provinces instead.

@ulmozzz
Copy link

ulmozzz commented Mar 26, 2020

Folow up da issue #3
Avevo visto questa issue

Visto che vedo il problema ora (26 marzo, ore 10 a.m.) deduco che non possa essere stato pienamente corretto. Sbaglio? O forse, semplicememente, il file usato online dal sito "https://covid19-ita.leonidev.com/" non è allineato a quel build?

Prove fatte:

  • cancellare chache
  • usare versione sul sito "app.powerbi.com"
  • usare un browser diverso, in cui il sito non è mai stato apert
  • usare un device diverso (android)
  • usare un device diverso (iPhone)
    in tutti i casi il problema c'è ancora

Da come vedo, il problema è stato corretto solo per la vista "valori assoluti" e non per la vista "incidenza".

A livello di pura SPECULAZIONE (mi piacerebbe sapere se ci ho preso) potrebbe essere che si selezionino sulla mappa le "regioni" con un particolare indice nell'array invece che le "province".
Infatti selezionando sulla tabella in alto a destra le province "per nome" (Genova, La Spezia, etc.) la mappa si aggiorna mostrando di volta in volta una sola regione delle 4, senza suddivisione per province.
Questo è veramente un blind shot, l'ho aggiunto solo per curiosità

Grazie ancora per il sito comunque. Molto utile e ben fatto.

@ulmozzz
Copy link

ulmozzz commented Mar 26, 2020

ho notato anche altri "glitches"

sono minori, sempre relativi alle mappe

fammi sapere se vuoi che li segnali

come detto, sarei anche interessato a contribuire per risolverli, ma mi sembra siano all'interno di codice non pubblicato, quindi non posso farci nulla

se mi sbaglio, fammi sapere

@leoniDEV leoniDEV reopened this Mar 26, 2020
@leoniDEV
Copy link
Owner

Certo che è allineato con la build e tutto il materiale che ho usato è nel repo (dovesse mancare qualcosa basta segnalarlo e lo metto a disposizione)

Per la mappa a pagina 4 serve il file provincie.json.
Ho verificato ancora e tutto sembra funzionare bene dalla mia parte, adesso ho provato a rinominare il file provincipe.json in prov.json.

Fammi sapere se si risolve anche da parte tua, dopo di che chiudo l'issue se tutto è andato bene

@leoniDEV
Copy link
Owner

Nevermind ho trovato il problema, è stata una mia dimenticanza da pivello 😳 (magari anche un po' la stanchezza visto che ci sto ancora lavorando da tutta la notte per implementare un sistema migliore per aggiornare i dati).

In realtà le mappe sono 2 sovrapposte, una per la vista con i numeri assoluti ed una con la vista con l'incidenza, quando si attiva in una vista se ne spegne una e quando si attiva l'altra vista si spegne l'altra.
Ho corretto la mappa nella vista con i dati assoluti ma non quella con i dati relativi all'incidenza.

@ulmozzz
Copy link

ulmozzz commented Mar 26, 2020

certamente, era chiaro fosse una svista, visto che credo fosse lo stesso baco di ieri

appena vedo la issue chiusa riprovo, per ora sembra non sia cambiato nulla

fammi sapere se ti interessano anche gli altri minor bugs

@leoniDEV
Copy link
Owner

certo che mi interessano.

Per io aspetto tue conferme per chiudere, da me sembrano funzionare tutte e due le mappe

@ulmozzz
Copy link

ulmozzz commented Mar 26, 2020

confermo che ora funziona anche a me

riguardo al resto, si è corretto con questa release; avevo notato che, sempre in modalità "incidenza", se per esempio per Lombardia cliccavi in tabella in alto a destra sulla riga BRESCIA, la mappa mostrava la Basilicata
Brescia_Basilicata

sempre come minor thing (non è degna nemmeno lontanamente di chiamarsi bug,
Se clicchi su una provincia sulla tabella in alto a destra con i valori "assoluti", la mappa continua a mostrare la regione

Lombardia_assoluti

se invece fai la stessa cosa in modalità "incidenza" la mappa fa zoom sulla provincia
Lombardia_incidenza

Personalmente preferirei un ibrido: lasciare la regione (che è facilmente riconoscibile dall'utente) ma colorerei la provincia in modo diverso, evidenziandola. Così come accade (in entrambe le visualizzazioni), se invece di cliccare sulla RIGA della TABELLA clicchi sulla provincia all'interno della mappa

Lombardia_click_su_mappa

Ovviamente sono commenti da profano che non deve fare il lavoro, vedi tu.

Ciao!

@leoniDEV
Copy link
Owner

Il comportamento corretto è quello che avviene nella vista dei valori assoluti, almeno era quello che pensavo fosse più adeguato, per il resto non posso fare niente, non ho molto controllo sul comportamento delle visual, il modo in cui queste reagiscono ai filtri ed agli slicer è al di fuori del mio controllo

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

No branches or pull requests

3 participants