Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-564

Update sqlalchemy eups package

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None

      Description

      We are currently using v1.2.2 in the stack. This generates lots of annoying warnings in python 3.7 that have been fixed upstream.

      The changelog is here: https://docs.sqlalchemy.org/en/latest/changelog/changelog_12.html

      Does anyone object to me updating the stack package to v1.2.16? This will not affect people who use sqlalchemy from the conda environment (which seems to be pinned to 1.2.8).

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            This is perhaps a separate issue, but if we need the conda package to provide a dependency to some other conda package, I would prefer to retire the EUPS package. Having both opens us up to that horrible setuptools sys.path bug (in that there seems to be no guarantee which one will be used in pytest-invoked tests regardless of what EUPS and conda set PYTHONPATH to be).

            Show
            jbosch Jim Bosch added a comment - This is perhaps a separate issue, but if we need the conda package to provide a dependency to some other conda package, I would prefer to retire the EUPS package. Having both opens us up to that horrible setuptools sys.path bug (in that there seems to be no guarantee which one will be used in pytest-invoked tests regardless of what EUPS and conda set PYTHONPATH to be).
            Hide
            tjenness Tim Jenness added a comment -

            Jim Bosch, Gabriele Comoretto is working on conda-izing all our dependencies so at some point the issue with conda vs eups will solve itself.

            Show
            tjenness Tim Jenness added a comment - Jim Bosch , Gabriele Comoretto is working on conda-izing all our dependencies so at some point the issue with conda vs eups will solve itself.
            Hide
            tjenness Tim Jenness added a comment -

            No objections so I will try to do the update.

            Show
            tjenness Tim Jenness added a comment - No objections so I will try to do the update.

              People

              • Assignee:
                tjenness Tim Jenness
                Reporter:
                tjenness Tim Jenness
                Watchers:
                Andy Salnikov, Fritz Mueller, Jim Bosch, Kian-Tat Lim, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Planned End:

                  Summary Panel