modin-project / modin-project/modin-performance-analysis
Parse Current PR Perf File
Open
Nobody has claimed this yet.
- Dominant language
- Jupyter Notebook
- Stars
- 6
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
The current code (with lines commented out) will only parse perf files for commits merged into master. Add support for parsing the perf file for the PR commit.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read the current parsing code and the commented-out lines that restrict perf files to commits merged into master. Trace how the PR commit is identified and parsed, then verify that its perf file is included without breaking the existing master-commit behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- jupyter-notebook, python
- Domain
- performance
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100