AINovice2005 commited on
Commit
1a93aeb
·
verified ·
1 Parent(s): 411d311

update paths.

Browse files
Files changed (1) hide show
  1. src/index.qmd +5 -5
src/index.qmd CHANGED
@@ -18,7 +18,7 @@ To understand contribution patterns, I converted raw GitHub activity into struct
18
 
19
  One of the first patterns comes from the **monthly PR volume**, which provides a high-level view of project activity over time. The visualization below shows how merged PRs fluctuate month-to-month, highlighting periods of higher contributor engagement (such as November 2024 and April 2025) and quieter phases (such as August 2024 or September 2025). These variations help understand review load, maintainer bandwidth and seasonal contribution patterns before diving into more granular metrics.
20
 
21
- ![Temporal Trends of PR Contributions](src/images/temporal_trend_graph.png)
22
 
23
  Before diving into patterns, let's establish what the key metrics actually measure:
24
 
@@ -40,7 +40,7 @@ This distribution highlights a consistent pattern across open-source ecosystems:
40
 
41
  To understand when contributors (including myself) tended to open PRs, I also generated a day-of-week × hour-of-day activity heatmap. The pattern shows that most PRs were created during weekday afternoons and early evenings, aligning with typical contributor availability and maintainer review windows.
42
 
43
- ![Activity Heatmap of PR Creation](src/images/activity_heatmap.png)
44
 
45
  As the size of my dataset grew, several consistent patterns emerged, both about my contribution style and about how maintainers respond. The top 3 insights were seen as follows:
46
 
@@ -139,7 +139,7 @@ Examples can be seen in this [**PR**](https://github.com/skorch-dev/skorch/pull/
139
 
140
  3. **Treat the CI Green Light as Non-Negotiable Before Merging**
141
 
142
- ![The CI Green Light](src/images/green_light_ci.png)
143
 
144
  Before committing your changes, it is important to execute linting and tests on the changed files to reduce the CI failure rate. CI also helps maintainers **reduce bugs and maintain consistency** in the codebase and if CI is difficult to restore to green, request to get pointers to fix the CI before merging the PR.
145
 
@@ -151,7 +151,7 @@ Since maintainers often batch-review PRs, mixed-scope PRs get deferred because t
151
 
152
  If we look at the **box plot showing the distribution of merge times**, we see a clear pattern: PRs that follow these practices consistently merge faster, while PRs having large, multi-scope or under-discussed contributions take significantly longer and sometimes become “stuck” for weeks or months.
153
 
154
- ![Boxplot for Response Times](src/images/response_time_boxplot.png)
155
 
156
  5. **The "Praise Specifics, Accept Generics" Mindset**
157
 
@@ -167,7 +167,7 @@ A consistent pattern observed across projects is that **timely responses from co
167
 
168
  Across the data, **PRs with a response time under 48 hours merged 4.2 days faster** than PRs where contributors responded after 72 hours. Staying engaged within the first two days keeps your PR visible and signals intent.
169
 
170
- ![Bar Chart of Merge Timing](src/images/merge_time_graph.png)
171
 
172
  It is also worth noting that maintainers may occasionally miss a PR due to workload or notification overload. A polite follow-up, usually after a few days is not only appropriate but often necessary to bring the PR back into the review cycle.
173
 
 
18
 
19
  One of the first patterns comes from the **monthly PR volume**, which provides a high-level view of project activity over time. The visualization below shows how merged PRs fluctuate month-to-month, highlighting periods of higher contributor engagement (such as November 2024 and April 2025) and quieter phases (such as August 2024 or September 2025). These variations help understand review load, maintainer bandwidth and seasonal contribution patterns before diving into more granular metrics.
20
 
21
+ ![Temporal Trends of PR Contributions](images/temporal_trend_graph.png)
22
 
23
  Before diving into patterns, let's establish what the key metrics actually measure:
24
 
 
40
 
41
  To understand when contributors (including myself) tended to open PRs, I also generated a day-of-week × hour-of-day activity heatmap. The pattern shows that most PRs were created during weekday afternoons and early evenings, aligning with typical contributor availability and maintainer review windows.
42
 
43
+ ![Activity Heatmap of PR Creation](images/activity_heatmap.png)
44
 
45
  As the size of my dataset grew, several consistent patterns emerged, both about my contribution style and about how maintainers respond. The top 3 insights were seen as follows:
46
 
 
139
 
140
  3. **Treat the CI Green Light as Non-Negotiable Before Merging**
141
 
142
+ ![The CI Green Light](images/green_light_ci.png)
143
 
144
  Before committing your changes, it is important to execute linting and tests on the changed files to reduce the CI failure rate. CI also helps maintainers **reduce bugs and maintain consistency** in the codebase and if CI is difficult to restore to green, request to get pointers to fix the CI before merging the PR.
145
 
 
151
 
152
  If we look at the **box plot showing the distribution of merge times**, we see a clear pattern: PRs that follow these practices consistently merge faster, while PRs having large, multi-scope or under-discussed contributions take significantly longer and sometimes become “stuck” for weeks or months.
153
 
154
+ ![Boxplot for Response Times](images/response_time_boxplot.png)
155
 
156
  5. **The "Praise Specifics, Accept Generics" Mindset**
157
 
 
167
 
168
  Across the data, **PRs with a response time under 48 hours merged 4.2 days faster** than PRs where contributors responded after 72 hours. Staying engaged within the first two days keeps your PR visible and signals intent.
169
 
170
+ ![Bar Chart of Merge Timing](images/merge_time_graph.png)
171
 
172
  It is also worth noting that maintainers may occasionally miss a PR due to workload or notification overload. A polite follow-up, usually after a few days is not only appropriate but often necessary to bring the PR back into the review cycle.
173