Skip to content

Conversation

@rly
Copy link
Collaborator

@rly rly commented Jun 10, 2025

I thought #49 fixed the issue where data yaml in this repo were being counted, but apparently the source dir name is different from the repo name so it did not work. This fixes that issue and updates the cached data.

Cache update on the way

@rly rly marked this pull request as ready for review June 12, 2025 02:50
@rly
Copy link
Collaborator Author

rly commented Jun 12, 2025

@oruebel I am trying to update the data in this repo, but the diff for data/cloc_stats.yaml is pretty significant. All of the headers for each analyzed commit have changed. For example:

-    header: {cloc_url: github.com/AlDanial/cloc, cloc_version: 1.98, 
-        elapsed_seconds: 0.0726571083068848, files_per_second: 1775.46289696994, 
-        lines_per_second: 280261.635434099, n_files: 129, n_lines: 20363, 
+    header: {cloc_url: github.com/AlDanial/cloc, cloc_version: 2.04, 
+        elapsed_seconds: 0.0671398639678955, files_per_second: 1921.36224853962, 
+        lines_per_second: 303292.243930328, n_files: 129, n_lines: 20363, 
         report_file: 
-        /home/runner/work/nwb-project-analytics/nwb-project-analytics/data/AqNWB.yaml}
+        /Users/rly/Documents/NWB/nwb-project-analytics/data/AqNWB.yaml}

What is the recommended way of updating the data in this repo?

@oruebel
Copy link
Contributor

oruebel commented Jul 8, 2025

@rly maybe a better approach would be to move the data files to a different location (e.g., GoogleDrive) or use GitLFS. Currently the cloc stats are being computed from scratch each time (see #9) and versioning those files isn't really necessary. Storing them here was mainly a convenient solution.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants