• Home
  • About
    • Anirban photo
    • About Me
    • Email
  • Blog Posts
    • Writings
    • Tags
  • Skill Set

data.table work diary: Mar 11 - 15

  • Mar 11, ‘24:

1) Fixed issues with old PRs to demonstrate my Action with atime on my fork of data.table, but had to delete my fork (also learned that fetching all branches is a must or that the ‘copy the main branch only’ option should be unchecked when forking).

2) Encountered an error with git rev-parse not finding references (#27) and solved it.

3) Got merge-base to appear on plots.

4) Working towards creating a good example to share with the community :D

  • Mar 12, ‘24:

1) Made a couple of changes to my action to publish it to the Marketplace. A bit of research. Published: Try it!

2) Made several releases to my action as I kept encountering and fixing issues (please check #29). I thoroughly tested it and provided a concrete template/example to use in the readme, along with other details I wrote of.

  • Mar 13, ‘24:

1) The latest release (v1.1.0) allows for the action to work on any GitHub Repository for an R package (and not just data.table; generalization of safe directory checks is working)

2) After much trial-and-error, resolved an issue where merge-base was not showing up consistently even though it appeared before.

3) Made PRs to showcase examples of my action working on a data.table refork: - Using my action from the GA-Marketplace and for the latest master branch and a fork of it with no regression for two test cases: #1 - Using my action directly from .github/workflows and for branches before and after #4491 got merged with observable regression (based on test case #5424) from that PR: #2

  • Mar 14, ‘24:

1) Working on the changes required for this PR as pointed out by Toby: - Making test case names consistent (easy, found that #5463 fixed that and a few others) - Trying to add another test that is not related to that performance regression for e.g.), WIP

  • Mar 15, ‘24:

1) Still trying to get a third example to work locally for me. Doris hasn’t responded yet (comment) to having another example apart from the ones I got to work with atime::atime_pkg (which I need to make it work with my action), so I’m trying to make one myself for scratch now.

2) Tried to close issue #6003.