search

ondata / appaltipop

ETL scripts and issue tracking for AppaltiPOP project.
https://www.appaltipop.it
MIT License
1 stars 0 forks source link

[Discussione] Ridefinizione completa dei nomi degli attributi #9

Closed jenkin closed 4 years ago

jenkin commented 4 years ago

I nomi attuali degli attributi hanno delle criticità:

Tenendo anche conto che non sono nascosti, ma si vedranno in una eventuale API.

aborruso commented 4 years ago

Ale, non mi torna una cosa: tu non elabori quindi il json in OCDS?

Ne parliamo tra un po'

jenkin commented 4 years ago

Al momento io parto dai file nella cartella json/ di questo repo.

jenkin commented 4 years ago

Proposta: usiamo i nomi dei campi di OCDS con prefisso OCDS: e tutti gli altri con prefisso APPALTIPOP: e nome custom (in inglese senza spazi).

patrunomeister commented 4 years ago

Ho rinominato tutti i campi. Vorrei che diate una occhiata

jenkin commented 4 years ago

Perfette le chiavi ocds:, ecco la mia proposta per quelle appaltipop:...

{
    "appaltipop:releases/0/tender/partecipants/total": 1.0,
    "appaltipop:releases/0/partecipants/total": 3290.0,
    "appaltipop:releases/0/partecipants/mean": 1.93,
    "appaltipop:releases/0/suppliers": Array[1][
      {
        "appaltipop:releases/0/supplier/amountbysinglebuyer/total": 5635.46,
        "appaltipop:releases/0/supplier/contractsbysupplier/total": 2.0
      }
    ],
    "appaltipop:releases/0/redflags": Array[1][
      {
        "appaltipop:releases/0/redflag/code": "01",
        "appaltipop:releases/0/redflag/description": "Low number of mean bidders per tender for competitive processes"
      }
    ],
    "appaltipop:releases/0/buyers": Array[1][
      {
        "ocds:releases/0/buyer/id": "IT-CF-84006890481",
        "ocds:releases/0/buyer/name": "COMUNE DI PRATO"
      }
    ]
  }
jenkin commented 4 years ago

@patrunomeister chiedo scusa, propongo di usare sempre il camelCase:

Inoltre per i partecipanti meglio usare la forma "participants" con la "i" e non la "e"... :)

jenkin commented 4 years ago

Ma l'OCDS chiama "parties" quello che per noi sono i "suppliers"? Ci va bene cambiare nome?

patrunomeister commented 4 years ago

@jenkin OCDS chiama parties tutti: suppliers, tenderers, buyers,.... Poi ognuno di questi ha un ruolo

jenkin commented 4 years ago

@patrunomeister l'attributo appaltipop:releases/0/supplier/contractsBySupplier/total mi sembra abbia un nome sbagliato, si tratta del numero totale di contratti che quello specifico supplier ha con quello specifico buyer, giusto? Quindi dovrebbe essere appaltipop:releases/0/supplier/contractsBySingleBuyer/total.

Se poi contracts === tenders, userei sempre quest'ultimo termine, quindi appaltipop:releases/0/supplier/tendersBySingleBuyer/total.

Mi rimane il dubbio su come possano essere considerati questi numeri se ci sono più buyers associati al singolo tender...

patrunomeister commented 4 years ago

Ho fatto la modifica. Ne parliamo domani in call. Per il momento chiudo l'issue

jenkin commented 4 years ago