New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DM-13246: Synchronize DMTN-056 with Butler prototype #6
Conversation
python/lsst/butler/schema.py
Outdated
Column('visit_begin', Integer, ForeignKey( | ||
'Visit.visit_number'), primary_key=True, nullable=False), | ||
Column('visit_end', Integer, ForeignKey( | ||
'Visit.visit_number'), primary_key=True, nullable=False), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm surprised it makes sense to actually make this a foreign key, given that it's defining a range.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was assuming the range had to be valid. So both need to exist. But perhaps this is not true?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we'll want at least half-open ranges, presumably marked by null endpoints.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, we'll remove the foreign key checks and hope for the best.
Additionally: - Rename tag -> collection - Rename Path -> StorageHint
763b331
to
e84dddf
Compare
Changes ------- - added `ospath` property to ButlerURI that localizes the posix-like uri.path - Updated docstrings in ButlerURI class to correctly describe what properties/methods do. - File vs Dir is now resolved by checking if "." is present in path for both Config and ButlerConfig. If not Dir, path is no longer forced to be the top level directory before updating file name. - Fixed various badly merged Formatter docstrings and file IO. - Removed file extension update call in 'to/from'Bytes' methods in Formatters. - Restored `super().setConfigRoot` call in SqliteRegistry. - Removed extra boto3 present check from test_butler.py - Fixed badly formatted docstrings in s3utils. - Renamed `cheap` kwarg to `slow` in `s3CheckFileExists`.
Changes ------- - added `ospath` property to ButlerURI that localizes the posix-like uri.path - Updated docstrings in ButlerURI class to correctly describe what properties/methods do. - File vs Dir is now resolved by checking if "." is present in path for both Config and ButlerConfig. If not Dir, path is no longer forced to be the top level directory before updating file name. - Fixed various badly merged Formatter docstrings and file IO. - Removed file extension update call in 'to/from'Bytes' methods in Formatters. - Restored `super().setConfigRoot` call in SqliteRegistry. - Removed extra boto3 present check from test_butler.py - Fixed badly formatted docstrings in s3utils. - Renamed `cheap` kwarg to `slow` in `s3CheckFileExists`.
Changes ------- - added `ospath` property to ButlerURI that localizes the posix-like uri.path - Updated docstrings in ButlerURI class to correctly describe what properties/methods do. - File vs Dir is now resolved by checking if "." is present in path for both Config and ButlerConfig. If not Dir, path is no longer forced to be the top level directory before updating file name. - Fixed various badly merged Formatter docstrings and file IO. - Removed file extension update call in 'to/from'Bytes' methods in Formatters. - Restored `super().setConfigRoot` call in SqliteRegistry. - Removed extra boto3 present check from test_butler.py - Fixed badly formatted docstrings in s3utils. - Renamed `cheap` kwarg to `slow` in `s3CheckFileExists`.
Changes ------- - added `ospath` property to ButlerURI that localizes the posix-like uri.path - Updated docstrings in ButlerURI class to correctly describe what properties/methods do. - File vs Dir is now resolved by checking if "." is present in path for both Config and ButlerConfig. If not Dir, path is no longer forced to be the top level directory before updating file name. - Fixed various badly merged Formatter docstrings and file IO. - Removed file extension update call in 'to/from'Bytes' methods in Formatters. - Restored `super().setConfigRoot` call in SqliteRegistry. - Removed extra boto3 present check from test_butler.py - Fixed badly formatted docstrings in s3utils. - Renamed `cheap` kwarg to `slow` in `s3CheckFileExists`.
Changes ------- - added `ospath` property to ButlerURI that localizes the posix-like uri.path - Updated docstrings in ButlerURI class to correctly describe what properties/methods do. - File vs Dir is now resolved by checking if "." is present in path for both Config and ButlerConfig. If not Dir, path is no longer forced to be the top level directory before updating file name. - Fixed various badly merged Formatter docstrings and file IO. - Removed file extension update call in 'to/from'Bytes' methods in Formatters. - Restored `super().setConfigRoot` call in SqliteRegistry. - Removed extra boto3 present check from test_butler.py - Fixed badly formatted docstrings in s3utils. - Renamed `cheap` kwarg to `slow` in `s3CheckFileExists`.
Changes ------- - added `ospath` property to ButlerURI that localizes the posix-like uri.path - Updated docstrings in ButlerURI class to correctly describe what properties/methods do. - File vs Dir is now resolved by checking if "." is present in path for both Config and ButlerConfig. If not Dir, path is no longer forced to be the top level directory before updating file name. - Fixed various badly merged Formatter docstrings and file IO. - Removed file extension update call in 'to/from'Bytes' methods in Formatters. - Restored `super().setConfigRoot` call in SqliteRegistry. - Removed extra boto3 present check from test_butler.py - Fixed badly formatted docstrings in s3utils. - Renamed `cheap` kwarg to `slow` in `s3CheckFileExists`.
No description provided.