clarin-eric / standards

work space for the Standards and Interoperability Committee
https://www.clarin.eu/content/standards
3 stars 13 forks source link

KPI page: enhancement needed #273

Open bansp opened 3 months ago

bansp commented 3 months ago
  1. It seems to me that we need a date stamp for the KPI calculation.
  2. We also need to make this calculation easy to verify, so the phrase "depositing centres" needs to link to the list of exactly those centres that we use for calculation.
  3. The last paragraph has become gibberish. It needs to be fixed ASAP, before release 2.7.0.
bansp commented 3 months ago

Regarding point (3) above,

The SIS provides information on all <a href="{app:link("views/list-centres.xq")}">centres that allow
                                    for data depositions</a>

lacks the appropriate filter for centres that allow data deposition. (Apart from that, the "all" is a bit bold and needs to be even further qualified. "All CLARIN" seems the easiest to do.)

bansp commented 3 months ago
  1. This is wrong, isn't it:
    <td class="column">CLARIN deposition centres recorded in the SIS</td>
    <td class="column">{$numOfBCentres}</td>

We've been through this for the paper (when recalculating the KPI at the time of submission), but we may have forgotten (?) to modify the KPI page as well.

bansp commented 3 months ago

I have handled point 3., and point 4. was just a misnamed variable, fixed on the way to release 2.7.0.

margaretha commented 1 month ago

We can add current timestamp for the KPI calculation.