Skip to content
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

"Stars Over Time" release tag bug #600

Open
LRWeber opened this issue Jan 11, 2023 · 2 comments
Open

"Stars Over Time" release tag bug #600

LRWeber opened this issue Jan 11, 2023 · 2 comments

Comments

@LRWeber
Copy link
Member

LRWeber commented Jan 11, 2023

The "Number of Stars Over Time" graphs on the individual repo pages are meant to include markers along the timeline for each release tag. However, it appears that the current visualizations end up with only a single marker showing release version numbers as a range, rather than showing a marker for each version. (Sample shown from LLNL/scraper)

image

@LRWeber
Copy link
Member Author

LRWeber commented Jan 11, 2023

This appears to be caused by additional data processing intended to prevent multiple release lines tightly clumped together.
This behavior could be improved by adjusting the tolerance variable and/or scaling it according to the total time range of the individual graph.

var start = -1;
var current = -1;
var past = -1;
var tolerance = 40;
var indexArray = [];
// Process release data to prevent clumps
for (var i = 0; i < releaseDates.length; i++) {
var date = parseTime(releaseDates[i].slice(0, releaseDates[i].indexOf('T')));
current = x(date);
if (start < 0) {
start = current;
past = i;
i--;
} else if (current - start < tolerance) {
past = i;
} else {
indexArray.push(past);
start = -1;
current = -1;
past = -1;
i--;
}
}

@hauten
Copy link
Contributor

hauten commented Nov 21, 2023

Still an issue, e.g., https://software.llnl.gov/repo/#!/openzfs/zfs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants