search

lsst / rubin_sim

Scheduler, survey strategy analysis, and other simulation tools for Rubin Observatory.
https://rubin-sim.lsst.io
GNU General Public License v3.0
41 stars 37 forks source link

DM-39490: Fix __all__ and remove useless imports #343

Closed parejkoj closed 1 year ago

parejkoj commented 1 year ago

My removal of builtins and __future__ imports (all of which were holdovers from python2) was mixed-in with my various scripted __all__ cleanups, which is why I couldn't rebase flatten all the __all__ commits together.

rhiannonlynne commented 1 year ago

It's fine that this doesn't run correctly on the unit tests - we have fixed this on other PRs.

parejkoj commented 1 year ago

Looks like it won't let me merge without those tests passing: I'm about to disappear for a couple of days, so if someone could merge it for me this time, that would be great. It's rebased to main as of now.