-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider adding other comparison lines #128
Comments
I'm thinking this wouldn't be too difficult to add but what say you @brianlove ? If you think it will be particularly time-consuming maybe we save this for after we've got the MVP together |
@jmelot I've been thinking about how to best handle this, and there are several ways that I've thought of:
I think Option 4 is the best route to take, as it minimizes data duplication while being fairly easy to implement. Here's a rough draft of the structure for the {
1234: {
articles: {
ai_publications: {
counts: [ 2, 14, 55, ... ],
},
ai_pubs_top_conf: {
counts: [ 0, 2, 1, ... ],
},
// ...
},
patents: {
// ...
},
other_metrics: {
// ...
},
},
567: {
// ...
},
// ...
} This structure could potentially be simplified if we only want comparisons on a small number of charts. Questions:
|
I don't think we need to compare rankings, especially if it makes a material difference to the engineering. |
Building on #84:
If there is a not nightmarish way to implement this, we might want to include additional comparison lines for some companies. Like when you're looking at a chart for Google, you could see the S&P 500 line, but also a line for Microsoft, Tencent, Meta. Something like that.
The text was updated successfully, but these errors were encountered: