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

Update sqlalchemy eups package

    XMLWordPrintable

    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 [X] 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 [X] 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:

                  Jenkins

                  No builds found.