Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

See Trading reviews on Pull Requests for how to get immediate attention to that PR!

Notes

Meeting schedule

  • Tim will be out of office next week; there will be no dev meeting next week
  • Our next meeting will be on Aug 21

Merger discussion

  • On the technical side, a plan is developing how a merger could occur
  • The strategy group is in the process of finalizing the Strategic Vision Document, which will soon be shared with broader community
  • No final decisions have been made
  • We will continue to share more updates as they develop

DSpace 10.0

  • No updates, first deadlines don't come until later this year

Other Topics

  • Aggressive bot discussion via Google Scholar
    • Please add information how you are dealing with aggressive bots on the ticket in GitHub
    • There is a PR that helps update the rate limiter; this is a light way to lessen the impact of aggressive bots
      • This only counts server side hits, does not affect client side
      • Currently allows for 500 requests per minute; Tim has done some testing to limit this to 20 r.p.m. 
      • There are a lot of contingencies here, it's going to difficult to make this perfect, but there may be a sweet spot we could aim for
      • Please provide feedback if you have used this rate limiter and have found a good value
      • Important Note: we need to be careful about browser caching, as caching may prevent catching all requests in tests and provide inaccurate test results
      • Kim was added as a reviewer to the PR
      • Question: What values for rate limiter are used in the sandbox.dspace.org? Answer: The specific rate limiter built into DSpace has default values; need to ask to find out more details
      • If we could document or recommend tools for especially smaller sites with less resources, this would be a good goal for future discussions and agendas
    • Fedora is also spearheading a platform agnostic discussion series around the impact of AI bots on repositories
  • JUnit 5 migration, update by Kim
    • This PR has been sitting around for a while; Kim has been running up against frequent merge conflicts, needing to update this frequently
    • Kim will let the devs know when it's ready to be reviewed
    • Tim will review once it's ready to be reviewed
    • Will try to move this forward quickly

Upcoming Topics

  • Migration to NX
    • Initial preparation work for NX migration PR was done by 4Science, to prepare for migration to NX
    • Goal of this preparatory PR is to isolate part of the application in order to create the first library after migration to NX; this PR contains mostly patch changes
    • This is mostly moving the code around; everything works as is, code was restructured to remove dependencies
    • Art and Tim will try to review this initial PR; anyone else interested, please feel free to help out
  • Angular dynamic decorators
    • We are waiting to see what happens with the NX migration PR; It would be good to have a decision whether we move to NX before continuing work on the decorator PR
    • Giuseppe will lead discussion on NX migration proposal at an upcoming meeting (Aug 21) 
      • Especially as to the benefits of switching to NX from a developers perspective
    • In the meantime, everyone is encouraged to review the initial PR and the wiki page on the initial proposal to prepare questions for Giuseppe

10.0 Board

  • There are two PRs in the Reviewer Approved column; first one: Sascha should resolve the merge conflicts so it can be merged; the second: please help out with this one in case Tim doesn't get to it in the next couple of days
  • #10491:
    • Tim will add feedback that it looks good overall; would like to see one extra test to AIP file
    • If you have any other thoughts or feedback, please add it to the PR


Action items