Uploaded image for project: 'Python Driver'
  1. Python Driver
  2. PYTHON-4289

Describe new patching feature to AI/ML Testing Pipeline README

    • Type: Icon: Task Task
    • Resolution: Fixed
    • Priority: Icon: Unknown Unknown
    • 5.0
    • Affects Version/s: None
    • Component/s: AI/ML
    • None
    • Fully Compatible
    • Python Drivers
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      Context

      During ChatGPT Retrieval Plugin integration, we ran into build issues on Evergreen hosts. In this case, because the package failed to build from source (it required a library that wasn't available on the host) and no wheel was on PyPI. However, the requirement was actually optional. So just a one-line change to `pyproject.toml` did the trick.

      I realized that we could easily get this working without requiring a pull-request (and acceptance) on the upstream simply by applying a git patch file. This is a common technique used by `conda package` maintainers, as they often have to build for a more broad set of scenarios than the original authors intended.

      So, I added a mechanism to apply a collection of patches. This is done generically, and I believe will come in useful going forward.

      Definition of done

      Add instructions on how to add patches in the README.md. Refer to chatgpt as example.

            Assignee:
            casey.clements@mongodb.com Casey Clements
            Reporter:
            casey.clements@mongodb.com Casey Clements
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: