search

OSSOS / MOP

The Moving Object Pipeline: discovery & tracking of trans-Neptunian objects
GNU General Public License v3.0
4 stars 9 forks source link

[track] new short object names have spaces at wrong side #288

Closed mtbannister closed 10 years ago

mtbannister commented 10 years ago

Object names have the blank-space padding at the left side instead of the right, preventing track from reading them:

ERROR: MPCFieldFormatError: Field Provisional name: must start with a letter; but was E08 [ossos.mpc] Traceback (most recent call last): File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/validate.py", line 53, in main() File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/validate.py", line 49, in main launch_app(args.task, args.input, output, args.dry_run, args.debug) File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/validate.py", line 18, in launch_app dry_run=dry_run, debug=debug) File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/gui/app.py", line 39, in create_application dry_run=dry_run, debug=debug) File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/gui/app.py", line 212, in init working_directory, output_directory, dry_run=dry_run, debug=debug) File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/gui/app.py", line 94, in init model.start_work() File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/gui/models/validation.py", line 36, in start_work self.next_workunit() File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/gui/models/validation.py", line 79, in next_workunit self._get_new_workunit() File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/gui/models/validation.py", line 109, in _get_new_workunit self.add_workunit(self.workunit_provider.get_workunit()) File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/gui/models/workload.py", line 620, in get_workunit ignore_list=self.fetched_files) File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/gui/models/workload.py", line 566, in get_workunit self.directory_context.get_full_path(potential_file)) File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/gui/models/workload.py", line 691, in build_workunit parsed_data = self.parser.parse(input_fullpath) File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/ssos.py", line 66, in parse mpc_observation = mpc.Observation.from_string(line) File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/mpc.py", line 431, in from_string obsrec = cls(*struct.unpack(mpc_format, mpc_line)) File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/mpc.py", line 397, in init self.provisional_name = provisional_name File "/Users/michele/Dropbox/OSSOS/ossos-pipeline/src/ossos-pipeline/ossos/mpc.py", line 512, in provisional_name provisional_name) ossos.mpc.MPCFieldFormatError: Field Provisional name: must start with a letter; but was E08

ijiraq commented 10 years ago

resolved by commit 36b0e4ffa3cb58f446fb22cf7201b68eccaf886e