georgetown-cset / parat

🦜 PARAT: CSET's Private-sector AI-Related Activity Tracker
https://parat.cset.tech
Other
4 stars 0 forks source link

Recent pubs growth value of -100% #458

Open za158 opened 1 week ago

za158 commented 1 week ago

image

za158 commented 1 week ago

The issue being: how can the average change be -100% for more than one year? Or are years with 0 change getting dropped?

jmelot commented 6 days ago

Years with 0 in the denominator of the pct change calculation are getting dropped. These cases seem to crop up when the org does very sporadic publishing, a paper or patent every now and then. I agree this might be confusing... maybe we should update the metric or just not display the percentage change for these orgs.

za158 commented 5 days ago

How about we display "n/a" with a tooltip indicating "Insufficient data to calculate"? @brianlove

brianlove commented 5 days ago

@za158 This "N/A"+tooltip is for any growth column with a total of -100?

jmelot commented 5 days ago

Yeah I'm not sure we should do this for any such column. We could instead update the preprocessing code so that in cases where any of the pct changes would have a 0 in the denominator, we return null, triggering the existing N/A on the front end

za158 commented 5 days ago

Seems reasonable