indecis-it / data

Italian electoral programs data. Open data source for indecis.it
Creative Commons Attribution 4.0 International
6 stars 0 forks source link

installare versione di miller recente #22

Closed dennisangemi closed 2 years ago

dennisangemi commented 2 years ago

Ciao @aborruso, ti disturbo per commissionarti una missione speciale.

Questo è il repo in cui buttiamo i dati di indecisit. Usiamo questa action per fare tante cose ma a noi attualmente interessano gli step in cui si installa miller e in cui si convertono i csv in json.

Il problema: ogni tanto miller si mangia un ] alla fine del json.

A giudicare dal log dell'action , credo che incappiamo in questo vecchio bug perchè pare che l'action installi una versione vecchiotta di miller.

Io ho provato a scopiazzare dai tuoi script per provare ad installare una versione più aggiornata di miller ma non sono riuscito nell'impresa

Potresti svelarci il segreto per installare una versione recente di miller tramite gh action? Grazie ;)

aborruso commented 2 years ago

Ciao @dennisangemi mi puoi allegare uno dei file JSON di input che manda in errore lo script?

dennisangemi commented 2 years ago

Ciao @aborruso , questo input.csv è il csv che convertito in json è privo di una ].

Il comando che lancio per eseguire la conversione è

mlr --icsv --ojson --jlistwrap --jvstack cat input.csv > output.json

Ottengo questo warning/errore

mlr: unmatched double quote at line 12

e il json in output non ha la ] alla fine

aborruso commented 2 years ago

caro @dennisangemi non ho errori, né con la versione 5.10.2, né con la 6.4.0.

Ho visto la tua action. Pesca una versione di Miller vecchia, la 5.7.

L'ho aggiornata, adesso anche quella è 5.10.2. Riprova a lanciare l'action

dennisangemi commented 2 years ago

Ciao @aborruso, grazie per l'aggiornamento.

Non riesco comunque a installare eseguibile tramite action perchè ottengo questo errore .

Provavo ad installare con questo codice qui https://github.com/indecis-it/data/blob/main/.github/workflows/main.yml#L36

p.s. abbiamo risolto il problema del json rotto con la #23 ma sarei comunque curioso di capire come installare eseguibile <3

aborruso commented 2 years ago

Ciao @dennisangemi , penso che sia una questione di variabili di ambiente.

Devi esportare il PATH nello stesso run. Perché la variabile poi sparisce.

Quindi da

      - name: Test Install miller
        run: |-
          curl -kL "https://raw.githubusercontent.com/ondata/covid19italia/master/bin/mlr" > mlr
          mkdir -p ~/bin
          cp mlr ~/bin
          cd ~/bin
          chmod +x mlr
          export PATH=$PATH:~/bin

      - name: Build json
        run: |-
          chmod +x script/convert.sh
          script/convert.sh

a

      - name: Test Install miller
        run: |-
          curl -kL "https://raw.githubusercontent.com/ondata/covid19italia/master/bin/mlr" > mlr
          mkdir -p ~/bin
          cp mlr ~/bin
          cd ~/bin
          chmod +x mlr

      - name: Build json
        run: |-
          export PATH=$PATH:~/bin
          chmod +x script/convert.sh
          script/convert.sh
dennisangemi commented 2 years ago

Arcano svelato! Grazie grazie grazie @aborruso <3