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

fix valid commit dict #399

Merged
merged 14 commits into from
May 3, 2021
Merged

fix valid commit dict #399

merged 14 commits into from
May 3, 2021

Conversation

hsd-dev
Copy link
Contributor

@hsd-dev hsd-dev commented Feb 3, 2021

FIxes some errors for "What is the size of the change difference?" in the Jupyter notebook.

Note: some bug files have "'bad fix hash" errors when the commit is not part of a PR:

skipping bug [missing stats]: BugDescription(filename='/home/ipa-hsd/projects/rosin/robust/care-o-bot/ca7b536/ca7b536.bug')
skipping bug [missing stats]: BugDescription(filename='/home/ipa-hsd/projects/rosin/robust/geometry2/15b2e3c/15b2e3c.bug')
skipping bug [missing stats]: BugDescription(filename='/home/ipa-hsd/projects/rosin/robust/care-o-bot/43705f7/43705f7.bug')
skipping bug [missing stats]: BugDescription(filename='/home/ipa-hsd/projects/rosin/robust/universal_robot/89145c4/89145c4.bug')
skipping bug [missing stats]: BugDescription(filename='/home/ipa-hsd/projects/rosin/robust/geometry2/566092b/566092b.bug')
skipping bug [missing stats]: BugDescription(filename='/home/ipa-hsd/projects/rosin/robust/other/473ab5e/473ab5e.bug')
skipping bug [missing stats]: BugDescription(filename='/home/ipa-hsd/projects/rosin/robust/geometry2/e12e723/e12e723.bug')
skipping bug [missing stats]: BugDescription(filename='/home/ipa-hsd/projects/rosin/robust/geometry2/0c3aef3/0c3aef3.bug')
skipping bug [missing stats]: BugDescription(filename='/home/ipa-hsd/projects/rosin/robust/turtlebot/a4f35ee/a4f35ee.bug')

Also errors for confidential bugs since no repos.


Edit: Changes made in the PR:

  • generate commit statistics for the Jupyter notebook section "What is the size of the change difference?":

    • fix fix commit hashes and PRs (rationale added to bug descriptions): 39724ce, 7bda2ba, bf3595c
    • fix "no forks exist": 7195752, f4d6b7e
      additionally, also created forks on robust-rosin when missing.
    • Identify bugs that were not fixed and set fix field to null: 73cd29f, 885c016
    • Identify descriptions that are not bugs and add an identifier file NOT_A_BUG: 85a8ba1, f5d14bd
    • Finally, generate commit stats from the script fetch-commit-data: bab581c
  • Some additional changes not related to this PR:

@hsd-dev hsd-dev changed the title fix valid commit dict [WIP] fix valid commit dict Feb 3, 2021
@ChrisTimperley
Copy link
Member

What's the status on this one? It would be great to get this into master so we can run the notebook from there.

@hsd-dev
Copy link
Contributor Author

hsd-dev commented Mar 16, 2021

bad fix hash:
geometry2/0c3aef3/0c3aef3.bug: This ticket addresses the issue. Apparently the fix was never merged and the functionality was moved to tf2_geometry_msgs.h
universal_robot/89145c4/89145c4.bug: Was finally fixed in ros-industrial/universal_robot#268 in ros-industrial/universal_robot@2e078f5.
geometry2/566092b/566092b.bug: the fork was created (2018-04-09) before the commit
geometry2/15b2e3c/15b2e3c.bug: just bad format
care-o-bot/ca7b536/ca7b536.bug: correct one seems to be ipa320/cob_control@47adcee
geometry2/e12e723/e12e723.bug: does not exist. Correct one is mentioned here

bugs in other have URL https://git.code.tecnalia.com, which are private repos. confidential have no hash mentioned.

@hsd-dev
Copy link
Contributor Author

hsd-dev commented Mar 16, 2021

care-o-bot/43705f7/43705f7.bug: the PR was closed. Correct one in ipa320/cob_common@499dbb6

@hsd-dev
Copy link
Contributor Author

hsd-dev commented Apr 28, 2021

@ChrisTimperley it should be good to be merged. Could you please review it?

Copy link
Member

@ChrisTimperley ChrisTimperley left a comment

Choose a reason for hiding this comment

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

As far as I can tell by scanning through the files, this looks good.

@gavanderhoorn
Copy link
Member

This PR seems to remove the license field: was that discussed somewhere?

What's the rationale?

@gavanderhoorn

This comment has been minimized.

# Conflicts:
#	care-o-bot/43705f7/43705f7.bug
#	care-o-bot/ca7b536/ca7b536.bug
#	geometry2/15b2e3c/15b2e3c.bug
#	universal_robot/89145c4/89145c4.bug
created forks for
    'turtlebot_create',
    'rocon_app_platform',
    'turtlebot_apps',
    'cob_driver',
    'cob_common',
    'rqt',
    'gazebo_ros_pkgs',
    'ecl_core',
    'openni_launch',
    'qt_gui_core'
@hsd-dev
Copy link
Contributor Author

hsd-dev commented Apr 30, 2021

Changes made in the PR:

  • generate commit statistics for the Jupyter notebook section "What is the size of the change difference?":

    • fix fix commit hashes and PRs (rationale added to bug descriptions): 39724ce, 7bda2ba, bf3595c
    • fix "no forks exist": 7195752, f4d6b7e
      additionally, also created forks on robust-rosin when missing.
    • Identify bugs that were not fixed and set fix field to null: 73cd29f, 885c016
    • Identify descriptions that are not bugs and add an identifier file NOT_A_BUG: 85a8ba1, f5d14bd
    • Finally, generate commit stats from the script fetch-commit-data: bab581c
  • Some additional changes not related to this PR:

@hsd-dev
Copy link
Contributor Author

hsd-dev commented Apr 30, 2021

@gavanderhoorn
Copy link
Member

I've copied #399 (comment) to the PR description.

@hsd-dev
Copy link
Contributor Author

hsd-dev commented Apr 30, 2021

Edit: (Additional) changes made in the PR:

The 1st section ("generate commit statistics for the Jupyter notebook") was the purpose of the PR (referring to the "(Additional)" )

@gavanderhoorn
Copy link
Member

Edit: (Additional) changes made in the PR:

The 1st section ("generate commit statistics for the Jupyter notebook") was the purpose of the PR (referring to the "(Additional)" )

please feel free to update the PR description.

I merely felt 'hiding' it in a comment somewhere down in the timeline was not the best place for such information.

@hsd-dev
Copy link
Contributor Author

hsd-dev commented May 3, 2021

Thanks for your comments @ChrisTimperley and @gavanderhoorn.

@hsd-dev hsd-dev merged commit 041df50 into robust-rosin:master May 3, 2021
@hsd-dev hsd-dev deleted the fix_jupyter branch May 3, 2021 07:23
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.

3 participants