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

feat(#245): include workflows to d0-numerical #270

Merged
merged 7 commits into from
Dec 3, 2024
Merged

Conversation

h1alexbel
Copy link
Owner

@h1alexbel h1alexbel commented Dec 3, 2024

In this pull I've added workflows count to d0-numerical.csv dataset, and simplified naming.

ref #245
History:


PR-Codex overview

This PR focuses on updating the data handling and processing logic in the sr-data project to align with new column names in CSV files, enhancing the workflow management and ensuring accurate data representation.

Detailed summary

  • Updated column names in to-score.csv to match new schema.
  • Changed references from pulls_count to pulls in test_pulls.py, pulls.py, and scores.py.
  • Adjusted scoring calculations in scores.py to use new column names.
  • Added copying of numerical.csv to the collection in data.sh.
  • Updated JSON mappings in ghminer.json to reflect new field names.
  • Enhanced workflow tracking in workflows.py by adding workflow counts.
  • Introduced a new test in test_workflows.py to validate workflow counts.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@h1alexbel
Copy link
Owner Author

@rultor merge

@rultor
Copy link
Collaborator

rultor commented Dec 3, 2024

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here.

@rultor rultor merged commit 547814f into master Dec 3, 2024
17 checks passed
@rultor
Copy link
Collaborator

rultor commented Dec 3, 2024

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 12min).

@h1alexbel h1alexbel deleted the 245-numerical branch December 3, 2024 13:31
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