Skip to content

Conversation

@Ziyaur-Rahaman
Copy link
Contributor

No description provided.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@@ -0,0 +1,429 @@
{
Copy link
Collaborator

@Nivas10798 Nivas10798 Jul 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line #2.    all_dfs = []

Where is this being used?


Reply via ReviewNB

@@ -0,0 +1,429 @@
{
Copy link
Collaborator

@Nivas10798 Nivas10798 Jul 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review for this code block

  1. Why was violin plot preferred for visualising this data
  2. The graphs are not conveying any information or the visualisation is not showing violin graph
  3. It is good to have consistency and order in the code outputs (ascending, alphabetical, having same scale)
  4. Comments are missing


Reply via ReviewNB

@@ -0,0 +1,429 @@
{
Copy link
Collaborator

@Nivas10798 Nivas10798 Jul 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WHEN page_namespace = 5 AND page_title LIKE 'Village_pump%' THEN 'Village Pump'

WHEN page_namespace = 5 THEN "Project Talk"

Why are two different values considered for same page_namespace value?


Reply via ReviewNB

@@ -0,0 +1,429 @@
{
Copy link
Collaborator

@Nivas10798 Nivas10798 Jul 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing comments


Reply via ReviewNB

@@ -0,0 +1,429 @@
{
Copy link
Collaborator

@Nivas10798 Nivas10798 Jul 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review for this code block

  1. It is good to have consistency and order in the code outputs (ascending, alphabetical, having same scale

Suggestion:

  1. It would be good to try other permutations as well, like Visualisation of all three language Wikis per Talk page type


Reply via ReviewNB

@@ -0,0 +1,429 @@
{
Copy link
Collaborator

@Nivas10798 Nivas10798 Jul 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing Comments


Reply via ReviewNB

@@ -0,0 +1,429 @@
{
Copy link
Collaborator

@Nivas10798 Nivas10798 Jul 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Scale of the graph
  2. missing comments

Reply via ReviewNB

I removed the “Village Pump” filter because it’s only used in the English Wikipedia. Other languages have different names, so keeping it would miss data in those databases.
@Ziyaur-Rahaman Ziyaur-Rahaman changed the title Branch1 Analysis of Edit Sizes, Talk Page Activity, and Unedited Pages Across Wikipedia Languages Aug 2, 2025
Could you please review the notebook
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